Posted	Nick	Remark
#openstack-nova - 2018-07-19
04:35:22	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Use Tempest scenario job to run all scenario tests https://review.openstack.org/583840
05:13:24	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix ServerMigrationSampleJsonTests to use sample files from version dir https://review.openstack.org/582702
05:47:39	openstackgerrit	fupingxie proposed openstack/nova master: Delete allocations when it is re-allocated https://review.openstack.org/582899
06:33:40	openstackgerrit	Chen proposed openstack/python-novaclient master: Fix inconsistency https://review.openstack.org/572770
06:35:12	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge server create for config_drive extension https://review.openstack.org/583538
06:38:45	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge server create for bdm extensions https://review.openstack.org/583559
06:42:54	openstackgerrit	Chen proposed openstack/python-novaclient master: Fix inconsistency https://review.openstack.org/572770
06:51:11	openstackgerrit	Chen proposed openstack/nova master: Fix soft deleting vm fails after "nova resize" vm https://review.openstack.org/546920
07:48:25	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge server create for bdm extensions https://review.openstack.org/583559
08:20:44	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Remove stevedore extensions server_create method https://review.openstack.org/583882
08:22:58	openstackgerrit	Stephen Finucane proposed openstack/os-vif master: add upper_constraints support https://review.openstack.org/583582
08:25:33	openstackgerrit	fupingxie proposed openstack/nova master: Delete allocations when it is re-allocated https://review.openstack.org/582899
08:31:21	openstackgerrit	sahid proposed openstack/nova master: hardware: fix memory check usage for small/large pages https://review.openstack.org/532168
08:40:41	openstackgerrit	Vishakha Agarwal proposed openstack/nova master: 'Updated_at' is NULL when show aggregate info https://review.openstack.org/580271
09:17:20	openstackgerrit	Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
09:17:21	openstackgerrit	Lei Zhang proposed openstack/nova master: Docs for reporting CPU traits feature https://review.openstack.org/582902
09:46:04	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] disallow additional fiels in allocations https://review.openstack.org/583907
09:46:30	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] disallow additional fields in allocations https://review.openstack.org/583907
10:57:35	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add regression test for bug 1781880 https://review.openstack.org/583927
10:57:36	openstack	bug 1781880 in OpenStack Compute (nova) "nova service-list for new compute service causes TypeError with servicegroup_driver mc " [Medium,Triaged] https://launchpad.net/bugs/1781880 - Assigned to Zhenyu Zheng (zhengzhenyu)
11:03:07	openstackgerrit	Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
11:12:03	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add regression test for bug 1781880 https://review.openstack.org/583927
11:12:04	openstack	bug 1781880 in OpenStack Compute (nova) "nova service-list for new compute service causes TypeError with servicegroup_driver mc " [Medium,Triaged] https://launchpad.net/bugs/1781880 - Assigned to Zhenyu Zheng (zhengzhenyu)
11:41:35	openstackgerrit	Merged openstack/os-vif master: add upper_constraints support https://review.openstack.org/583582
11:47:12	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add regression test for bug 1781880 https://review.openstack.org/583927
11:47:14	openstack	bug 1781880 in OpenStack Compute (nova) "nova service-list for new compute service causes TypeError with servicegroup_driver mc " [Medium,Triaged] https://launchpad.net/bugs/1781880 - Assigned to Zhenyu Zheng (zhengzhenyu)
11:50:42	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Fix nova service-list for new compute service causes TypeError https://review.openstack.org/583936
12:13:10	openstackgerrit	do3meli proposed openstack/nova master: docs: add nova host-evacuate command to evacuate documentation https://review.openstack.org/578040
12:51:30	sean-k-mooney	melwitt: FYI https://review.openstack.org/#/c/583319/ should be ready now. i have merged the patch in os-vif so zull should now pass when the gate job runs
12:59:39	efried	mriedem: https://review.openstack.org/#/c/577863/ may be what you're looking for.
13:02:13	mriedem	aye
13:09:18	tssurya	mriedem: are there any known bugs linking "Limiting Placement Allocation Candidates" with "rebuild" ?
13:10:58	sean-k-mooney	mriedem: oh you online. can you take a look at the os-vif release patch when you get a change https://review.openstack.org/#/c/583319/
13:11:10	mriedem	tssurya: can you be more specific?
13:11:16	efried	tssurya: What do you mean, targeting specific hosts or just using the limit= qparam?
13:11:29	efried	The latter should be happening already
13:11:30	tssurya	mriedem, efried: targeting specific hosts
13:11:39	mriedem	https://review.openstack.org/#/c/576693/
13:11:45	mriedem	needs +W
13:11:48	mriedem	that's not rebuild though
13:12:17	tssurya	rebuild is reschedule right ? for new images ?
13:12:27	mriedem	yes, then you're looking for https://review.openstack.org/#/q/Ie252271ecfd38a0a1c61c26e323cc03869889f0a
13:12:30	efried	tssurya: This one is related (though not quite a match) https://bugs.launchpad.net/nova/+bug/1777591
13:12:31	openstack	Launchpad bug 1777591 in OpenStack Compute (nova) "‘limit’ in allocation_candidates where sometimes make force_hosts invalid" [High,In progress] - Assigned to xulei (605423512-j)
13:12:34	sean-k-mooney	tssurya: i did not think we allowed you to target a specific host with rebild. we dont allow you to bypass the scudler in anycase even if we do
13:12:43	mriedem	sean-k-mooney: we don't,
13:12:48	mriedem	but we run through the scheduler to validate the new image
13:12:56	mriedem	on the existing instance.host
13:12:56	tssurya	ah thanks, looks like that is what I am looking for,
13:13:07	efried	oh, I didn't realize we had a fix up for that bug mriedem, looking now...
13:13:38	sean-k-mooney	mriedem: in case i request something in the extraspecs that would make it invalid for that host im guessing
13:13:54	mriedem	s/extraspec/image properties/ yes
13:14:06	tssurya	we keep hitting https://github.com/openstack/nova/blob/f7e6d721f586ecbfee08c621f9e10174a4c6b35d/nova/scheduler/host_manager.py#L521 for new images
13:14:06	mriedem	could be a windows image and you're on a linux host
13:14:12	mriedem	well, not really that, but you get the idea
13:14:20	mriedem	or something that violates aggregates
13:14:21	sean-k-mooney	mriedem: ya
13:14:40	mriedem	tssurya: that's not rebuild then...
13:14:44	mriedem	you can't force_hosts/force_nodes on rebuild
13:14:59	mriedem	unless it's coming out of the request spec
13:15:04	mriedem	from when the instance was originally built
13:16:01	sean-k-mooney	mriedem: for that to happen you would have first had to force the host on inital boot though right?
13:16:02	mriedem	we don't call placement on rebuild
13:16:37	tssurya	mriedem: we don't ?
13:16:44	mriedem	no, see https://review.openstack.org/#/c/546357/5/nova/scheduler/manager.py
13:17:10	mriedem	for a rebuild, quantitative aspects like the flavor aren't changing,
13:17:20	mriedem	so we run qualitative filters and things that depend on the image
13:17:56	tssurya	hmm, checking prod code to see if this change is there
13:18:04	mriedem	i guess the api does set force_hosts during rebuild if you're rebuilding with a new image,
13:18:16	mriedem	that's what tells the scheduler to run the filters on that exact host that the instance is already running on
13:19:14	tssurya	awesome the above fix is not there in our queens
13:19:26	mriedem	i thought you guys pulled from stable?
13:19:29	tssurya	that's why rebuild is still going to placement
13:19:52	tssurya	mriedem: we do, but we have some version stuff and backports from rocky which we should not have
13:19:58	tssurya	and then it gets tricky
13:20:06	mriedem	ack
13:20:12	mriedem	well problem solved :)
13:20:19	tssurya	(for instance dansmith's placement-request filter)
13:20:24	tssurya	yea, thanks a ton!
13:21:17	efried	mriedem: I'm fixing my nits
13:22:02	openstackgerrit	Eric Fried proposed openstack/nova master: Disable limits if force_hosts or force_nodes is set https://review.openstack.org/576693
13:22:06	efried	mriedem: ^
13:24:08	mriedem	sean-k-mooney: your os-vif release doesn't seem like it needs to be a minor bump
13:24:12	mriedem	it's mostly test and docs changes
13:25:11	sean-k-mooney	i could make it 1.10.1 but made it 1.11.0 because i adde the noop plugin that said its a noop plugin...
13:25:41	mriedem	yeah i noted that,
13:25:49	mriedem	i guess there is more to that than what i first saw in the commit log
13:26:15	sean-k-mooney	do you want me to make it 1.10.1 instead?
13:26:21	mriedem	no i just +1ed
13:27:12	sean-k-mooney	mriok cool. well the real reason for the noop plugin is to unblock https://review.openstack.org/#/c/565471/
13:28:10	mriedem	efried: isn't https://review.openstack.org/#/c/576693/7..8/nova/tests/unit/scheduler/test_utils.py needed for the assertResourceRequestsEqual call?
13:28:13	sean-k-mooney	mriedem: ^ that was ment for you
13:28:47	mriedem	oh i guess probably not,
13:28:58	mriedem	assertResourceRequestsEqual is about comparing the request to the flavor right?
13:29:09	mriedem	i thought it was weird that they had to set expected.is_limit
13:29:14	mriedem	sean-k-mooney: yeah
13:29:54	sean-k-mooney	im not sure we would want to depend on a z stream release to delete that code
13:30:15	mriedem	sean-k-mooney: it's likely not going to happen in stein anyway - that removal from nova
13:30:19	mriedem	*rocky
13:31:16	sean-k-mooney	oh ok well in either case its not blocked anymore so it can be done when it makes sense

Earlier   Later