Posted	Nick	Remark
#openstack-nova - 2018-09-12
22:43:30	gmann	i am trying to get tempest-full job fix in since 4.50 AM but random failure there.
22:48:35	mriedem	gmann: yeah this is killing us http://status.openstack.org/elastic-recheck/#1783405
22:48:53	mriedem	and http://status.openstack.org/elastic-recheck/#1463631
23:03:26	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Fix mock.patch usage in unit tests https://review.openstack.org/602198
23:04:00	gibi	an easy unit test fix removing interference ^^
23:07:17	melwitt	gibi: LGTM but not sure I understand your commit message. what bad thing happens because of the bad usage?
23:08:24	openstackgerrit	John Garbutt proposed openstack/nova-specs master: Add Unified Limits Spec https://review.openstack.org/602201
23:09:12	openstackgerrit	Tetsuro Nakamura proposed openstack/nova stable/rocky: [placement] split gigantor SQL query, add logging https://review.openstack.org/602202
23:40:53	openstackgerrit	Ghanshyam Mann proposed openstack/nova stable/rocky: DNM: testing greande job depends-on things https://review.openstack.org/602207
#openstack-nova - 2018-09-13
00:35:20	openstackgerrit	Merged openstack/nova master: Update RequestSpec.flavor on resize_revert https://review.openstack.org/588689
00:40:38	openstackgerrit	Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276
02:31:23	openstackgerrit	Chen proposed openstack/nova master: Fix evacuate logging https://review.openstack.org/593055
02:34:04	openstackgerrit	Chen proposed openstack/nova master: Fix evacuate logging https://review.openstack.org/593055
02:54:13	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Fix mock.patch usage in unit tests https://review.openstack.org/602198
02:56:14	gibi	melwitt: replied in the review
03:11:33	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): Add API to support vgpu resource provider create https://review.openstack.org/520313
03:17:11	openstackgerrit	fupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node https://review.openstack.org/602219
04:32:23	openstackgerrit	Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276
05:02:48	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): Add API to support vgpu resource provider create https://review.openstack.org/520313
05:30:12	openstackgerrit	Tony Breeds proposed openstack/python-novaclient stable/queens: import zuul job settings from project-config https://review.openstack.org/601400
05:30:12	openstackgerrit	Tony Breeds proposed openstack/python-novaclient stable/queens: Switch to stestr https://review.openstack.org/601933
05:47:41	openstackgerrit	fupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node https://review.openstack.org/602219
06:14:54	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): Add API to support vgpu resource provider create https://review.openstack.org/520313
07:01:06	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/601047
07:17:42	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
07:19:00	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
07:33:37	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): support compute node resource provider update https://review.openstack.org/521041
07:41:31	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
07:42:15	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
07:42:27	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): support compute node resource provider update https://review.openstack.org/521041
07:49:41	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041
07:50:54	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
07:51:13	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041
11:50:26	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova-specs master: Support initial allocation ratios https://review.openstack.org/552105
12:36:26	openstackgerrit	Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276
13:15:17	openstackgerrit	Hamdy Khader proposed openstack/nova master: Set defult value of num_nvme_discover_tries=5 https://review.openstack.org/602351
13:34:59	openstackgerrit	Claudiu Belu proposed openstack/nova master: conductor: add live_resize task https://review.openstack.org/248579
13:35:00	openstackgerrit	Claudiu Belu proposed openstack/nova master: API: Adds instance live-resize https://review.openstack.org/184584
14:01:10	openstackgerrit	Claudiu Belu proposed openstack/nova master: WIP: replace spec with autospec https://review.openstack.org/557299
14:38:03	litao__	hi
14:39:15	litao__	This is my patch:https://review.openstack.org/#/c/592252/ I need some reviews , thanks
14:40:48	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
14:40:55	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
14:42:40	kashyap	litao__: Hi, many folks are at a conference this week, so expect some delays in responses
14:45:43	mriedem	dansmith: this was the thing i mentioned at breakfast re resize reschedule up-calls; do the proposed solutions inline make sense? https://blueprints.launchpad.net/nova/+spec/fix-reschedule-up-calls
14:50:22	mriedem	bauzas: here is the fugly cross_az_attach=False bug fix i mentioned yesterday https://review.openstack.org/#/c/469675/
14:50:49	mriedem	note that i plan on writing an alternate, less tightly coupled approach to fixing this laid out in https://review.openstack.org/#/c/469675/4/nova/compute/api.py@918
14:53:07	lyarwood	bauzas: mind taking a look at https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1787606 this morning if you get a chance
14:53:27	bauzas	mriedem: ta, will look at both
14:54:37	mriedem	edleafe: you might want to take a look at https://review.openstack.org/#/c/593074/ from lyarwood above
14:54:38	openstackgerrit	Claudiu Belu proposed openstack/nova master: conductor: add live_resize task https://review.openstack.org/248579
14:54:39	openstackgerrit	Claudiu Belu proposed openstack/nova master: API: Adds instance live-resize https://review.openstack.org/184584
14:54:42	bauzas	lyarwood: yup, saw your change yesterday
14:57:34	lyarwood	bauzas: well rebase, I'll fix the failing func tests if it's a valid fix
14:57:40	bauzas	lyarwood: mriedem: could we just one day say we deprecate multiple instances creation ? :)
14:58:31	edleafe	mriedem: One of the early discussions on alternate hosts was the concern about providing an alternate that may have already been consumed by another instance from the request. If that's no longer a concern, then that patch is fine.
14:59:00	bauzas	edleafe: the problem is that we don't know yet if we will consume the host
14:59:09	mriedem	edleafe: yeah that's what i mentioned to lya
14:59:20	mriedem	*lyarwood
14:59:28	bauzas	I mean, when it goes to pick a destination, we don't know yet if the allocation is possible
15:02:51	bauzas	nevermind, we consumed the host before we get the alternates
15:03:19	mriedem	i think the point is we don't want several reschedules trying to clobber the same alternate
15:03:25	mriedem	maybe we should randomize the alternats?
15:03:28	mriedem	*alternates?
15:03:28	edleafe	bauzas: the selected_host has already claimed the resources at that point. Alternates only are provided if something else has prevented the selected host from bulding the instance. The feeling expressed when I wrote that is that we didn't want to include a host that already had some of its resources consumed
15:03:48	melwitt	gibi: where did you send a mail about the minimum bandwidth scheduling demo location? I didn't see it
15:06:57	gibi	melwitt: http://lists.openstack.org/pipermail/openstack-dev/2018-September/134622.html
15:07:58	bauzas	mriedem: edleafe: yeah, the problem is that we don't optimize how we sort the alternates
15:08:20	melwitt	gibi: oh, yeah, ok. the same one I read /facepalm
15:08:29	melwitt	yesterday. I was thinking there was some other new one
15:08:40	bauzas	mriedem: edleafe: meaning, instances could be optimized to be claimed spreadly, but then they could have the same host as alternate
15:09:01	gibi	melwitt: if there was a new one yesterday then I didn't see that :)
15:09:08	bauzas	so, yeah, we could just randomize the alternates, or find a better strategy
15:09:57	edleafe	lyarwood: I'm curious as to the motivation for this. Have you seen a number of build failures on hosts that would require more alternates?
15:10:09	bauzas	edleafe: see the bug report
15:10:20	bauzas	in a small env, this could fail
15:10:23	bauzas	5 instances, 2 nodes
15:10:58	bauzas	so, yeah, we're over rejecting
15:11:19	edleafe	Ah, when you are requesting more instances in a single build than you have available hosts
15:11:23	bauzas	we should less reject, but we should also shuffle a little bit to spread more
15:11:33	bauzas	edleafe: that's correct
15:12:06	edleafe	Patient: "Doctor, it hurts when I do this" Doctor: "Then stop doing that!"
15:12:12	edleafe	:)
15:14:18	mriedem	then i'd make the logic conditional on whether or not we have num_instances > filtered_hosts
15:16:46	bauzas	mriedem: edleafe: well
15:18:27	bauzas	mriedem: edleafe: the above is a real bug if and only if what was claimed is not able to be spawned, right?
15:18:57	bauzas	because we claimed before we look for alternates, so from a pure capacity point of view, we should be good
15:19:21	edleafe	bauzas: it depends on the reason for the failure.
15:19:47	edleafe	bauzas: if it was because of insufficient resources, then we have a BIG problem
15:19:47	bauzas	edleafe: we trigger alternates because something else but a capacitiy issue, right ?
15:20:03	edleafe	bauzas: yes, that's the design
15:20:29	edleafe	We've already claimed resources before a host is designated as the selected_host
15:22:23	bauzas	edleafe: yup, I'm just stating this
15:22:46	bauzas	edleafe: alternates are for derailed requests that shouldn't be capacity problems
15:22:57	bauzas	so
15:23:59	bauzas	back to the possible solutions, I think lyarwood's proposal is good with the slight detail that we could just shuffle a bit
15:24:21	mriedem	i just want to make sure that if i'm spawning 5 instances with 4 hosts, is the first alternate for all instances going to be the same host?
15:24:52	mriedem	or is the answer "it depends"
15:26:11	bauzas	mriedem: there is a TODO on this
15:27:04	bauzas	mriedem: edleafe: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L411

Earlier   Later