Posted	Nick	Remark
#openstack-nova - 2017-06-07
19:44:49	mriedem	which is https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4568
19:45:35	mriedem	seems we should have stored the compute_nodes.uuid in the host_mappings table,
19:45:46	mriedem	then if we're given a uuid, we could have looked up the host mapping by uuid to find the cell rather than iterate
19:46:15	dansmith	mriedem: you can't, because a hostmapping is for nova-compute not for the ComputeNode right?
19:46:20	dansmith	actually,
19:46:36	dansmith	maybe that's wrong. I guess ironic needs discovery when adding new ironic nodes
19:46:54	mriedem	the host mapping is the the compute node
19:47:00	mriedem	mapped by compute_node.host
19:47:04	mriedem	rather than uuid
19:47:07	mriedem	for whatever reason...
19:47:13	dansmith	my "you can't" statement was in response to your "get by compute uuid, then look up host mapping" comment
19:47:22	dansmith	if you've found it, you need not lookup the hostmapping at that point
19:47:28	mriedem	we can look up the host mapping via compute_node.host
19:47:41	mriedem	sure, i was thinking about the paging across cells thing
19:47:44	dansmith	you can, but you've already found it at that point
19:47:55	dansmith	you don't need the hostmapping if you already know what cell it's in,
19:47:59	dansmith	and if you found it, you know
19:48:06	dansmith	right?
19:48:54	mriedem	idk
19:48:57	mriedem	been up since 4
19:48:58	mriedem	this sucks
19:49:19	mriedem	i'm just going to TODO bomb this change for now
20:05:13	zigo	rm_work: You should ask jamespage about lxd.
20:05:46	rm_work	will do, thanks
20:08:17	melwitt	merge conflict extravaganza!
20:10:40	openstackgerrit	Eric Fried proposed openstack/nova master: Adopts keystoneauth with glance client. https://review.openstack.org/412634
20:11:08	openstackgerrit	Eric Fried proposed openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662
20:11:40	openstackgerrit	Eric Fried proposed openstack/nova master: Add PowerVM to nova support matrix https://review.openstack.org/470999
20:11:53	openstackgerrit	Eric Fried proposed openstack/nova master: Trim the fat from InstanceInfo https://review.openstack.org/471146
20:13:10	openstackgerrit	Eric Fried proposed openstack/nova master: Query deleted instance records during _destroy_evacuated_instances https://review.openstack.org/467774
20:13:22	sdague	efried: +2 on the glanceclient conversion
20:13:35	efried	sdague Sweet, thanks.
20:13:53	sdague	and the one after it with the service token
20:13:55	sdague	all very nice
20:13:57	efried	jenkins just posted supposed merge conflicts on a whole boatload of changes.
20:14:30	sdague	yeh, I went through a lot of +2ed changes yesterday and sent a bunch of stuff to merge
20:14:40	sdague	so that's not super surprising
20:14:44	efried	sdague I can't really take credit - Sarafraj did it all before he got the OSIC punt
20:15:00	sdague	efried: well thanks for carrying it across the finish line
20:15:03	efried	sdague Yeah, but I don't see any real conflicts there. Everything I've been able to rebase with the button.
20:15:19	sdague	yeh, gerrit is super conservative
20:15:25	efried	Is there, like, a certain number of intervening merges that automatically posts merge conflicts?
20:16:50	efried	Anyway, sdague, thanks for all the reviews! Seems to be really helping to clear the queue.
20:16:54	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Handle uuids in os-hypervisors API https://review.openstack.org/471899
20:17:14	openstackgerrit	Gage Hugo proposed openstack/nova master: Admin API Policy contingent on is_admin_project https://review.openstack.org/384148
20:18:58	comstud	mriedem: when you think of shit, you think of comstud?
20:20:42	openstackgerrit	Eric Fried proposed openstack/nova master: remove ugly local import https://review.openstack.org/450787
20:22:14	efried	melwitt They all seem trivially gerrit-button-resolvable.
20:25:32	melwitt	efried: thanks, was just complaining without looking first :) I was in the middle of updating the bottom change anyway
20:28:51	openstackgerrit	melanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246
20:28:52	openstackgerrit	melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244
20:28:52	openstackgerrit	melanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247
20:28:53	openstackgerrit	melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242
20:28:53	openstackgerrit	melanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245
20:28:54	openstackgerrit	melanie witt proposed openstack/nova master: Count server groups to check quota https://review.openstack.org/446240
20:28:54	openstackgerrit	melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
20:28:55	openstackgerrit	melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521
20:28:55	openstackgerrit	melanie witt proposed openstack/nova master: Count networks to check quota https://review.openstack.org/446241
20:28:56	openstackgerrit	melanie witt proposed openstack/nova master: Count server group members to check quota https://review.openstack.org/457859
20:28:56	openstackgerrit	melanie witt proposed openstack/nova master: Add InstanceGroup._remove_members_in_db https://review.openstack.org/457858
20:28:57	openstackgerrit	melanie witt proposed openstack/nova master: Count security groups to check quota https://review.openstack.org/457860
20:28:59	openstackgerrit	melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
20:29:00	openstackgerrit	melanie witt proposed openstack/nova master: Count fixed ips to check quota https://review.openstack.org/457861
20:29:01	openstackgerrit	melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
20:29:02	openstackgerrit	melanie witt proposed openstack/nova master: Count floating ips to check quota https://review.openstack.org/457862
20:31:17	openstackgerrit	melanie witt proposed openstack/nova master: Add functional test for local delete allocations https://review.openstack.org/470578
20:41:28	openstackgerrit	Jianghua Wang proposed openstack/nova master: Fix the race condition with novnc https://review.openstack.org/470798
20:52:36	sdague	huh, yeh, I see on the merge conflict side
20:52:38	sdague	that seems odd
20:54:05	openstackgerrit	Sean Dague proposed openstack/nova master: Handle alternative UEFI boot loader locations on SLES https://review.openstack.org/348394
20:54:23	openstackgerrit	guang-yee proposed openstack/nova master: libvirt: Reconnect volumes and encryptors during a hard reboot https://review.openstack.org/400384
21:03:56	openstackgerrit	Sean Dague proposed openstack/nova master: Add earliest-version tags for stable branch renos https://review.openstack.org/469969
21:04:14	openstackgerrit	Sean Dague proposed openstack/nova master: conf: Resolve TODOs in 'database' https://review.openstack.org/393695
21:04:19	openstackgerrit	Sean Dague proposed openstack/nova master: conf: Fix indentation of database options https://review.openstack.org/443097
21:04:27	openstackgerrit	Sean Dague proposed openstack/nova master: Enhancement comments on CountableResource https://review.openstack.org/402051
21:04:34	openstackgerrit	Sean Dague proposed openstack/nova master: xen: pass Xen console in cmdline https://review.openstack.org/469826
21:04:39	openstackgerrit	Sean Dague proposed openstack/nova master: Remove unused CONF import from placement/auth.py https://review.openstack.org/471716
21:04:47	openstackgerrit	Sean Dague proposed openstack/nova master: nfs fix for xenial images https://review.openstack.org/329466
21:04:54	openstackgerrit	Sean Dague proposed openstack/nova master: Set default disk bus for libvirt+Xen HVM to "xen" https://review.openstack.org/457320
21:05:02	openstackgerrit	Sean Dague proposed openstack/nova master: api-ref: Add tags to server(s) detail response https://review.openstack.org/431149
21:05:12	openstackgerrit	Sean Dague proposed openstack/nova master: Fix html_last_updated_fmt for Python3 https://review.openstack.org/470676
21:06:14	sdague	yeh, the merge conflicts seem to be because some external ci system was pseudo DOSing gerrit
21:06:19	sdague	infra is looking into it
21:07:03	melwitt	ack, thanks for the heads up
21:07:53	efried	Cause at this rate, the gate will take days to catch up.
21:09:12	sdague	efried: no, it was registered as jenkins votes
21:09:22	sdague	so they all need to flush
21:09:37	efried	whee
21:30:56	openstackgerrit	Jay Pipes proposed openstack/nova-specs master: Return alternate allocation requests to scheduler https://review.openstack.org/471927
21:38:18	openstackgerrit	Merged openstack/nova master: XenAPI: use os-xenapi 0.2.0 in nova https://review.openstack.org/453493
21:43:02	openstackgerrit	Matthew Edmonds proposed openstack/nova master: [WIP] Deactivate powervm compute until ready https://review.openstack.org/471773
21:44:11	openstackgerrit	Merged openstack/nova master: libvirt: Check if domain is persistent before detaching devices https://review.openstack.org/441204
21:53:26	openstackgerrit	Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288
21:54:29	mikal	mriedem: do you have any appetite to take a look at a privsep patch sometime? I'm about the upload the latest version.
22:00:07	openstackgerrit	Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288
22:03:02	openstackgerrit	Merged openstack/nova master: placement: use separate tables for projects/users https://review.openstack.org/470645
22:04:37	openstackgerrit	Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
22:10:28	lyarwood	mriedem: so I don't even get a Co-Authored-By in https://review.openstack.org/#/c/441204/ ? :D
22:13:16	melwitt	lyarwood: that's weird. I don't know how/why it changed the author. usually it only changes committer when uploading a new PS for someone
22:15:41	lyarwood	melwitt: yeah no idea, it's really not an issue anyway, I've not being paying attention to it for the last 48 hours so mriedem should get the credit.

Earlier   Later