Posted	Nick	Remark
#openstack-nova - 2017-10-09
21:29:56	melwitt	okay. yes, oslo.db TransactionContextManager
21:30:30	melwitt	well, so we have one for cell0 and one for cell1 in the test env, and I wanted to assert that the notify one is targeted to cell1 and not cell0
21:30:38	melwitt	that's what my thinking was
21:30:52	melwitt	to make sure we're targeted to the right cell
21:31:08	dansmith	which you're still doing in the new patch, L426 of fixtures.py
21:31:30	melwitt	yeah, sorry I misunderstood you. I thought you were talking about the last_ctxt_mgr thing that I took out
21:31:41	dansmith	yeah, I understand what you're doing, I just think it's obscure and an artifact of the fixture that is awkward to test for
21:31:43	melwitt	I noticed that wasn't actually doing anything anymore so I removed it
21:32:04	melwitt	but yeah I think asserting it's not None accomplishes the same thing
21:32:05	dansmith	since we fixed a bunch of stuff, I think we don't need that now, yeah
21:32:55	dansmith	my instance list stuff could avoid another db hit if I could tell what cell a context was targeted for, so .. we might want to consider doing something other than hackery for detecting that at some point
21:34:20	melwitt	yeah. eons ago I suggested putting cell_uuid in the context for something else. so maybe we could do that?
21:34:58	melwitt	when I brought it up it was back when we had the mq client cache thing in compute/rpcapi and we could have done cache lookups with the cell_uuid
21:35:27	dansmith	yeah, that's what I meant
21:37:15	melwitt	yeah, I'm +1 on that
21:43:56	dansmith	gdi gerrit, come on
21:44:27	efried	Is there chatter on -infra about it?
21:44:32	efried	(I just rejoined)
21:44:40	dansmith	yeah
21:46:06	efried	cause yeah
21:46:30	efried	Didn't realize how completely I rely on gerrit to do basically anything (and how much I took it for granted).
21:47:22	openstackgerrit	Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247
21:48:03	dansmith	melwitt: I'm going to run off for a bit but I'll hit those when I'm back if you get them updated
21:48:19	melwitt	dansmith: cool, thanks
21:49:02	efried	Rebase successful, but still complaining of a merge conflict. That's odd, isn't it?
21:49:26	melwitt	if it's Zuul, I think it's posting not-really-merge-conflicts
21:49:50	efried	ah
21:49:54	melwitt	right now things are back to being gated on Jenkins, so I'd only pay attention to that as far as passing/not
21:50:06	openstackgerrit	Kaustubh Phatak proposed openstack/nova master: Fixing: Bug #1722409 vmware network matches wrong substring from vSphere https://review.openstack.org/510702
21:50:07	openstack	bug 1722409 in OpenStack Compute (nova) "nova wrongly attaches different network with similar name" [Undecided,New] https://launchpad.net/bugs/1722409 - Assigned to Kaustubh Phatak (kphatak-pf9)
21:51:36	openstackgerrit	Kaustubh Phatak proposed openstack/nova master: Fixing: Bug #1722409 vmware network matches https://review.openstack.org/510703
21:51:37	openstack	bug 1722409 in OpenStack Compute (nova) "nova wrongly attaches different network with similar name" [Undecided,New] https://launchpad.net/bugs/1722409 - Assigned to Kaustubh Phatak (kphatak-pf9)
22:02:36	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Deprecate file injection https://review.openstack.org/509013
22:02:37	mriedem	sdague: for your morrow ^
22:11:59	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/510324
22:33:30	openstackgerrit	Matt Riedemann proposed openstack/nova master: Move user_data max length check to schema https://review.openstack.org/510704
22:47:49	openstackgerrit	Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239
22:47:49	openstackgerrit	Ed Leafe proposed openstack/nova master: Add alternate hosts https://review.openstack.org/486215
22:47:50	openstackgerrit	Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/510159
22:47:50	openstackgerrit	Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854
22:59:13	melwitt	grr, I thought I came up with a good way to test the scatter gather thing, but SpawnIsSynchronous messes that up. or rather, what I assert is still true without the fix bc it's synchronous
23:12:08	melwitt	hm, it works when I remove SpawnIsSynchronous. I dunno if that makes it susceptible to timing bugs
23:12:42	melwitt	probably.
23:22:44	openstackgerrit	melanie witt proposed openstack/nova master: Fix target_cell usage for scatter_gather_cells https://review.openstack.org/510691
23:22:44	openstackgerrit	melanie witt proposed openstack/nova master: Target context for build notification in conductor https://review.openstack.org/509967
23:22:45	openstackgerrit	melanie witt proposed openstack/nova master: WIP Improve the CellDatabases test fixture and usage https://review.openstack.org/508432
23:22:45	openstackgerrit	melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968
23:29:28	openstackgerrit	melanie witt proposed openstack/nova master: Fix target_cell usage for scatter_gather_cells https://review.openstack.org/510691
23:29:28	openstackgerrit	melanie witt proposed openstack/nova master: Target context for build notification in conductor https://review.openstack.org/509967
23:29:29	openstackgerrit	melanie witt proposed openstack/nova master: WIP Improve the CellDatabases test fixture and usage https://review.openstack.org/508432
23:29:29	openstackgerrit	melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968
#openstack-nova - 2017-10-10
01:20:09	openstackgerrit	Yikun Jiang proposed openstack/nova-specs master: Add pagination and changes since filter support for os-instance-action API https://review.openstack.org/507762
01:36:21	openstackgerrit	jichenjc proposed openstack/nova master: Remove duplicate error info https://review.openstack.org/510719
02:43:10	openstackgerrit	Merged openstack/nova master: rp: de-ORM ResourceProvider.get_by_uuid() https://review.openstack.org/509025
03:02:49	openstackgerrit	Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423
03:02:50	openstackgerrit	Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537
03:02:50	openstackgerrit	Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516
03:02:51	openstackgerrit	Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541
03:02:51	openstackgerrit	Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538
03:02:52	openstackgerrit	Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351
03:02:52	openstackgerrit	Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664
03:02:53	openstackgerrit	Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398
03:02:53	openstackgerrit	Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354
03:09:41	openstackgerrit	Merged openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137
04:31:59	openstackgerrit	jichenjc proposed openstack/nova master: Add description for reousrce class creation https://review.openstack.org/508083
04:50:20	openstackgerrit	Wonil Choi proposed openstack/nova master: libvirt: add a condition to check the same host https://review.openstack.org/510737
06:27:53	openstackgerrit	Merged openstack/nova master: rp: Move RP._get|set_aggregates() to module scope https://review.openstack.org/509026
07:07:12	openstackgerrit	Yikun Jiang proposed openstack/nova-specs master: Add pagination and changes since filter support for os-instance-action API https://review.openstack.org/507762
07:19:22	openstackgerrit	Yikun Jiang proposed openstack/nova-specs master: Add pagination and timestamp filtering support for os-migrations API https://review.openstack.org/506030
07:39:35	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
08:00:14	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache https://review.openstack.org/472773
08:36:45	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/504895
08:44:32	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
08:46:08	ildikov	johnthetubaguy: hi, pretty please take a look at this one: https://review.openstack.org/#/c/463987/ :)
08:47:02	johnthetubaguy	ildikov: I want to get to that ASAP, just running on few downstream deadlines right now I am afraid
08:47:52	ildikov	johnthetubaguy: understood, thanks
08:49:19	openstackgerrit	jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458
09:05:31	bauzas	morning, folks
09:05:55	bauzas	so, I have my daughters with me today
09:05:58	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306
09:24:23	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
10:05:20	openstackgerrit	jichenjc proposed openstack/nova master: check query param for used_limits function https://review.openstack.org/499091
10:15:36	openstackgerrit	Elod Illes proposed openstack/nova master: Functional test: evacuate with no compute https://review.openstack.org/498482
10:18:01	openstackgerrit	melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968
10:18:02	openstackgerrit	melanie witt proposed openstack/nova master: Make expected notifications output easier to read in tests https://review.openstack.org/510823
10:18:02	openstackgerrit	melanie witt proposed openstack/nova master: WIP Improve the CellDatabases test fixture and usage https://review.openstack.org/508432
10:18:45	openstackgerrit	jichenjc proposed openstack/nova master: remove CONF.vendordata_driver https://review.openstack.org/501510
10:20:24	openstackgerrit	jichenjc proposed openstack/nova master: remove CONF.vendordata_driver https://review.openstack.org/501510
11:24:34	cdent	johnthetubaguy: is this dead? https://review.openstack.org/#/c/438936/ and if so can it be abandoned?
11:35:54	openstackgerrit	Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844
11:40:06	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Document the changes to PUT /allocations/{consumer_uuid} https://review.openstack.org/510626
11:40:06	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Enable PUT dict to /allocations/{consumer_uuid} https://review.openstack.org/510625
11:57:47	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051
11:57:47	openstackgerrit	Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410
11:57:48	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Enable PUT dict to /allocations/{consumer_uuid} https://review.openstack.org/510625
11:57:48	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Limit number of attempts to delete allocations https://review.openstack.org/507224
11:57:49	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
11:57:49	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Document the changes to PUT /allocations/{consumer_uuid} https://review.openstack.org/510626
12:21:16	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
12:29:07	cdent	dansmith, jaypipes, edleafe: if you get a chance can you gander upon https://review.openstack.org/#/c/500073/ (and its parents) and confirm that’s in line with your understandings of what we need? is POST /allocations. If it is I’ll tack the docs on the end.

Earlier   Later