Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-14
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 mriedem yeah that's why i'm saying,
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: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?!
03:29:15 Kevin_Zheng everyday is a long day for mriedem
03:31:22 openstackgerrit Matthew Oliver proposed openstack/nova master: Update the obs placement-api port https://review.openstack.org/527846
04:02:48 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
05:31:46 openstackgerrit zhufl proposed openstack/nova master: Fix some typos in nova doc https://review.openstack.org/527856
06:55:23 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Updated common create server sample request because of microversion 2.57 https://review.openstack.org/527878
06:57:07 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Updated common create server sample request because of microversion 2.57 https://review.openstack.org/527878
07:10:18 openstackgerrit Ya Lian Pan proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
07:32:54 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
07:42:18 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
07:44:22 openstackgerrit Ya Lian Pan proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
07:59:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
08:01:41 openstackgerrit Chason Chan proposed openstack/nova master: Fix the bug report link of API Guide https://review.openstack.org/527660
09:00:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263
09:00:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3) https://review.openstack.org/526557
09:17:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
09:30:11 openstackgerrit Dai Dang Van proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047
09:36:00 openstackgerrit Ya Lian Pan proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
10:01:09 openstackgerrit Matthew Booth proposed openstack/nova master: Improve error message on invalid BDM fields https://review.openstack.org/527915
10:02:28 openstackgerrit Matthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346
10:02:28 openstackgerrit Matthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr https://review.openstack.org/524167
10:02:29 openstackgerrit Matthew Booth proposed openstack/nova master: Use real block_device_info data in test_blockinfo https://review.openstack.org/527916
10:02:29 openstackgerrit Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347
10:16:10 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix possible TypeError in VIF.fixed_ips https://review.openstack.org/527920
10:21:19 openstackgerrit Balazs Gibizer proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119
10:21:20 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617
10:21:20 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380
10:21:21 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982
10:21:21 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633
10:21:22 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add aggregates check in allocation candidates https://review.openstack.org/522407
10:21:22 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189
10:21:23 openstackgerrit Balazs Gibizer proposed openstack/nova master: placement: func tests for multiple shared RPs https://review.openstack.org/498737
10:21:23 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix missing rps in allocation candidates https://review.openstack.org/522409
10:23:41 gibi jaypipes: ^^ rebased the series as master already has the fix for the frequently failing functional test that prevented these to be merged
10:26:48 openstackgerrit rahul bardia proposed openstack/python-novaclient master: nova limits ERROR (Exception): Field names must be unique This happens when 'Others' field is occuring more than once which is not handled in code causing multiple columns having same name. Which throws above exception and does not pretty prints absolute https://review.openstack.org/527922
11:11:50 kashyap stephenfin: Hi there, when you get a moment, you know why is Zuul complaning here - https://review.openstack.org/#/c/527691/
11:21:35 stephenfin kashyap: Looking
11:22:45 stephenfin kashyap: That job's been failing a lot lately, but I don't know why. It's unrelated to this, in any case
11:23:03 stephenfin We should probably investigate that, actually
11:24:38 stephenfin bauzas: Ta for the review :)
11:25:32 kashyap stephenfin: Yeah, noted; right now I need to run an errand.
11:34:45 openstackgerrit Merged openstack/nova stable/ocata: Add regression test for rebuilding a volume-backed server https://review.openstack.org/525664
11:41:42 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095
11:41:43 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242
11:41:43 openstack bug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)
11:51:35 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
11:51:36 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459
12:01:57 gibi kashyap, stephenfin: the test_live_migration_actions functional test failure has already been fixed on master so you just have to rebase
12:02:22 stephenfin gibi: Aha. What patch fixed it, out of curiosity?
12:02:23 gibi kashyap, stephenfin: this was the fix https://review.openstack.org/#/c/527440/
12:02:32 stephenfin jinx :)_
12:02:54 stephenfin Oh, nice
12:03:01 gibi stephenfin: there is one more fix in the pipe that fixes another failure mode of that test here https://review.openstack.org/#/c/527444/
12:04:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Extract instance allocation removal code https://review.openstack.org/513041
12:35:32 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Enable limiting GET /allocation_candidates https://review.openstack.org/513526
12:40:44 openstackgerrit Chason Chan proposed openstack/nova master: Fix the bug report link of API Guide https://review.openstack.org/527660
12:43:31 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Modify signature of _filter_non_requested_pfs https://review.openstack.org/527473
12:43:31 openstackgerrit Stephen Finucane proposed openstack/nova master: Add PCI NUMA policies https://review.openstack.org/527472
12:44:30 efried_cya_jan stephenfin gibi Thanks for looking after those patches!
12:45:26 cdent efried_cya_jan: since you're here (naughty!) I should report that I've not had a chance to do anything with your stuff: I've got a nasty cold so mostly done nothing for the past couple of days. But when I surface, I will.
12:46:17 efried_cya_jan cdent I somehow caught your cold, which is a sucky way to start a vacation. It looks like stephenfin has been addressing review comments and he & gibi have been rebasing stuff, so I think you're in the clear... for now :)
12:46:35 cdent internet germs are the worst
12:46:49 gibi efried_cya_jan: o/
12:48:19 openstackgerrit Raghad Qutteneh proposed openstack/nova master: Allow force-delete to delete instance in task_state block_device_mapping https://review.openstack.org/527951
12:53:13 gibi efried_cya_jan: if you are here I have a question. Do I understand correctly that the GET allocation_candidate API does not handle nested RPs yet?

Earlier   Later