Posted	Nick	Remark
#openstack-nova - 2017-05-26
00:54:12	gyee	I think there are two checks there
00:54:46	gyee	first check if recreate and scheduled_node is not none prior to resource_tracker
00:55:19	gyee	then if not recreate and not scheduled_node after that
00:56:41	gyee	the second check covers the scenario where instance is being rebuilt on the same host
00:56:58	mriedem	yup i'm just fixing tests
01:08:19	mriedem	this is going to take some fancy word-smithing in the commit message
01:15:20	mriedem	gyee: https://review.openstack.org/#/c/468227/
01:16:35	gyee	mriedem: we are letting ComputeHostNotFound bubble up?
01:16:50	mriedem	w/o the compute node we can't get the resource tracker and make the claim
01:16:52	mriedem	so we need to fail
01:17:06	mriedem	as noted in the commit message
01:17:10	gyee	make sense
01:17:56	mriedem	that 3rd scenario for forced evacuation was basically totally busted in newton
01:18:01	mriedem	when that microversion was introduced for forced evacuate
01:18:11	mriedem	since it doesn't make the rebuild claim on the target host
01:24:13	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI: nova-compute cannot restart after manually delete VM https://review.openstack.org/467926
01:31:56	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/468089
01:36:32	openstackgerrit	Matt Riedemann proposed openstack/nova master: Avoid lazy-loading instance.id when cross_az_attach=False https://review.openstack.org/468232
01:53:13	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/468089
02:30:04	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
02:39:54	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/468089
03:13:33	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
03:15:02	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
03:29:48	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
03:34:34	openstackgerrit	suzhengwei proposed openstack/nova-specs master: availability zone return all services in api https://review.openstack.org/468261
03:45:36	openstackgerrit	suzhengwei proposed openstack/nova master: availability zone return all services in api https://review.openstack.org/462897
03:52:08	openstackgerrit	suzhengwei proposed openstack/nova-specs master: availability zone return all services in api https://review.openstack.org/468261
05:08:04	openstackgerrit	Rikimaru Honjo proposed openstack/nova master: Calculate stopped instance's disk sizes for disk_available_least https://review.openstack.org/468269
05:18:22	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
05:19:04	openstackgerrit	Chris Friesen proposed openstack/nova master: show flavor info in server details https://review.openstack.org/434514
05:19:11	gmann	alex_xu: can you check this now - https://review.openstack.org/#/c/461728/4
05:24:31	openstackgerrit	Chris Friesen proposed openstack/nova master: show flavor info in server details https://review.openstack.org/434514
05:33:51	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
05:50:54	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI: Remove bittorrent.py which is not used or referred https://review.openstack.org/468273
06:09:54	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
06:19:17	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
06:27:13	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
06:34:34	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
06:34:47	alex_xu	gmann: yea, will try to reach that soon
06:35:05	gmann	alex_xu: thanks
06:35:20	alex_xu	gmann: np
07:08:29	openstackgerrit	Shunli Zhou proposed openstack/nova master: Remove mox from unit/cells/test_cells_messaging.py https://review.openstack.org/448021
07:11:23	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
07:16:15	openstackgerrit	jichenjc proposed openstack/nova master: Enhancement comments on CountableResource https://review.openstack.org/402051
07:16:27	openstackgerrit	jichenjc proposed openstack/nova master: add zvm migration object https://review.openstack.org/414090
07:16:39	openstackgerrit	jichenjc proposed openstack/nova master: Check instance existing before check in mapping https://review.openstack.org/433470
07:20:24	alex_xu	gmann: done
07:21:26	gmann	alex_xu: thanks again
07:21:56	alex_xu	gmann: np
08:07:44	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI: Remove bittorrent.py which is not used or referred https://review.openstack.org/468273
08:12:00	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI: Remove bittorrent.py which is not used or referred https://review.openstack.org/468273
08:41:15	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-virtual-interfaces endpoint https://review.openstack.org/468317
08:41:29	alex_xu	gmann: one comment https://review.openstack.org/#/c/467938/2
08:42:42	gmann	alex_xu: nice catch, ll update
08:44:17	alex_xu	gmann: thanks :)
08:50:08	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
08:57:40	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI: Remove bittorrent.py which is not used or referred https://review.openstack.org/468273
08:59:32	openstackgerrit	Andrea Visnyei proposed openstack/nova master: Parameter verification of os-security-groups https://review.openstack.org/383598
09:03:57	openstackgerrit	Tao Li proposed openstack/nova master: Fix the display of updated_at time when using memcache driver. https://review.openstack.org/414463
09:09:57	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix a context argument in _set_vm_state_and_notify https://review.openstack.org/468322
09:10:52	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-server-external-events endpoint https://review.openstack.org/468324
09:15:09	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-server-external-events endpoint https://review.openstack.org/468324
09:25:49	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for server diagnostics endpoint https://review.openstack.org/468331
09:34:22	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for server ips endpoint https://review.openstack.org/468334
09:36:07	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
09:50:04	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for instance actions endpoint https://review.openstack.org/468338
09:51:13	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
10:05:08	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
10:13:06	andymccr	Morning Nova - I'm seeing gate failures on OSA since https://review.openstack.org/458635 and https://review.openstack.org/458634 merged - essentially the compute host isnt seen as being in the cell yet so it fails to boot an instance - before those patches it works fine. The scheduler is saying the host isnt mapped to a cell. The boot works fine if you try a short while after though, so im guessing its a timi
10:13:06	andymccr	ng issue.
10:13:31	andymccr	is there anything we can change/do to ensure the host is in the cell and we avoid the race? (we're already performing a discover host)
10:23:06	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-fping endpoint https://review.openstack.org/468349
10:24:37	openstackgerrit	Deepak Mourya proposed openstack/nova master: Remove Babel from requirements.txt https://review.openstack.org/468350
10:36:14	openstackgerrit	Sean Dague proposed openstack/nova master: Migrate to oslo request_id middleware - mv 2.46 https://review.openstack.org/467998
10:41:38	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory https://review.openstack.org/455215
10:41:38	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET resource provider https://review.openstack.org/467541
10:41:39	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory https://review.openstack.org/455214
10:41:39	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT resource provider https://review.openstack.org/467542
10:41:40	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory https://review.openstack.org/455213
10:41:41	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for resource classes https://review.openstack.org/467596
10:41:41	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventories https://review.openstack.org/449765
10:41:41	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE resource provider https://review.openstack.org/467543
10:41:42	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventories https://review.openstack.org/452708
10:41:43	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for POST resource provider https://review.openstack.org/456176
10:41:43	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventories https://review.openstack.org/452707
10:47:54	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
10:53:07	openstackgerrit	Sean Dague proposed openstack/nova master: Send request_id on cinder calls https://review.openstack.org/468355
10:53:07	openstackgerrit	Sean Dague proposed openstack/nova master: Send request_id on glance calls https://review.openstack.org/467242
10:53:08	openstackgerrit	Sean Dague proposed openstack/nova master: Send request_id on neutron calls https://review.openstack.org/468356
10:56:56	mriedem	sfinucan: you want to put this veritas volume driver bp patch through? https://review.openstack.org/#/c/443951/ mdbooth and lyarwood have been through it several times.
10:57:18	sfinucan	mriedem: sure
11:00:20	cdent	mriedem: did you see that I de-conflicted your https://review.openstack.org/#/c/454426/ ?
11:00:35	mriedem	ooo i did not
11:01:19	mriedem	thanks, i've got a few older changes i need to address
11:01:31	mriedem	waking up at 5:30 and not being able to get back to sleep might allow me time to do that
11:01:55	mdbooth	mriedem: Get it done before you crash later :)
11:02:38	mriedem	cdent: as for your question, "Out of curiosity do scheduler.client.query.sync_instance_info and scheduler.client.query.update_instance_info eventually result in the same end state?" they are sent in different cases i believe
11:03:12	mriedem	sync_instance_info is called in a periodic in all computes, every minute by default
11:04:12	mriedem	update_instance_info is called on init_host for the compute service and when an instance is changed (when we do claims type stuff i think)
11:04:17	mriedem	it's all very chatty

Earlier   Later