Thursday, 2018-09-13

*** jangutter has quit IRC00:01
*** janki has quit IRC00:01
*** nicolasbock has quit IRC00:08
*** jangutter has joined #openstack-nova00:25
*** Swami has quit IRC00:25
*** savvas has quit IRC00:26
*** whoami-rajat has quit IRC00:30
*** savvas has joined #openstack-nova00:32
*** jangutter has quit IRC00:34
openstackgerritMerged openstack/nova master: Update RequestSpec.flavor on resize_revert  https://review.openstack.org/58868900:35
*** gryf has quit IRC00:39
openstackgerritBrin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter  https://review.openstack.org/59927600:40
*** gryf has joined #openstack-nova00:41
*** dkehn has quit IRC00:46
*** dkehn has joined #openstack-nova00:56
*** gyee has quit IRC00:59
*** bhagyashris has quit IRC01:03
*** savvas has quit IRC01:06
*** slaweq_ has quit IRC01:06
*** annp has joined #openstack-nova01:11
*** sapd1 has joined #openstack-nova01:15
*** Dinesh_Bhor has joined #openstack-nova01:18
*** dklyle has joined #openstack-nova01:19
*** Dinesh_Bhor has quit IRC01:19
*** savvas has joined #openstack-nova01:20
*** savvas has quit IRC01:25
*** Dinesh_Bhor has joined #openstack-nova01:27
*** bhagyashris has joined #openstack-nova01:28
*** savvas has joined #openstack-nova01:30
*** savvas has quit IRC01:34
*** dave-mccowan has quit IRC01:36
*** dklyle has quit IRC01:39
*** savvas has joined #openstack-nova01:40
*** jangutter has joined #openstack-nova01:43
*** mschuppe1 has joined #openstack-nova01:45
*** mrsoul` has joined #openstack-nova01:46
*** brinzhang has joined #openstack-nova01:46
*** cdent has joined #openstack-nova01:47
*** mrsoul has quit IRC01:48
*** mschuppert has quit IRC01:48
*** bhagyashris has quit IRC01:59
*** hongbin has quit IRC02:00
*** jangutter has quit IRC02:04
*** tssurya has joined #openstack-nova02:04
*** cdent has quit IRC02:11
*** tssurya has quit IRC02:12
*** savvas has quit IRC02:12
*** awaugama has quit IRC02:14
*** tssurya has joined #openstack-nova02:14
*** brinzh has joined #openstack-nova02:14
*** brinzh has quit IRC02:15
*** savvas has joined #openstack-nova02:18
*** mtreinish has quit IRC02:25
*** mtreinish has joined #openstack-nova02:26
*** brinzhang_ has joined #openstack-nova02:31
openstackgerritChen proposed openstack/nova master: Fix evacuate logging  https://review.openstack.org/59305502:31
openstackgerritChen proposed openstack/nova master: Fix evacuate logging  https://review.openstack.org/59305502:34
*** sapd1 has quit IRC02:34
*** sapd1 has joined #openstack-nova02:36
*** sapd1_ has joined #openstack-nova02:36
*** brinzhang__ has joined #openstack-nova02:45
*** brinzhang_ has left #openstack-nova02:48
*** brinzhang has quit IRC02:49
*** brinzhang__ is now known as brinzhang02:49
*** Bhujay has joined #openstack-nova02:50
*** savvas has quit IRC02:51
*** brinzhang has quit IRC02:52
*** brinzhang has joined #openstack-nova02:52
*** vladikr has quit IRC02:53
*** vladikr has joined #openstack-nova02:53
*** gbarros has joined #openstack-nova02:53
*** tssurya has quit IRC02:53
openstackgerritBalazs Gibizer proposed openstack/nova master: Fix mock.patch usage in unit tests  https://review.openstack.org/60219802:54
gibimelwitt: replied in the review02:56
*** savvas has joined #openstack-nova02:58
*** Bhujay has quit IRC02:59
*** vladikr has quit IRC03:10
*** vladikr has joined #openstack-nova03:11
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): Add API to support vgpu resource provider create  https://review.openstack.org/52031303:11
*** cfriesen has joined #openstack-nova03:12
openstackgerritfupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node  https://review.openstack.org/60221903:17
*** Bhujay has joined #openstack-nova03:17
*** 77FAAKC1H has joined #openstack-nova03:17
*** 77FAAKC1H has quit IRC03:18
*** Bhujay has quit IRC03:18
*** Bhujay has joined #openstack-nova03:19
*** dpawlik has joined #openstack-nova03:23
*** dpawlik has quit IRC03:28
*** savvas has quit IRC03:30
*** savvas has joined #openstack-nova03:36
*** Bhujay has quit IRC03:41
*** vladikr has quit IRC03:42
*** vladikr has joined #openstack-nova03:43
*** slaweq_ has joined #openstack-nova03:44
*** gbarros has quit IRC03:48
*** Dinesh_Bhor has quit IRC03:48
*** sorrison has joined #openstack-nova03:49
*** slaweq_ has quit IRC03:49
*** med_ has joined #openstack-nova03:51
*** sorrison has quit IRC03:51
*** jangutter has joined #openstack-nova03:54
*** jangutter has quit IRC03:58
*** erlon has joined #openstack-nova04:08
*** savvas has quit IRC04:10
*** ivve has joined #openstack-nova04:10
*** med_ has quit IRC04:18
*** vladikr has quit IRC04:18
*** vladikr has joined #openstack-nova04:18
*** nicolasbock has joined #openstack-nova04:23
*** savvas has joined #openstack-nova04:23
*** savvas has quit IRC04:28
openstackgerritBrin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter  https://review.openstack.org/59927604:32
*** savvas has joined #openstack-nova04:33
*** Dinesh_Bhor has joined #openstack-nova04:36
*** savvas has quit IRC04:37
*** savvas has joined #openstack-nova04:42
*** claudiub has joined #openstack-nova04:44
*** claudiub|2 has joined #openstack-nova04:45
*** krtaylor_ has quit IRC04:45
*** krtaylor_ has joined #openstack-nova04:46
*** claudiub has quit IRC04:48
*** Dinesh_Bhor has quit IRC04:51
*** claudiub has joined #openstack-nova04:51
*** claudiub|2 has quit IRC04:53
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): Add API to support vgpu resource provider create  https://review.openstack.org/52031305:02
*** Dinesh_Bhor has joined #openstack-nova05:04
*** claudiub|2 has joined #openstack-nova05:04
*** sapd1 has quit IRC05:06
*** sapd1 has joined #openstack-nova05:06
*** claudiub has quit IRC05:07
*** claudiub|2 has quit IRC05:09
*** erlon has quit IRC05:12
*** phuongnh has quit IRC05:15
*** savvas has quit IRC05:16
*** brinzhang has quit IRC05:19
*** brinzhang has joined #openstack-nova05:20
*** savvas has joined #openstack-nova05:21
*** nicolasbock has quit IRC05:27
*** artom has joined #openstack-nova05:29
*** luksky11 has joined #openstack-nova05:29
*** artom has quit IRC05:29
*** artom has joined #openstack-nova05:30
openstackgerritTony Breeds proposed openstack/python-novaclient stable/queens: Switch to stestr  https://review.openstack.org/60193305:30
openstackgerritTony Breeds proposed openstack/python-novaclient stable/queens: import zuul job settings from project-config  https://review.openstack.org/60140005:30
*** jiaopengju has joined #openstack-nova05:38
*** sapd1_ has quit IRC05:45
openstackgerritfupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node  https://review.openstack.org/60221905:47
*** savvas has quit IRC05:55
*** Eran_Kuris has joined #openstack-nova05:56
*** savvas has joined #openstack-nova06:01
*** mschuppe1 is now known as mschuppert06:13
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): Add API to support vgpu resource provider create  https://review.openstack.org/52031306:14
*** cfriesen has quit IRC06:22
*** dpawlik has joined #openstack-nova06:23
*** sahid has joined #openstack-nova06:32
*** savvas has quit IRC06:33
*** fanzhang has quit IRC06:34
*** cfriesen has joined #openstack-nova06:34
*** fanzhang has joined #openstack-nova06:34
*** adrianc has joined #openstack-nova06:36
*** jangutter has joined #openstack-nova06:36
*** Luzi has joined #openstack-nova06:36
*** jangutter has quit IRC06:37
*** savvas has joined #openstack-nova06:39
*** jangutter has joined #openstack-nova06:42
*** savvas has quit IRC06:44
*** savvas has joined #openstack-nova06:57
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/60104707:01
*** rcernin has quit IRC07:01
*** tesseract has joined #openstack-nova07:01
*** savvas has quit IRC07:02
*** jangutter has quit IRC07:04
*** savvas has joined #openstack-nova07:06
*** ivve has quit IRC07:16
*** cfriesen has quit IRC07:16
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create  https://review.openstack.org/52031307:17
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations`  https://review.openstack.org/52171707:19
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): support compute node resource provider update  https://review.openstack.org/52104107:33
*** AlexeyAbashkin has joined #openstack-nova07:34
*** Dinesh_Bhor has quit IRC07:34
*** kaliya has joined #openstack-nova07:38
*** savvas has quit IRC07:40
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create  https://review.openstack.org/52031307:41
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations`  https://review.openstack.org/52171707:42
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): support compute node resource provider update  https://review.openstack.org/52104107:42
*** jpena|off is now known as jpena07:44
*** savvas has joined #openstack-nova07:46
*** Dinesh_Bhor has joined #openstack-nova07:47
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update  https://review.openstack.org/52104107:49
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations`  https://review.openstack.org/52171707:50
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update  https://review.openstack.org/52104107:51
*** Dinesh_Bhor has quit IRC07:54
*** marvin_mhg has quit IRC07:59
*** marvin_mhg has joined #openstack-nova08:01
*** oanson has quit IRC08:04
*** artom has quit IRC08:18
*** artom has joined #openstack-nova08:18
*** artom has quit IRC08:19
*** artom has joined #openstack-nova08:19
*** oanson has joined #openstack-nova08:19
*** artom has quit IRC08:20
*** savvas has quit IRC08:20
*** artom has joined #openstack-nova08:20
*** artom has quit IRC08:21
*** artom has joined #openstack-nova08:21
*** artom has quit IRC08:22
*** artom_ has joined #openstack-nova08:22
*** artom_ is now known as artom08:22
*** artom has quit IRC08:23
*** savvas has joined #openstack-nova08:25
*** Dinesh_Bhor has joined #openstack-nova08:26
*** s10 has joined #openstack-nova08:31
*** AlexeyAbashkin has quit IRC08:38
*** AlexeyAbashkin has joined #openstack-nova08:38
*** annp has quit IRC08:46
*** jiapei has quit IRC08:52
*** trungnv has quit IRC08:54
*** trungnv has joined #openstack-nova08:54
*** adrianc_ has joined #openstack-nova08:56
*** adrianc has quit IRC08:57
*** savvas has quit IRC08:58
*** savvas has joined #openstack-nova09:04
*** dtantsur|afk is now known as dtantsur09:04
*** brinzh has joined #openstack-nova09:05
*** savvas has quit IRC09:37
*** maciejjozefczyk has quit IRC09:41
*** dpawlik has quit IRC09:43
*** annp has joined #openstack-nova09:44
*** Dinesh_Bhor has quit IRC09:45
*** savvas has joined #openstack-nova09:51
*** maciejjozefczyk has joined #openstack-nova09:54
*** dpawlik has joined #openstack-nova09:54
*** savvas has quit IRC09:56
*** adrianc_ has quit IRC09:59
*** savvas has joined #openstack-nova10:01
*** Dinesh_Bhor has joined #openstack-nova10:04
*** savvas has quit IRC10:05
*** adrianc_ has joined #openstack-nova10:10
*** savvas has joined #openstack-nova10:10
*** Dinesh_Bhor has quit IRC10:26
*** annp has quit IRC10:28
*** slaweq__ has joined #openstack-nova10:36
*** savvas has quit IRC10:43
*** savvas has joined #openstack-nova10:49
*** slaweq__ has quit IRC10:52
*** s1061123 has joined #openstack-nova11:15
*** jamesdenton has joined #openstack-nova11:40
*** nicolasbock has joined #openstack-nova11:40
*** vladikr has quit IRC11:48
*** vladikr has joined #openstack-nova11:49
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: Support initial allocation ratios  https://review.openstack.org/55210511:50
*** Eran_Kuris has quit IRC12:16
*** tbachman has joined #openstack-nova12:22
*** szaher_ has joined #openstack-nova12:24
*** savvas has quit IRC12:25
*** tetsuro has joined #openstack-nova12:28
*** tetsuro has quit IRC12:29
*** tetsuro has joined #openstack-nova12:29
*** munimeha1 has joined #openstack-nova12:30
*** savvas has joined #openstack-nova12:32
*** belmoreira has quit IRC12:33
*** lyan has joined #openstack-nova12:36
openstackgerritBrin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter  https://review.openstack.org/59927612:36
*** savvas has quit IRC12:37
*** brinzhang has quit IRC12:39
*** brinzh has quit IRC12:40
*** savvas has joined #openstack-nova12:42
*** savvas has quit IRC12:51
*** belmoreira has joined #openstack-nova12:51
*** dave-mccowan has joined #openstack-nova12:52
*** dave-mccowan has quit IRC12:57
*** med_ has joined #openstack-nova12:58
*** dave-mcc_ has joined #openstack-nova12:59
*** markvoelker has joined #openstack-nova12:59
*** savvas has joined #openstack-nova13:00
*** adrianc_ has quit IRC13:01
*** jaypipes has joined #openstack-nova13:01
*** adrianc_ has joined #openstack-nova13:02
*** szaher_ has quit IRC13:02
*** szaher has joined #openstack-nova13:05
*** gbarros has joined #openstack-nova13:07
openstackgerritHamdy Khader proposed openstack/nova master: Set defult value of num_nvme_discover_tries=5  https://review.openstack.org/60235113:15
*** savvas has quit IRC13:18
*** savvas has joined #openstack-nova13:18
*** jaypipes has quit IRC13:19
*** lbragstad has quit IRC13:25
*** Eran_Kuris has joined #openstack-nova13:28
*** savvas has quit IRC13:30
*** savvas has joined #openstack-nova13:31
openstackgerritClaudiu Belu proposed openstack/nova master: conductor: add live_resize task  https://review.openstack.org/24857913:34
openstackgerritClaudiu Belu proposed openstack/nova master: API: Adds instance live-resize  https://review.openstack.org/18458413:35
*** awaugama has joined #openstack-nova13:36
*** munimeha1_ has joined #openstack-nova13:37
*** erlon has joined #openstack-nova13:39
*** jangutter has joined #openstack-nova13:48
*** tetsuro has quit IRC13:48
*** mvenesio has joined #openstack-nova13:50
*** trozet has joined #openstack-nova13:56
*** tetsuro has joined #openstack-nova13:59
*** tetsuro_ has joined #openstack-nova13:59
openstackgerritClaudiu Belu proposed openstack/nova master: WIP: replace spec with autospec  https://review.openstack.org/55729914:01
*** claudiub has joined #openstack-nova14:01
*** belmoreira has quit IRC14:02
*** jangutter has quit IRC14:03
*** erlon has quit IRC14:03
*** tetsuro has quit IRC14:04
*** mdbooth has joined #openstack-nova14:10
*** munimeha1_ has quit IRC14:14
*** munimeha1 has quit IRC14:14
*** savvas has quit IRC14:21
*** maciejjozefczyk has quit IRC14:22
*** lbragstad has joined #openstack-nova14:23
*** Luzi has quit IRC14:27
*** pcaruana has joined #openstack-nova14:28
*** savvas has joined #openstack-nova14:30
*** slaweq__ has joined #openstack-nova14:32
*** savvas has quit IRC14:34
*** eharney has joined #openstack-nova14:35
litao__hi14:38
litao__This is my patch:https://review.openstack.org/#/c/592252/ I need some reviews , thanks14:39
*** savvas has joined #openstack-nova14:39
*** sapd1 has quit IRC14:40
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create  https://review.openstack.org/52031314:40
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations`  https://review.openstack.org/52171714:40
*** mriedem has joined #openstack-nova14:41
*** sapd1 has joined #openstack-nova14:42
*** hoangcx has joined #openstack-nova14:42
kashyaplitao__: Hi, many folks are at a conference this week, so expect some delays in responses14:42
*** belmoreira has joined #openstack-nova14:43
*** savvas has quit IRC14:44
*** whoami-rajat has joined #openstack-nova14:45
mriedemdansmith: 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-calls14:45
*** bhagyashris has joined #openstack-nova14:46
*** tetsuro_ has quit IRC14:48
*** savvas has joined #openstack-nova14:48
*** med_ has quit IRC14:49
mriedembauzas: here is the fugly cross_az_attach=False bug fix i mentioned yesterday https://review.openstack.org/#/c/469675/14:50
mriedemnote 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@91814:50
*** bnemec has quit IRC14:52
*** jangutter has joined #openstack-nova14:52
*** tetsuro has joined #openstack-nova14:52
*** tetsuro has quit IRC14:52
*** tetsuro_ has joined #openstack-nova14:53
lyarwoodbauzas: 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 chance14:53
*** slaweq__ has quit IRC14:53
bauzasmriedem: ta, will look at both14:53
*** tetsuro has joined #openstack-nova14:53
mriedemedleafe: you might want to take a look at https://review.openstack.org/#/c/593074/ from lyarwood above14:54
openstackgerritClaudiu Belu proposed openstack/nova master: conductor: add live_resize task  https://review.openstack.org/24857914:54
openstackgerritClaudiu Belu proposed openstack/nova master: API: Adds instance live-resize  https://review.openstack.org/18458414:54
bauzaslyarwood: yup, saw your change yesterday14:54
*** slaweq__ has joined #openstack-nova14:55
*** savvas has quit IRC14:55
*** adrianc_ has quit IRC14:56
*** efried has joined #openstack-nova14:57
lyarwoodbauzas: well rebase, I'll fix the failing func tests if it's a valid fix14:57
*** tetsuro_ has quit IRC14:57
bauzaslyarwood: mriedem: could we just one day say we deprecate multiple instances creation ? :)14:57
edleafemriedem: 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:58
*** mdbooth has quit IRC14:58
*** bhagyashris has quit IRC14:58
*** savvas has joined #openstack-nova14:58
*** mdbooth has joined #openstack-nova14:58
bauzasedleafe: the problem is that we don't know yet if we will consume the host14:59
mriedemedleafe: yeah that's what i mentioned to lya14:59
mriedem*lyarwood14:59
*** jaosorior has joined #openstack-nova14:59
bauzasI mean, when it goes to pick a destination, we don't know yet if the allocation is possible14:59
*** bhagyashris has joined #openstack-nova15:02
*** tbachman has quit IRC15:02
bauzasnevermind, we consumed the host before we get the alternates15:02
*** eharney has quit IRC15:03
mriedemi think the point is we don't want several reschedules trying to clobber the same alternate15:03
mriedemmaybe we should randomize the alternats?15:03
edleafebauzas: 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 consumed15:03
mriedem*alternates?15:03
*** abhishekk has joined #openstack-nova15:03
melwittgibi: where did you send a mail about the minimum bandwidth scheduling demo location? I didn't see it15:03
*** jiapei has joined #openstack-nova15:04
*** cfriesen has joined #openstack-nova15:06
gibimelwitt: http://lists.openstack.org/pipermail/openstack-dev/2018-September/134622.html15:06
*** owalsh has quit IRC15:07
*** owalsh has joined #openstack-nova15:07
bauzasmriedem: edleafe: yeah,  the problem is that we don't optimize how we sort the alternates15:07
melwittgibi: oh, yeah, ok. the same one I read /facepalm15:08
melwittyesterday. I was thinking there was some other new one15:08
*** lei-zh has joined #openstack-nova15:08
bauzasmriedem: edleafe: meaning, instances could be optimized to be claimed spreadly, but then they could have the same host as alternate15:08
gibimelwitt: if there was a new one yesterday then I didn't see that :)15:09
bauzasso, yeah, we could just randomize the alternates, or find a better strategy15:09
*** itlinux has quit IRC15:09
edleafelyarwood: 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:09
*** erlon has joined #openstack-nova15:09
*** takashin has joined #openstack-nova15:10
bauzasedleafe: see the bug report15:10
bauzasin a small env, this could fail15:10
bauzas5 instances, 2 nodes15:10
bauzasso, yeah, we're over rejecting15:10
*** tssurya has joined #openstack-nova15:10
edleafeAh, when you are requesting more instances in a single build than you have available hosts15:11
*** dklyle has joined #openstack-nova15:11
bauzaswe should less reject, but we should also shuffle a little bit to spread more15:11
*** cdent has joined #openstack-nova15:11
bauzasedleafe: that's correct15:11
*** damien_r has quit IRC15:11
*** sorrison has joined #openstack-nova15:11
edleafePatient: "Doctor, it hurts when I do this" Doctor: "Then stop doing that!"15:12
edleafe:)15:12
mriedemthen i'd make the logic conditional on whether or not we have num_instances > filtered_hosts15:14
*** kaliya has quit IRC15:14
*** cdent has quit IRC15:14
*** imacdonn has quit IRC15:14
*** cdent has joined #openstack-nova15:15
*** imacdonn has joined #openstack-nova15:15
*** holser|ptg has joined #openstack-nova15:15
*** gyee has joined #openstack-nova15:15
*** nicolasbock has quit IRC15:15
*** bnemec has joined #openstack-nova15:15
bauzasmriedem: edleafe: well15:16
*** savvas has quit IRC15:17
*** sorrison has quit IRC15:17
*** savvas has joined #openstack-nova15:17
*** sorrison has joined #openstack-nova15:17
bauzasmriedem: edleafe: the above is a real bug if and only if what was claimed is not able to be spawned, right?15:18
bauzasbecause we claimed before we look for alternates, so from a pure capacity point of view, we should be good15:18
edleafebauzas: it depends on the reason for the failure.15:19
bauzasedleafe: we trigger alternates because something else but a capacitiy issue, right ?15:19
edleafebauzas: if it was because of insufficient resources, then we have a BIG problem15:19
*** mdbooth has quit IRC15:19
edleafebauzas: yes, that's the design15:20
*** mdbooth has joined #openstack-nova15:20
*** alexchadin has joined #openstack-nova15:20
edleafeWe've already claimed resources before a host is designated as the selected_host15:20
*** sorrison has quit IRC15:21
*** sorrison has joined #openstack-nova15:21
bauzasedleafe: yup, I'm just stating this15:22
bauzasedleafe: alternates are for derailed requests that shouldn't be capacity problems15:22
bauzasso15:22
*** med_ has joined #openstack-nova15:23
*** sahid has quit IRC15:23
bauzasback to the possible solutions, I think lyarwood's proposal is good with the slight detail that we could just shuffle a bit15:23
mriedemi 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
mriedemor is the answer "it depends"15:24
*** eharney has joined #openstack-nova15:25
*** hoangcx has quit IRC15:25
bauzasmriedem: there is a TODO on this15:26
bauzasmriedem: edleafe: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L41115:27
bauzasmriedem: that's the shuffling bit I'd like to have15:27
bauzaswe just get the first element and we consider it to be the alternate15:27
bauzasthat's highly probabilistic that we gonna end on the same alternate15:28
bauzasso, just fix the TODO and do a bit of a guess instead of getting the first element of the list15:28
bauzasedleafe: alternates are all seen as viable targets, rightN15:29
bauzasright?15:29
bauzasfrom a capacity pov15:29
bauzasso, there is no harm to pick a random one and not just the first in the list15:29
bauzasedleafe: you agree ?15:29
bauzaslyarwood: I made some very technical comments on the functional test15:30
bauzaslyarwood: if you need help for a respin, lemme know15:30
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data  https://review.openstack.org/40751415:31
edleafebauzas: that's a different version of shuffling, for the nested case15:31
edleafeIOW, if a single RP has multiple allocation_candidates, just pick the first for now15:31
edleafeThe TODO is to evaluate each for "best fit"15:31
*** Eran_Kuris has quit IRC15:33
*** savvas has quit IRC15:33
*** slaweq__ is now known as slaweq15:34
lyarwoodbauzas: thanks, I'll get to it after this session15:35
edleafebauzas: I don't see a problem with a random selection, but in the two-node setup described in the bug, it won't help :)15:35
bauzasedleafe: well, it could help or not15:36
bauzasedleafe: but at least, we would ensure that if we have more than 1 host as alternate, we don't pick the same for all the instances :)15:36
edleafebauzas: in a two-node setup, there is only one possible alternate :)15:36
bauzasedleafe: yeah, sure, no disagreement on it :D15:36
bauzasactually, my guess is that the example we have in the bug report will fail anyway15:37
edleafebauzas: the other thing to remember is that the list of hosts have been weighed at this point15:37
edleafeSo a random choice will ignore all weighting15:38
bauzasah good point15:38
bauzasmmmm15:38
bauzasok, let's keep it simple then15:38
bauzasif alternates follow the packing/spreading strategy given by the weighers, they're already shuffled15:39
bauzasso, what lyarwood did is sufficient15:39
*** savvas has joined #openstack-nova15:41
*** macza has joined #openstack-nova15:41
openstackgerritEric Fried proposed openstack/os-traits master: Add OWNERSHIP_$SERVICE traits  https://review.openstack.org/60216015:43
*** hamzy has quit IRC15:43
*** belmoreira has quit IRC15:46
*** savvas has quit IRC15:46
bauzasedleafe: wait a sec15:47
bauzasedleafe: is https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L417 a weighed list ?15:48
*** belmoreira has joined #openstack-nova15:49
edleafeARs are not weighed. The hosts they represent are15:49
bauzasthat's old greek to me :)15:49
bauzasoh shit15:50
bauzasI think I misunderstood the logic15:51
*** Sukhdev has joined #openstack-nova15:51
bauzaswe got selected hosts, we iterate over those and see whether they have an AR against each15:51
*** savvas has joined #openstack-nova15:51
bauzasgotcha15:51
bauzasso, yeah, selected hosts is weighed15:52
edleafebauzas: this is where the alternates are created: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L260, and the 'hosts' variable is defined above: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L26015:52
*** belmoreira has quit IRC15:53
*** betherly_ is now known as betherly15:54
bauzasyeah I got it15:54
edleafebauzas: oops: the second link should be https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L20115:54
bauzashttps://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L397 is the loop over hosts that are weighed15:54
bauzasso, yeah, we basically either spread or pack that list15:55
bauzasso, we're fine15:55
edleafeyup15:55
bauzasno need to shuffle any bit15:55
bauzasif the operator chose the packing strategy, that's their decision15:55
*** med_ has quit IRC15:58
*** savvas has quit IRC16:00
*** itlinux has joined #openstack-nova16:00
openstackgerritsean mooney proposed openstack/os-vif master: add support for generic tap device plug  https://review.openstack.org/60238416:01
*** nicolasbock has joined #openstack-nova16:02
*** savvas has joined #openstack-nova16:05
bauzasedleafe: you know what ? I'm horribly confused16:08
openstackgerritEric Fried proposed openstack/os-traits master: Add OWNERSHIP_$SERVICE traits  https://review.openstack.org/60216016:11
bauzasedleafe: how many times are we calling _get_alternate_hosts() if we have multiple instances ? one AFAICS16:11
*** sorrison has quit IRC16:11
mriedemmdbooth: i'm not sure why but my evacuate script is thinking that we only have 1 hypervisor in this job but we should have 2 https://review.openstack.org/#/c/602174/ - i'm guessing it's a bug in my script doing the word count16:13
mriedemoh i see why16:14
mriedemwell i think16:14
*** sorrison has joined #openstack-nova16:15
*** itlinux is now known as itlinux-away16:15
mriedem$ echo "Missing value auth-url required for auth plugin password" | wc -l16:15
mriedem116:15
*** itlinux-away is now known as itlinux16:17
*** med_ has joined #openstack-nova16:17
*** itlinux is now known as itlinux-away16:17
*** dave-mcc_ has quit IRC16:25
*** itlinux-away is now known as itlinux16:25
*** itlinux is now known as itlinux-away16:25
*** itlinux-away is now known as itlinux16:25
*** itlinux is now known as itlinux-away16:25
*** mdbooth_ has joined #openstack-nova16:27
*** mdbooth_ has quit IRC16:27
*** itlinux-away is now known as itlinux16:27
openstackgerritBalazs Gibizer proposed openstack/nova master: consumer gen: more tests for delete allocation cases  https://review.openstack.org/59181116:29
edleafebauzas: We call it once for each requested instance16:29
*** tetsuro has quit IRC16:31
*** cfriesen has quit IRC16:32
*** munimeha1 has joined #openstack-nova16:32
*** awaugama has quit IRC16:32
*** tetsuro has joined #openstack-nova16:34
*** eharney has quit IRC16:35
*** bodgix has left #openstack-nova16:35
*** munimeha1_ has joined #openstack-nova16:38
*** dave-mcc_ has joined #openstack-nova16:38
*** cfriesen has joined #openstack-nova16:39
*** alexchadin has quit IRC16:40
*** savvas has quit IRC16:41
*** munimeha1 has quit IRC16:42
*** dtantsur is now known as dtantsur|afk16:43
*** adrianc_ has joined #openstack-nova16:43
*** savvas has joined #openstack-nova16:44
*** s10 has quit IRC16:45
*** bhagyashris has quit IRC16:46
*** munimeha1_ has quit IRC16:47
*** alexchadin has joined #openstack-nova16:48
openstackgerritMerged openstack/nova master: Fix mock.patch usage in unit tests  https://review.openstack.org/60219816:50
*** bhagyashris has joined #openstack-nova16:51
bauzasedleafe: I tried to summarize my thoughts in one single comment https://review.openstack.org/#/c/593074/2/nova/scheduler/filter_scheduler.py@39716:55
*** erlon has quit IRC16:56
*** tesseract has quit IRC17:00
*** brault has quit IRC17:00
*** brault has joined #openstack-nova17:02
*** alexchadin has quit IRC17:02
*** brault has quit IRC17:05
*** sorrison has quit IRC17:05
*** alexchadin has joined #openstack-nova17:05
*** sorrison has joined #openstack-nova17:05
*** jpena is now known as jpena|off17:07
*** sorrison has quit IRC17:07
*** sorrison has joined #openstack-nova17:08
*** priteau has joined #openstack-nova17:10
*** lei-zh has quit IRC17:10
*** eharney has joined #openstack-nova17:11
*** lei-zh has joined #openstack-nova17:11
*** brault has joined #openstack-nova17:12
*** erlon has joined #openstack-nova17:17
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Update RequestSpec.flavor on resize_revert  https://review.openstack.org/60240317:17
*** savvas has quit IRC17:18
*** hamzy has joined #openstack-nova17:18
*** cdent has quit IRC17:21
*** savvas has joined #openstack-nova17:24
*** itlinux is now known as itlinux-away17:27
*** itlinux-away is now known as itlinux17:29
*** itlinux is now known as itlinux-away17:29
*** sorrison has quit IRC17:30
openstackgerritMatt Riedemann proposed openstack/nova master: Noop CantStartEngineError in targets_cell if API DB not configured  https://review.openstack.org/58191217:31
*** itlinux-away is now known as itlinux17:32
melwittefried: can you add some notes about the whitelist plan on the etherpad? I wasn't sure which line we're talking17:32
*** itlinux is now known as itlinux-away17:32
*** AlexeyAbashkin has quit IRC17:32
*** sorrison has joined #openstack-nova17:32
efriedmelwitt: roger that. It's here: https://review.openstack.org/#/c/591037/ Where do you want it on which etherpad? :)17:33
*** itlinux-away is now known as itlinux17:34
melwittefried: oh, if it's already known, that's cool17:34
*** itlinux is now known as itlinux-away17:34
efriedmelwitt: Agree it would be useful to have on one or both etherpads17:34
melwittefried: yeah, I wanted there to be a reference for cyborg team to see how that part will be handled. I don't see a relevant section in the etherpad https://etherpad.openstack.org/p/stein-ptg.cyborg-nova-new so far17:35
efriedoh, yeah, that spec is in a topic of its own later on....17:35
efriedmelwitt: L291 of https://etherpad.openstack.org/p/nova-ptg-stein17:35
melwittah thanks17:36
mriedemactually 21417:38
mriedemi see what efried is trying to do17:38
*** itlinux-away is now known as itlinux17:38
*** itlinux is now known as itlinux-away17:38
mriedemoh nvm - different topic :)17:38
*** cdent has joined #openstack-nova17:39
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: cover migration cases with functional tests  https://review.openstack.org/60240817:39
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Update RequestSpec.flavor on resize_revert  https://review.openstack.org/60240917:39
*** vladikr is now known as vladikr_brb17:40
*** itlinux-away is now known as itlinux17:41
*** luksky11 has quit IRC17:41
*** savvas has quit IRC17:43
*** savvas has joined #openstack-nova17:43
*** bnemec has quit IRC17:45
*** alexchadin has quit IRC17:48
*** mdbooth has quit IRC17:52
lyarwoodedleafe: sorry missed your ping earlier, I'm not sure how that additional conditional would help tbh17:53
*** mdbooth has joined #openstack-nova17:53
bauzasmriedem: that's the bug right ? https://bugs.launchpad.net/nova/+bug/178402017:55
openstackLaunchpad bug 1784020 in OpenStack Compute (nova) "Shared storage providers are not supported and will break things if used" [Medium,Fix released]17:55
bauzasthanks for the heads-up17:56
openstackgerritMerged openstack/nova master: hyperv: Cleans up live migration Planned VM  https://review.openstack.org/47894317:56
*** cfriesen has quit IRC17:56
mriedemyes17:56
*** cfriesen has joined #openstack-nova17:56
edleafelyarwood: because the bug only is an issue when there aren't enough potential alternate hosts, right?17:57
lyarwoodedleafe: right and alternates listed are per instance17:58
lyarwoodalternates are listed*17:58
*** vladikr_brb is now known as vladikr17:58
edleafelyarwood: so in the more common case where there are many more alternates available, we would be better off not adding hosts that may not have enoguh resources17:59
edleafelyarwood: only in the case where there aren't enough alternates should we include a host that is actively being built on18:00
*** tbachman has joined #openstack-nova18:01
*** cfriesen has quit IRC18:01
lyarwoodedleafe: ah right, got it18:02
*** cfriesen has joined #openstack-nova18:02
*** kaisers has quit IRC18:03
openstackgerritEric Fried proposed openstack/os-traits master: Add OWNERSHIP_$SERVICE traits  https://review.openstack.org/60216018:04
*** luksky has joined #openstack-nova18:04
*** abhishekk has quit IRC18:06
*** sorrison has quit IRC18:10
*** mdbooth has quit IRC18:11
*** savvas has quit IRC18:16
*** savvas has joined #openstack-nova18:17
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add post-test hook for testing evacuate  https://review.openstack.org/60217418:18
*** kaisers has joined #openstack-nova18:20
*** Sukhdev has quit IRC18:23
*** AJaeger has joined #openstack-nova18:24
*** AJaeger has left #openstack-nova18:24
*** slaweq has quit IRC18:25
*** priteau has quit IRC18:27
*** tssurya has quit IRC18:30
*** lbragstad has quit IRC18:33
*** hemna_ has joined #openstack-nova18:36
*** bhagyashris has quit IRC18:37
*** efried has quit IRC18:38
*** jangutter has quit IRC18:39
*** cdent has quit IRC18:39
*** tetsuro has quit IRC18:39
*** jangutter has joined #openstack-nova18:39
*** jangutter has quit IRC18:39
*** tbachman has quit IRC18:39
*** mriedem has quit IRC18:39
*** lei-zh has quit IRC18:41
*** erlon has quit IRC18:42
*** takashin has quit IRC18:42
*** cfriesen has quit IRC18:43
*** burt has joined #openstack-nova18:43
*** nicolasbock has quit IRC18:51
*** rcernin has joined #openstack-nova19:00
*** alexchadin has joined #openstack-nova19:00
*** adrianc_ has quit IRC19:00
*** med_ has quit IRC19:00
*** rtjure has joined #openstack-nova19:03
*** geekinutah has joined #openstack-nova19:06
*** cdent has joined #openstack-nova19:11
*** itlinux is now known as itlinux-away19:22
*** itlinux-away is now known as itlinux19:23
*** itlinux is now known as itlinux-away19:24
*** itlinux-away is now known as itlinux19:24
*** itlinux is now known as itlinux-away19:24
*** itlinux-away is now known as itlinux19:24
*** itlinux is now known as itlinux-away19:24
*** itlinux-away is now known as itlinux19:25
*** itlinux is now known as itlinux-away19:26
*** lbragstad has joined #openstack-nova19:26
*** slaweq has joined #openstack-nova19:28
*** lbragstad has quit IRC19:31
*** alexchadin has quit IRC19:31
*** itlinux-away is now known as itlinux19:33
*** itlinux is now known as itlinux-away19:33
*** gbarros has quit IRC19:35
*** itlinux-away is now known as itlinux19:36
*** itlinux is now known as itlinux-away19:36
*** priteau has joined #openstack-nova19:41
*** cfriesen has joined #openstack-nova19:45
*** alexchadin has joined #openstack-nova19:46
*** itlinux-away is now known as itlinux19:49
*** cfriesen has quit IRC19:50
*** itlinux is now known as itlinux-away19:50
*** lbragstad has joined #openstack-nova19:50
*** itlinux-away is now known as itlinux19:50
*** bnemec has joined #openstack-nova19:54
*** cdent has quit IRC19:55
*** awaugama has joined #openstack-nova19:55
*** sorrison has joined #openstack-nova19:55
*** tetsuro has joined #openstack-nova19:55
*** mriedem has joined #openstack-nova19:55
mriedemgibi: for your bw provider demo, osc commands take a wrap option, might be good to use - you're doing this at the summit too i think19:56
*** cfriesen has joined #openstack-nova19:56
*** tetsuro has quit IRC19:58
*** vladikr has quit IRC20:02
*** vladikr has joined #openstack-nova20:03
openstackgerritsean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif  https://review.openstack.org/60243220:03
*** cdent has joined #openstack-nova20:05
*** bhagyashris has joined #openstack-nova20:05
*** jangutter has joined #openstack-nova20:05
*** slaweq has quit IRC20:07
*** gbarros has joined #openstack-nova20:08
*** takashin has joined #openstack-nova20:08
stephenfinmriedem, gibi: You can also use '-c' to show only the columns you care out and filter out the noise20:08
*** tetsuro has joined #openstack-nova20:09
*** slaweq has joined #openstack-nova20:09
*** jangutter_ has joined #openstack-nova20:09
*** jangutter has quit IRC20:09
*** tssurya has joined #openstack-nova20:09
*** efried has joined #openstack-nova20:09
*** med_ has joined #openstack-nova20:11
*** erlon has joined #openstack-nova20:11
gibistephenfin, mriedem: thanks, it was a projector - laptop incompatibilty issue today about screen size. We will do better in Berlin :)20:13
mriedemstephenfin: oh i know all about -c20:14
mriedemand -f20:14
*** nicolasbock has joined #openstack-nova20:15
*** Sukhdev has joined #openstack-nova20:16
melwittour cycle highlights say live migration between neutron backends works https://github.com/openstack/releases/blob/master/deliverables/rocky/nova.yaml#L44-L4620:20
*** sorrison has quit IRC20:21
melwittgiven that, I think filling in the gaps would be good ^20:21
*** vladikr has quit IRC20:22
*** vladikr has joined #openstack-nova20:23
*** sorrison has joined #openstack-nova20:25
*** alexchadin has quit IRC20:28
*** priteau has quit IRC20:30
*** itlinux is now known as itlinux-away20:33
*** priteau has joined #openstack-nova20:35
*** itlinux-away is now known as itlinux20:35
*** pcaruana has quit IRC20:38
gibibauzas: if you want to try nested scheduling then this is the commit you need https://review.openstack.org/#/c/585672/ (our today's demo was based on it too)20:38
bauzasgibi: awesome, ta20:38
*** sorrison has quit IRC20:39
bauzasgibi: I first need to put my hands on some hardware20:39
openstackgerritMerged openstack/python-novaclient master: Follow the new PTI for document build  https://review.openstack.org/60215320:39
gibibauzas: please be aware that it is on an old base. I'm working on to rebase the whoel series20:39
*** sorrison has joined #openstack-nova20:39
bauzasbut that's something I'd like to do sooner than later clearly20:39
bauzasgibi: no worries, you said it's a substantial amount of work20:39
bauzasgibi: I guess it's worth it for me to help you if I can20:39
bauzasgibi: the only problem is that I was a bit on and off last cycle, so I need to ramp up a little bit20:40
gibibauzas: as soon as I catch up with the rebase you can help a lot by reviewing the series and finding gaps we missed20:42
gibibauzas: until that if you have any questions about the current solution then I'm happy to help20:43
bauzasgibi: sure, reviewing is one thing I can surely do20:45
openstackgerritMatt Riedemann proposed openstack/nova master: Fix docs and add functional test for AggregateMultiTenancyIsolation  https://review.openstack.org/60183520:47
*** hamzy has quit IRC20:48
*** rcernin has quit IRC20:51
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add post-test hook for testing evacuate  https://review.openstack.org/60217420:53
gibibauzas: this is the complication in the claim code path that I feel will cause trouble in move operations https://github.com/openstack/nova/blob/d8004bddaea167f85d5940e586cb5c9707adb278/nova/scheduler/client/report.py#L1820-L182720:53
gibibauzas: fortunately it seems we only hit this in evacuate case20:54
gibibauzas: as the rest of the move uses migration_uuid on the sorce host20:54
bauzaslooking20:55
bauzasgibi: oh shit, good catch20:56
bauzasgibi: do we have any way to ask placement whether we have allocations in a subtree ?20:57
bauzassomething like ?root_rp=<>20:57
* bauzas is seriously lagging on placement features20:57
gibibauzas: nothing I know of20:58
bauzashttp://p6.storage.canalblog.com/62/41/721385/52520059.jpg20:59
*** itlinux has quit IRC20:59
bauzasI miss sound20:59
* bauzas looking at the placement apiref21:00
bauzasoh wait21:00
bauzasgibi: https://github.com/openstack/nova/blob/d8004bddaea167f85d5940e586cb5c9707adb278/nova/scheduler/client/report.py#L1816 with some luck, we should get the allocation even if that's in a subtree21:01
bauzasI shall start looking at the query21:01
*** gbarros has quit IRC21:02
bauzasefried: btw. I said I have a patch up for resize/cold-migrate about allocations21:04
gibibauzas: one instance still only one allocation even if that allocation contains more than on RP including nested RPs. So from that API you will get the whole allocation21:04
bauzasefried: see https://review.openstack.org/#/c/589085/21:04
bauzasgibi: say it again21:04
bauzasah21:04
bauzasgibi: I'll get N allocations against N inventories, right?21:05
efriedbauzas: Ack, I'm on that guy, thx21:05
gibibauzas: GET /allocations/<instance_uuid> returns the allocation of the instance that contains multiple RPs including nested one21:05
gibibauzas: yes. one consumer (the instance_uuid) mapped to a list of allocations21:05
gibibauzas: but you cannot tell placement only return a subset of those allocations21:06
efriedwhy would you want to do that?21:06
gibiefried: I don't :)21:06
*** itlinux has joined #openstack-nova21:06
bauzasgibi: I don't get why I would restrict21:06
efriedwe're in violent agreement, then21:06
*** lbragstad has quit IRC21:06
gibibauzas: then I missunderstood "22:57 < bauzas> gibi: do we have any way to ask placement whether we have allocations in a subtree "21:06
bauzasgibi: if I'm getting all the allocations including the nested ones, that's perfect, nope?21:06
gibibauzas: I thought you only want to get the nested one. But sure you get all of them today21:07
bauzasgibi: I meant I wanted to make sure we were getting all the allocations including the "nested" ones21:07
gibibauzas: then you are good with the today's API21:07
bauzasthen I think we're cool, no ?21:07
efriedoh - you're asking for some form of /resource_provider/{u}/allocations where we get a subtree?21:07
bauzas\o/21:07
bauzasefried: no21:07
efriedtbc, that route gives you all allocations on the provider (from all consumers)21:08
bauzasefried: the context is, what do we need for nested RPs to be correctly scheduled21:08
efriedmm21:08
bauzasefried: and gibi pointed out we were looking up allocations on some cases21:08
efriedmm21:08
bauzasso I just wanted to make sure we were getting'em, including the ones that were against inventories on children21:09
efriedmm21:09
efriedand, for that matter, sharing providers21:09
bauzasyou know, if you have teeth problem, I can help you :p21:09
bauzasin my case, for the nested RPs21:09
bauzasnot for shared21:09
gibibauzas: what you will not get from placement is the mapping between granular request groups in the a_c request and the allocation that fulfills that group21:09
* bauzas google translates21:10
efried"not for shared" - yes, for shared too. If you had any. Which you kinda shouldn't.21:10
gibibauzas: this is where the spec https://review.openstack.org/#/c/597601/ comes into picture21:10
*** gbarros has joined #openstack-nova21:10
bauzasnah, kidding21:10
bauzasok21:10
bauzasgibi: ok, I'll look at the spec later in the day21:11
bauzasnaichuans: FWIW, I'm gonna creating a vgpu-stein BP21:11
gibibauzas: will GPUs use granular requet groups?21:11
bauzasgibi: it will for NUMA affinity21:11
*** savvas_ has joined #openstack-nova21:12
gibibauzas: then for NUMA you might hit the problem I describe in that spec21:12
bauzasyup21:12
bauzasfor the simple case where VGPUs are on a child but other resources are on root RP, we're all good21:12
bauzasprovided I'm getting RPs that support both requesrs21:13
efriedgibi: Skimming the spec, I don't think it goes far enough, I'm afraid.21:13
bauzasrequests, ie. VGPU in one child that's necessarly on the root RP having enough resources21:13
*** sorrison has quit IRC21:13
gibiefried: comments are very welcome :)21:13
* bauzas nods21:13
bauzasoh, awesome, there is already a vgpu-stein BP https://blueprints.launchpad.net/nova/+spec/vgpu-stein21:14
bauzasnaichuans: nevermind then my previous point21:14
*** itlinux has quit IRC21:15
*** sorrison has joined #openstack-nova21:15
openstackgerritLee Yarwood proposed openstack/nova stable/pike: [Stable Only] Remove soft-deleted instances from quota_usages  https://review.openstack.org/60245921:15
*** itlinux has joined #openstack-nova21:15
*** savvas has quit IRC21:16
efriedgibi: If I wind up with more than one value in my 'contributes_to' list, I'm right back where I started not knowing which request group corresponds to which chunk of the response.21:18
*** hshiina has joined #openstack-nova21:19
gibiefried: if you mean group_policy=None and two request groups end up fulfilled by the same RP then you are correct that still sucks21:19
efriedgibi: So21:19
gibiefried: I forget about that case21:19
efriedThe alternative you suggest at the end, breaking the allocation request up...21:19
bauzasoh shit, I just realize that the next PTG will litterally be 1 week after Brexit, and I'm usually flying BA21:19
bauzasthat could become fun21:19
efriedif you do that keyed by request group instead, you're golden.21:19
efriedgibi: ^21:19
efriedSucks to have to change the allocation request syntax, for sure, but there's no other complete solution.21:20
gibiefried: good point. I will need to work on that spec21:20
efriedgibi: Even for a granular request you have the problem.21:21
*** priteau has quit IRC21:23
*** slaweq has quit IRC21:24
*** bhagyashris has quit IRC21:24
*** bhagyashris has joined #openstack-nova21:24
*** luksky has quit IRC21:25
*** savvas has joined #openstack-nova21:25
*** lbragstad has joined #openstack-nova21:26
*** mvenesio has quit IRC21:26
efriedgibi: e.g. resources2=BW:100&resources3=BW:200&group_policy=none21:26
efriedtoday you might get back BW:300 from a single provider, which is :(21:26
openstackgerritLee Yarwood proposed openstack/nova stable/ocata: [Stable Only] Remove soft-deleted instances from quota_usages  https://review.openstack.org/57065321:26
*** s10 has joined #openstack-nova21:27
*** rcernin has joined #openstack-nova21:29
*** savvas_ has quit IRC21:29
gibiefried: true21:30
*** priteau has joined #openstack-nova21:30
bauzasgibi: efried: that's the exact problem I hit for VGPU affinity, right?21:31
*** hshiina has quit IRC21:31
bauzasif I want to have this NUMA node and that VGPU to be on the same tree21:31
efriedbauzas: It's less of a problem for VGPU, which is single unit21:31
efriedbauzas: Oh, no, that's a separate problem.21:31
bauzassorry, I learnt German and English at college, not Placement :p21:32
*** slaweq has joined #openstack-nova21:32
*** gbarros has quit IRC21:32
*** itlinux has quit IRC21:36
*** itlinux has joined #openstack-nova21:37
*** med_ has quit IRC21:42
*** awaugama has quit IRC21:42
*** itlinux has quit IRC21:50
openstackgerritSylvain Bauza proposed openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt  https://review.openstack.org/60247421:53
bauzasoh, we're not advertising on nova-specs changes ?21:53
bauzasI mean, I don't openstackgerrit telling my spec21:54
openstackgerritSylvain Bauza proposed openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt  https://review.openstack.org/60247421:55
bauzasmmm, strange but ok21:56
openstackgerritsean mooney proposed openstack/os-vif master: add support for generic tap device plug  https://review.openstack.org/60238421:58
bauzasmriedem: given what we said yesterday, shall we put a procedural -2 on https://review.openstack.org/#/c/599208/ and say we can't do it until https://review.openstack.org/#/q/topic:use-nested-allocation-candidates+(status:open+OR+status:merged) is merged ?21:59
mriedembecause once the vgpu is on the child provider, vgpu scheduling would be broken?22:01
*** itlinux has joined #openstack-nova22:01
*** itlinux has quit IRC22:02
bauzasmriedem: yup22:03
bauzasno valid hosts22:04
bauzassince it wouldn't find any RP that fulfills the VGPU resource class22:04
mriedembauzas: i guess? alternatively, we should have a functional test that does the reshape and then tries to schedule to the vgpu in the child - which we said we wanted as part of the checklist for placement extraction22:04
bauzasmriedem: yeah, I discussed that with gibi22:04
bauzasmriedem: the functional test should see that22:04
mriedemwe do have functional tests for reshaper with the fake virt driver,22:04
mriedemwe just don't try to schedule to that host after the reshape22:05
*** dpawlik has quit IRC22:05
openstackgerritMerged openstack/nova stable/rocky: Add functional test for affinity with multiple cells  https://review.openstack.org/59973122:05
bauzasmriedem: cool, I know what to do for next week22:05
mriedemthe functional test for reshaper mocks out the reshape code in the test itself; i suggested to efried that we move that to a fake virt driver - and if we did that, then i think we can just bolt the post-reshape scheduler request onto the same test22:05
mriedemmaybe we don't need to move that into a fake virt driver, idk22:06
*** dpawlik has joined #openstack-nova22:06
mriedembut would be nice if we can just add a few lines to that existing test to see if this passes or fails22:06
bauzasmriedem: lemme test that22:07
mriedembauzas: https://github.com/openstack/nova/blob/0ba34a818414823eda5e693dc2127a534410b5df/nova/tests/functional/test_servers.py#L183522:07
mriedemyeah so at the end, just try to create an instance on that host with vgpu and see if it blows up22:08
mriedemif it fails, we can assert it should fail until https://review.openstack.org/#/q/topic:use-nested-allocation-candidates+(status:open+OR+status:merged) fixes it22:08
mriedemand rebase those on top22:08
mriedemLIKE A CHAMP22:08
efriedmriedem: What ought to happen is: the fake libvirt driver should (by default? based on fixture setup?) call the real libvirt driver's impl of update_provider_tree, which should be modified to do reshaping for vgpus in this cycle (and maybe other things) so we can actually test those things in functional tests.22:08
bauzasmriedem: https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L1835 ?22:09
bauzasoh shit, sorry22:09
mriedembauzas: yes just linked that above22:09
mriedemgawd!22:09
efriedmriedem: But we should (still) absolutely not move the mocked-up update_provider_tree-of-bizarreness from that mock to the fake libvirt driver. It just doesn't make sense.22:09
bauzasI should employ someone to read my IRC22:09
mriedemefried: i will cut your damn letter jacket22:09
mriedemefried: honestly i don't care how it's done22:09
efriedJust try it. Actually, the last guy who said that was also named Matt.22:09
mriedemjust that it's tested22:09
openstackgerritsean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif  https://review.openstack.org/60243222:09
openstackgerritLee Yarwood proposed openstack/nova stable/rocky: Refix disk size during live migration with disk over-commit  https://review.openstack.org/60247722:15
openstackgerritLee Yarwood proposed openstack/nova stable/queens: Refix disk size during live migration with disk over-commit  https://review.openstack.org/60247822:15
jrollmelwitt: https://docs.openstack.org/ironic/latest/install/configure-nova-flavors.html22:21
melwittjroll: yeah, I'm pretty sure that override will _not_ stop quota counting from considering and summing the values in the flavor22:23
*** sean-k-mooney has joined #openstack-nova22:24
*** rtjure has quit IRC22:25
sean-k-mooneymriedem: o/ fixed now on irc22:25
melwittjroll: we count the 'vcpus' and 'memory_mb' columns in the 'instances' table to count as quota usage. and I don't think the extra spec overrides will affect how those columns are populated22:26
melwittand those values come from the flavor22:26
mriedemsean-k-mooney: https://etherpad.openstack.org/p/nova-network-removal-rocky22:30
mriedemthere are still some apis to investigate there22:31
sean-k-mooneymriedem: thanks i have bookmarked it ill take a look22:31
*** savvas has quit IRC22:32
*** savvas has joined #openstack-nova22:33
*** lyan has quit IRC22:34
openstackgerritJay Pipes proposed openstack/nova master: remove virt driver requires_allocation_refresh  https://review.openstack.org/60248122:34
*** savvas_ has joined #openstack-nova22:35
*** savvas has quit IRC22:39
jrollmelwitt: awesome, thanks :)22:39
*** jaosorior has quit IRC22:43
*** alexchadin has joined #openstack-nova22:43
*** priteau has quit IRC22:47
*** eharney has quit IRC22:50
*** holser|ptg has quit IRC22:50
*** erlon has quit IRC22:51
*** tssurya has quit IRC22:53
*** macza has quit IRC23:08
*** sorrison has quit IRC23:10
*** takashin has left #openstack-nova23:11
*** bhagyashris has quit IRC23:11
*** jangutter_ has quit IRC23:12
*** mriedem has quit IRC23:12
*** alexchadin has quit IRC23:14
openstackgerritChris Dent proposed openstack/nova-specs master: List resource providers having inventory  https://review.openstack.org/60001623:17
*** lbragstad has quit IRC23:17
*** s10 has quit IRC23:18
*** s10 has joined #openstack-nova23:19
*** cfriesen has quit IRC23:19
*** s10 has quit IRC23:19
*** s10 has joined #openstack-nova23:19
*** s10 has quit IRC23:20
*** s10 has joined #openstack-nova23:20
*** lbragstad has joined #openstack-nova23:21
*** s10 has quit IRC23:21
*** s10 has joined #openstack-nova23:21
*** sean-k-mooney has quit IRC23:21
*** s10 has quit IRC23:22
*** sorrison has joined #openstack-nova23:22
*** s10 has joined #openstack-nova23:22
*** s10 has quit IRC23:22
*** slaweq has quit IRC23:24
*** Sukhdev has quit IRC23:27
*** erlon has joined #openstack-nova23:29
*** efried has quit IRC23:30
openstackgerritMerged openstack/nova stable/rocky: Make scheduler.utils.setup_instance_group query all cells  https://review.openstack.org/59973223:30
*** nicolasbock has quit IRC23:34
openstackgerritMerged openstack/nova stable/rocky: Fix DB archiver AttributeError due to wrong table name attribute used  https://review.openstack.org/59988123:35
openstackgerritSam Morrison proposed openstack/nova master: Allow ability for non admin users to use all filters on server list.  https://review.openstack.org/52655823:37
*** tetsuro has quit IRC23:39
*** cfriesen has joined #openstack-nova23:41
*** lbragstad has quit IRC23:44
*** priteau has joined #openstack-nova23:44
*** cdent has quit IRC23:47
*** mchlumsky has quit IRC23:47
*** jangutter has joined #openstack-nova23:49
*** jangutter has quit IRC23:50
*** priteau has quit IRC23:50
*** jangutter has joined #openstack-nova23:53

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!