Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-13
21:42:41 edleafe mriedem: since we're only expecting db_exc.DBDuplicateEntry, not a DBDeadlock
22:11:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova-status check for ironic flavor migration https://review.openstack.org/527541
22:26:25 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Don't persist could-be-stale InstanceGroup fields in RequestSpec https://review.openstack.org/527799
22:26:35 dansmith mriedem: this is what I'm thinking for the functional change ^
22:31:01 dansmith except with less obvious bugs
22:31:26 mriedem one question, but seems ok at first glance
23:06:53 openstackgerrit Dan Smith proposed openstack/nova master: Don't persist could-be-stale InstanceGroup fields in RequestSpec https://review.openstack.org/527799
23:13:55 jaypipes mriedem: that's odd... looks like multiple threads are all trying to insert the standard traits at the same time.
23:42:12 openstackgerrit melanie witt proposed openstack/nova master: Optionalize instance_uuid in console_auth_token_get_valid() https://review.openstack.org/481700
23:42:13 openstackgerrit melanie witt proposed openstack/nova master: Add console connection object https://review.openstack.org/320063
23:42:13 openstackgerrit melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381
23:42:14 openstackgerrit melanie witt proposed openstack/nova master: Use ConsoleConnection object to generate authorizations https://review.openstack.org/325414
23:42:14 openstackgerrit melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990
23:46:31 melwitt grr looks like I did something wrong to set the tempest conf option
23:48:38 melwitt need dashes instead of underscores ... okay
#openstack-nova - 2017-12-14
00:00:41 openstackgerrit Dan Smith proposed openstack/nova master: Don't persist could-be-stale InstanceGroup fields in RequestSpec https://review.openstack.org/527799
00:15:40 openstackgerrit melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990
00:29:50 openstackgerrit Merged openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473
00:59:03 openstackgerrit Merged openstack/nova master: Add instance action db and obj pagination support. https://review.openstack.org/526422
01:16:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Pass mountpoint to volume attachment_update https://review.openstack.org/527468
01:16:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Update and complete volume attachments during resize https://review.openstack.org/527228
01:16:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Pass bdms to versioned notifications during finish_revert_resize https://review.openstack.org/527446
01:16:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Pass mountpoint to volume attachment_create with connector https://review.openstack.org/527479
01:16:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove unnecessary connector stash in attachment_update https://review.openstack.org/527484
01:17:39 openstackgerrit Merged openstack/nova master: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/527564
01:41:00 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
01:47:39 yikun ^ rebase 2.57 --> 2.58
01:59:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
02:02:45 mriedem yikun: on it
02:03:56 yikun mriedem, :), thanks
02:05:37 mriedem yikun: you dropped something from the rest api history doc
02:07:24 yikun mr
02:07:39 yikun mriedem, oops, my bad, I will address soon
02:09:51 mriedem yikun: i can get it from gerrit
02:10:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
02:10:26 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
02:14:34 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
02:19:34 yikun mriedem, another typo fix, so, I update again...now I think it's a final one, ╮(╯▽╰)╭
02:20:13 Kevin_Zheng https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L391 are we ever going to set this to False?
02:24:12 openstackgerrit Matt Riedemann proposed openstack/nova master: Retry _trait_sync on deadlock https://review.openstack.org/527836
02:28:02 mriedem Kevin_Zheng: we can, but it has to be on a microversion
02:28:21 Kevin_Zheng Yeah, thats what I thought
02:29:04 gmann i feel if we do then we can do all together for query things of all APIs
02:30:12 mriedem yikun: got it, thanks
02:30:27 Kevin_Zheng Yeah, so maybe we should add them to all APIs first :)
02:30:34 Kevin_Zheng still alot missing?
02:30:59 mriedem Kevin_Zheng: there is a blacklist of filters that we don't allow
02:31:11 Kevin_Zheng yeah, I know
02:31:35 Kevin_Zheng Just curious :)
02:31:39 mriedem https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L295
02:31:45 mriedem because those will 500
02:33:22 mriedem and this https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L187
02:33:30 mriedem if you try filtering on private properties in the data model
02:38:49 mriedem yikun: looks like http://logs.openstack.org/26/326326/48/check/openstack-tox-functional/496db26/testr_results.html.gz is failing
02:39:01 mriedem i'm guessing it's using an older server create sample which has personality files in it?
02:39:07 Kevin_Zheng https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L186
02:39:25 Kevin_Zheng ^ JOINED_TABLE_QUERY_PARAMS_SERVERS also return 400
02:39:30 Kevin_Zheng not 500
02:39:45 gmann mriedem: is this looks fine now (server whitelist doc) - https://review.openstack.org/#/c/527042/3
02:39:47 mriedem Kevin_Zheng: right, i mean,
02:39:53 mriedem before that blacklist, those would be a 500
02:40:03 Kevin_Zheng Ah ok
02:55:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete https://review.openstack.org/525231
03:15:05 yikun mriedem, a bit confusing, we set version=2.58 (https://review.openstack.org/#/c/326326/48/nova/tests/functional/api_sample_tests/test_instance_actions.py@137), why call 2.1 API?
03:16:27 mriedem yikun: it's calling with 2.58
03:16:34 mriedem but using a 2.1 sample i think
03:16:39 Kevin_Zheng yeah
03:16:41 mriedem and the 2.1 sample has 'personality' files in it,
03:16:45 mriedem which aren't supported past 2.57
03:17:04 Kevin_Zheng _post_server() reads old samples by default
03:17:15 mriedem i think it's sending this https://github.com/openstack/nova/blob/master/doc/api_samples/servers/server-create-req.json#L13
03:17:30 mriedem yikun: you need it to send this request https://github.com/openstack/nova/blob/master/doc/api_samples/servers/v2.57/server-create-req.json
03:17:45 mriedem is there a parameter in _post_server() to override the sample to use?
03:18:48 mriedem i think it might not work if you override it though if you don't have a server-create-req.json sample in the 2.57 directory
03:18:49 mriedem you'd have to try it
03:19:02 mriedem otherwise, there is an easier way
03:19:10 mriedem don't use a sample to create the server
03:19:17 mriedem just use self.api.post_server(...)
03:19:41 mriedem https://github.com/openstack/nova/blob/master/nova/tests/functional/api/client.py#L271
03:19:50 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref https://review.openstack.org/527042
03:19:57 Kevin_Zheng just set use_common_server_api_samples to false
03:20:38 mriedem you can build the request using https://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L140
03:20:40 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref https://review.openstack.org/527042
03:20:54 mriedem i think use_common_server_api_samples=False might make it look in the 2.57 directory for the sample
03:20:55 mriedem not sure
03:21:41 Kevin_Zheng yeah, have to add a new sample file
03:27:03 gmann mriedem: yikun : if use_common_server_api_samples=False, it will use 'cls.sample_dir + /microversion' to find sample files. which is all set in https://review.openstack.org/#/c/326326/48/nova/tests/functional/api_sample_tests/test_instance_actions.py@137
03:27:25 gmann but you need to put server req and res sample in that dir
03:27:37 openstackgerrit Dai Dang Van proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
03:27:38 yikun yes, use_common_server_api_samples=False look in the dir 2.58. :), and not found if we don't add it
03:27:54 gmann current logic does not fallback to previous version directory, it has to be present in all directory you want to tests
03:27:54 mriedem yeah that's why i'm saying,
03:27:57 mriedem don't use a sample,
03:28:00 mriedem just create the server directly
03:28:06 mriedem https://github.com/openstack/nova/blob/master/nova/tests/functional/api/client.py#L271
03:28:09 mriedem https://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L140
03:28:19 mriedem the test only cares about the actions, not how the server is created
03:28:28 gmann yea, +1
03:28:34 yikun yea, got it
03:28:37 mriedem can i get a hell yeah?!

Earlier   Later