Posted	Nick	Remark
#openstack-nova - 2018-02-01
21:18:52	openstackgerrit	Merged openstack/nova master: ProviderTree.get_provider_uuids: Top-down ordering https://review.openstack.org/536624
21:19:02	openstackgerrit	Merged openstack/nova master: set_{aggregates|traits}_for_provider: tolerate set https://review.openstack.org/536625
21:22:16	mriedem	efried: there all your things are merged
21:25:48	openstackgerrit	Merged openstack/nova master: Fix nits in support traits changes https://review.openstack.org/537351
21:25:58	openstackgerrit	Merged openstack/nova master: Mention required traits in the flavors user docs https://review.openstack.org/539300
21:26:02	mriedem	sdague: if you're around, this is a simple docs migration that we missed in pike https://review.openstack.org/#/c/540048/ - content is unchanged except for fixing links; the patch on top cleans up some of the content to modernize it
21:26:13	openstackgerrit	Merged openstack/nova master: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/539654
21:26:16	edleafe	efried: I told you to bribe the people in infra!
21:26:25	openstackgerrit	Merged openstack/nova master: Use util.validate_query_params in list_traits https://review.openstack.org/539407
21:26:38	openstackgerrit	Merged openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162
21:26:51	openstackgerrit	Merged openstack/nova master: Deduplicate service status notification samples https://review.openstack.org/534335
21:27:17	openstackgerrit	Merged openstack/nova master: Log options at debug when starting API services under wsgi https://review.openstack.org/519462
21:28:13	openstackgerrit	Matt Riedemann proposed openstack/nova stable/ocata: Regenerate context during targeting https://review.openstack.org/540145
21:31:44	traskat	hey, i have a question about nova and ceph. how can cinder volume gets resized to the flavor disk size if the VM image gets cloned by cinder?
21:31:51	sdague	mriedem: done
21:34:30	mriedem	sdague: cool thanks
21:34:43	mriedem	wow this is a super latent bug, and goes to show no one uses the IsolatedHostsFilter https://bugs.launchpad.net/nova/+bug/1746483
21:34:44	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Provide support matrix and doc for VGPU https://review.openstack.org/539266
21:34:44	openstack	Launchpad bug 1746483 in OpenStack Compute (nova) "Not able to boot from Volume / Volume snapshot when using isolated_images" [Medium,Triaged]
21:35:12	mriedem	the IsolatedHostsFilter assumes the request spec has a valid image id in it, which isn't the case for boot from volume when you don't provide an image id
21:38:18	openstackgerrit	Merged openstack/nova master: De-duplicate _numa_get_flavor_XXX_map_list https://review.openstack.org/385074
21:38:35	openstackgerrit	Merged openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513717
21:54:45	tssurya	mriedem: regarding the scatter_cells bug thanks for backporting it to pike, we have upgraded to ocata today, so far so good (will keep you posted if we find anything major/interesting)
21:57:11	mriedem	tssurya: which one?
21:57:12	openstackgerrit	Matt Riedemann proposed openstack/nova master: Ensure the JSON-Schema covers the legacy v2 API https://review.openstack.org/540154
21:57:51	tssurya	mriedem : https://bugs.launchpad.net/nova/+bug/1746509
21:57:53	openstack	Launchpad bug 1722404 in OpenStack Compute (nova) ocata "duplicate for #1746509 Database transactions can fail with "TypeError: Can't upgrade a READER transaction to a WRITER mid-transaction" because of scatter_gather_cells" [Undecided,In progress] - Assigned to Matt Riedemann (mriedem)
21:57:59	tssurya	the one you mentioned in the meeting
21:58:48	mriedem	ah that's not scatter/gather
21:59:05	mriedem	and the ocata backport isn't ready yet, unit tests are failing, i haven't dug into how to fix those yet
21:59:35	mgagne	mriedem: I don't use IsolatedHostsFilter because you would need to redeploy your config to add new hosts/images, that's suboptimal.
22:00:18	mgagne	but good point about image_ref not being available with BFV, I'm not sure how we didn't catch that one in a similar filter we wrote
22:02:37	mriedem	the request spec is also not good at telling anything if it's boot from volume or not
22:03:10	mriedem	it doesn't even have the original bdms requested in it
22:03:21	mgagne	I would have to dig into it a bit
22:03:26	mriedem	so you'd basically have to assume that if the request spec doesn't have an image, it's volume-backed
22:04:00	mriedem	maybe bauzas has thoughts on that one
22:07:48	mriedem	rgerganov: can you take a look at this? (or someone from vmware?) https://bugs.launchpad.net/nova/+bug/1744182
22:07:49	openstack	Launchpad bug 1744182 in OpenStack Compute (nova) "can not create instance when using vmware nova driver" [Undecided,New]
22:09:40	melwitt	mriedem: commented on the review, I think yielding the copy of the context won't help because the callers aren't using the yielded context
22:10:09	melwitt	I'm not sure if there's any clever thing we can do instead of having to fix all the callers in ocata
22:11:20	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add functional tests for traits-based scheduling https://review.openstack.org/539310
22:12:02	mriedem	oh right...
22:12:07	mriedem	guh
22:12:14	melwitt	yeah :(
22:14:44	melwitt	mriedem: looks like you've already documented the lack of neutron event in certain situations https://github.com/openstack/nova/blob/stable/ocata/nova/virt/libvirt/driver.py#L7508-L7512
22:15:51	mriedem	that's the 2nd time i've looked at that today for different reasons
22:16:24	mriedem	i don't remember writing that though
22:16:28	mriedem	might have been blacked out
22:16:51	melwitt	yeah. ugh, it's my fault I even told lyarwood not to pass vifs_already_plugged=True :(
22:17:17	melwitt	suck
22:19:05	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add functional tests for traits-based scheduling https://review.openstack.org/539310
22:19:06	openstackgerrit	Matt Riedemann proposed openstack/nova master: doc: placement upgrade notes for queens https://review.openstack.org/539658
22:19:22	mriedem	alex_xu: gibi: had to rebase that approved functional test ^
22:20:00	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
22:20:27	mriedem	time to leave, i've got single parenting duty tonight
22:30:10	openstackgerrit	melanie witt proposed openstack/nova master: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540168
22:33:53	openstackgerrit	Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648
22:33:54	openstackgerrit	Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246
22:33:54	openstackgerrit	Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821
22:33:55	openstackgerrit	Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517
22:33:55	openstackgerrit	Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
22:33:56	openstackgerrit	Eric Fried proposed openstack/nova master: Make generation optional in ProviderTree https://review.openstack.org/539324
22:34:18	efried	jaypipes: ^ rebased and now tagged under bp update-provider-tree
22:42:47	jaypipes	efried: cool, thanks. will review shortly.
22:42:50	jaypipes	after dinner...
22:42:59	efried	thx
22:47:45	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
22:48:07	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289
22:48:29	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Move body examples to an isolated directory https://review.openstack.org/529998
22:49:22	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
22:49:42	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201
22:50:01	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
22:50:16	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
22:51:52	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258
23:22:40	openstackgerrit	Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax (Rocky) https://review.openstack.org/540179
23:22:52	efried	mriedem_parent, jaypipes: Re-proposed ^
23:23:09	efried	dansmith too - this was your favorite spec
23:32:39	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
23:55:56	openstackgerrit	Merged openstack/nova master: conf: Validate '[api] vendordata_providers' options https://review.openstack.org/513318
#openstack-nova - 2018-02-02
00:11:52	Spazmotic	Morning novers
00:39:10	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
01:16:09	openstackgerrit	Merged openstack/nova master: Fix invalid UUIDs in remaining tests https://review.openstack.org/539254
01:37:31	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
01:57:57	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Provide support matrix and doc for VGPU https://review.openstack.org/539266
02:02:12	openstackgerrit	Merged openstack/nova master: Updated from global requirements https://review.openstack.org/539841
02:02:43	openstackgerrit	Merged openstack/nova master: trivial: Fix few policy doc https://review.openstack.org/539780
02:02:56	openstackgerrit	Merged openstack/nova master: Add server filters whitelist in server api-ref https://review.openstack.org/527042
02:22:49	Spazmotic	Hey jianghuaw if you get a chance today could you take a look at Bob's comment on the Live Migration commit? If he's correct here we can just remove almost everythign except the swallow on this
02:23:05	Spazmotic	Hopefully in that case with such a simple fix we can get it pushed through the gate quickly and move on with our lives for right now :D
02:27:44	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258
02:46:25	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Improve performance when list instances with IP filter https://review.openstack.org/539469
02:50:03	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
02:54:54	jianghuaw	Spazmotic, for OpenStack live-migration, the VDI_NOT_IN_MAP exception was found in the XS7 and forwarding releases.
02:55:14	Spazmotic	So no way for it to get thrown previous to that?
02:55:18	jianghuaw	Spazmotic, but the error type was added long ago: https://github.com/xapi-project/xen-api/commit/6eaad3b6745c0392ac3b3ef91c33451b1f110aed
02:55:35	Spazmotic	That was my concern.. can we assume it is possible for it to trigger legitimately still outside of that path?
02:56:04	jianghuaw	So I wondering there is case to trigger this error somehow; but I don't know how to trigger it in OpenSTack.
02:56:11	Spazmotic	hehe same here..
02:56:37	Spazmotic	I have access to some 6.5 test boxes, I could run some fake asserts directly through Python i guess

Earlier   Later