Friday, 2020-09-11

*** zzzeek has joined #openstack-nova00:02
*** bbowen has quit IRC00:04
*** martinkennelly has quit IRC00:26
*** rcernin has quit IRC00:28
*** rcernin has joined #openstack-nova00:31
luyaostephenfin: Hi, recheck succeeds at '00:34
luyaoRollback when reset running live migration' https://review.opendev.org/#/c/747899/, thanks  for review00:34
*** yingjisun has joined #openstack-nova00:53
*** yingjisun has quit IRC00:58
*** tetsuro has quit IRC01:00
*** hamalq_ has quit IRC01:03
*** Liang__ has joined #openstack-nova01:30
openstackgerritzhufl proposed openstack/nova master: Fix invalid assert_has_calls  https://review.opendev.org/75114301:30
*** lbragstad_ has joined #openstack-nova01:33
*** jhui has joined #openstack-nova01:34
*** yingjisun has joined #openstack-nova01:35
*** lbragstad has quit IRC01:36
*** artom has quit IRC02:12
*** lbragstad_ has quit IRC02:17
*** Liang__ has quit IRC02:24
openstackgerritEric Xie proposed openstack/nova master: Fix error that cannot overwrite policy rule for 'forced_host'  https://review.opendev.org/75115202:24
*** Liang__ has joined #openstack-nova02:24
*** zzzeek has quit IRC02:27
openstackgerritMerged openstack/nova master: Track error migrations in resource tracker  https://review.opendev.org/74528102:31
*** zzzeek has joined #openstack-nova02:33
*** jhui_ has joined #openstack-nova02:44
*** jhui has quit IRC02:44
*** jhui_ is now known as jhui02:44
*** suryasingh has joined #openstack-nova02:50
*** gyee has quit IRC02:54
openstackgerritRui Zang proposed openstack/nova master: DO NOT REVIEW  https://review.opendev.org/75115602:56
*** mkrai has joined #openstack-nova02:58
*** dave-mccowan has quit IRC03:06
*** mkrai has quit IRC03:46
*** brinzhang_ has joined #openstack-nova03:52
*** rcernin has quit IRC03:53
*** brinzhang0 has quit IRC03:55
*** yingjisun has quit IRC04:02
*** yingjisun has joined #openstack-nova04:02
*** yingjisun has quit IRC04:03
*** rcernin has joined #openstack-nova04:15
*** ratailor has joined #openstack-nova04:18
*** jhui has quit IRC04:20
*** lemko8 has joined #openstack-nova04:25
*** lemko has quit IRC04:28
*** lemko8 is now known as lemko04:28
*** tetsuro has joined #openstack-nova04:29
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-nova04:33
*** Liang__ has quit IRC04:37
*** Liang__ has joined #openstack-nova04:38
*** hongbin has quit IRC04:46
*** mkrai has joined #openstack-nova04:49
*** manuvakery has joined #openstack-nova04:52
*** lemko7 has joined #openstack-nova04:53
*** lemko has quit IRC04:54
*** lemko7 is now known as lemko04:54
*** ratailor has quit IRC05:06
*** ratailor has joined #openstack-nova05:07
*** ratailor has quit IRC05:11
*** aj_mailing has joined #openstack-nova05:23
*** xek has joined #openstack-nova05:35
*** slaweq has joined #openstack-nova06:01
*** jsuchome has joined #openstack-nova06:07
*** tetsuro has quit IRC06:11
*** tetsuro has joined #openstack-nova06:14
*** slaweq has quit IRC06:18
*** jhui has joined #openstack-nova06:21
*** jhui has left #openstack-nova06:21
*** zzzeek has quit IRC06:23
*** zzzeek has joined #openstack-nova06:24
*** xek has quit IRC06:27
*** rcernin has quit IRC06:28
*** yedongcan has joined #openstack-nova06:33
*** ralonsoh has joined #openstack-nova06:43
*** rcernin has joined #openstack-nova06:45
*** rcernin has quit IRC06:45
*** rcernin has joined #openstack-nova06:46
*** zzzeek has quit IRC06:51
*** zzzeek has joined #openstack-nova06:53
*** dklyle has quit IRC06:55
*** manuvakery has quit IRC07:02
*** slaweq has joined #openstack-nova07:03
gibisean-k-mooney: sounds interesting. I will have to dig into it a bit07:11
*** mkrai has quit IRC07:23
*** tetsuro has quit IRC07:25
*** yedongcan has quit IRC07:25
*** tesseract has joined #openstack-nova07:27
*** tetsuro has joined #openstack-nova07:28
*** links has joined #openstack-nova07:33
bauzasgood Friday Nova07:39
bauzasgibi: I have a specific concern on the relnote for the vmwareapi driver undeprecation but I forgot to tell it during the meeting yesterday07:39
bauzasgibi: https://review.opendev.org/#/c/742407/5/releasenotes/notes/undeprecate-vmware-victoria-2eaf5d877733f8d9.yaml07:39
bauzasnot worth holding the change, but maybe a FUP could be nice07:39
gibibauzas: will push a follow up, thanks07:42
bauzasthanks07:42
gibiand good Friday morning to you too07:42
bauzasgibi: and I -w'd the prelude because of the xenapi change still in the gate + this one that I think which needs to be in the prelude07:42
gibiyeah, I kikked the xenapi series this morning with a recheck07:43
gibi*kicked07:43
bauzasI saw07:43
*** tetsuro_ has joined #openstack-nova07:45
*** tetsuro has quit IRC07:48
*** mkrai has joined #openstack-nova07:50
*** martinkennelly has joined #openstack-nova08:02
*** yankcrime has joined #openstack-nova08:04
*** songwenping__ has joined #openstack-nova08:09
*** brinzhang0 has joined #openstack-nova08:09
*** songwenping_ has quit IRC08:12
*** brinzhang_ has quit IRC08:12
*** tetsuro has joined #openstack-nova08:20
*** tetsuro_ has quit IRC08:23
sean-k-mooneyo/08:23
sean-k-mooneygibi: so i would still like to procee with the deprecation of the az filter and enableing the placment version by default in Victoria since its behavior is equivalent, but i think i am going to split out the Comptefilter changes and look to adress that in Wallaby when we have more time to think about it08:25
sean-k-mooneyi dont know if we want to reuse the COMPUTE_STATUS_DISABLED trait for force down or not and i dont really want ot rush figuring that out08:27
sean-k-mooneyi think having a sperate COMPUTE_STATUS_DOWN trait is proably more correct but that would need an os-traits release and i dont think this warrents a FFE for os-triats since its just tech debt removal08:28
gibisean-k-mooney: make sense, keep the ComputeFilter then for V and please open a bug to track the problem you detected with the pre-filter08:28
sean-k-mooneyyep i can do that that was actully going to be my question should i open a bug or blueprint. e.g. is this an oversight in the original filter that it ignore force down or a new feature08:29
gibii consider this as a bug08:31
sean-k-mooneycool ill file one so then respin the patch to drop the computefitler changes08:35
gibithanks08:36
bauzassean-k-mooney: wait, what ?08:38
bauzassean-k-mooney: which filter are you going to deprecate ?08:38
*** tetsuro_ has joined #openstack-nova08:39
bauzasAZFilter or ComputeFilter ?08:39
sean-k-mooneyi was deprecating both08:39
sean-k-mooneybut the behavior of the ComputeFilter08:39
sean-k-mooneyis not identical to the placment prefilter for compute status08:39
sean-k-mooneybecause the prefilter does not take accound of force_down08:39
bauzasok, now I better understand and yes08:39
bauzasI was about to say this08:39
bauzasbut... are we considering the AZfilter deprecation for the Victoria cycle ?08:40
sean-k-mooneyso for now im just going to depercate teh az filter then file a bug for the gap in behavior and we can figure out the correct way to handel force_down in wallaby08:40
bauzasI'd say it's a Wallaby thing08:40
sean-k-mooneybauzas: yes i was08:40
sean-k-mooneyhttps://review.opendev.org/#/c/745605/08:41
sean-k-mooneyi have had that open for a while but only dug into the fun test fialure properly yesterday08:41
bauzasok, I feel we need to talk then08:41
bauzasbecause I agree with the fact that the AZfilter is on par with the pre-filter08:42
*** tetsuro has quit IRC08:42
*** derekh has joined #openstack-nova08:42
sean-k-mooneyi could delay this if you want by the way but if you dont have an object ot the AZfilter then i would prefer to proceed with just that08:42
bauzasbut I know ops use filters by ordering them like they want08:42
sean-k-mooneyyes08:43
sean-k-mooneybut the AZ filter always goes first right08:43
bauzasno08:43
sean-k-mooneyno?08:43
bauzasby default, ComputeFilter goes first08:43
bauzasand then AZ08:43
bauzasif we deprecate the AZ, this will change08:44
bauzasoh shit08:44
bauzasI'm wrong08:44
openstackgerritDat Le proposed openstack/nova stable/rocky: Fix unplugging VIF when migrate/resize/rebuild VM between hosts have difference VIF type  https://review.opendev.org/75118608:44
sean-k-mooneyya i was going to say https://review.opendev.org/#/c/745605/3/nova/conf/scheduler.py08:44
bauzasthat's... disturbing08:44
sean-k-mooneyit makes sense from an information gain point of view08:45
bauzasI'm almost sure this wasn't the case years ago08:45
bauzasbut anyway08:45
bauzasin this case, this sounds legit08:45
bauzassean-k-mooney: if so, I'll review your deprecation patch08:45
sean-k-mooneythe point is in anycase to move both operation eventually to the placment query08:46
sean-k-mooneysince in that case ordering does not matter08:46
sean-k-mooneyat least form teh point of view of the query construction08:47
*** tosky has joined #openstack-nova08:47
sean-k-mooneybauzas: ill ping you with the update patch with just the AZfilter deprecation in a few once its ready08:48
openstackgerritBalazs Gibizer proposed openstack/nova master: Spell out 3pp in vmware undeprecation reno  https://review.opendev.org/75118808:48
sean-k-mooneywhats 3pp08:48
sean-k-mooneyoh hehe08:49
sean-k-mooneythird party08:49
sean-k-mooneyi see that was bauzas nit08:49
bauzassean-k-mooney: also, I left a comment on your deprecation patch, having the sheduler being underoptimized is not a performance issue08:51
*** tetsuro_ has quit IRC08:53
*** xek has joined #openstack-nova08:53
sean-k-mooneybauzas: thanks that said the orginal doc state that enabling the prefilter improves performance, so when i flip the docs to state its enabled by default now does it not make sense to now also state that reverting to the az filter woudl reduce performance08:55
sean-k-mooneybauzas: i can remove that i just basically inverted the previous statement but if you dont think it adds value ill just drop it08:56
sean-k-mooneybauzas: you were refering to your comment here right https://review.opendev.org/#/c/745605/3/doc/source/admin/availability-zones.rsthttps://review.opendev.org/#/c/745605/3/doc/source/admin/availability-zones.rst08:56
sean-k-mooneythe funny thing is that change was orginally just ment to be the az filter08:57
sean-k-mooneyi only added the compute filter when i was doing the doc update and was like waith there is a prefilter that "does the same thing" that can go too. i was wrong :(08:58
*** vishalmanchanda has joined #openstack-nova09:00
*** k_mouza has joined #openstack-nova09:04
*** rcernin has quit IRC09:10
openstackgerritFederico Ressi proposed openstack/nova master: Log stack trace when fails launching a defined domain  https://review.opendev.org/75065209:13
openstackgerritFederico Ressi proposed openstack/nova master: Log stack trace when fails launching a defined domain  https://review.opendev.org/75065209:13
bauzassean-k-mooney: I know, we have a lot of docs that are wrong by saying "heh, prefilters are better for performance"09:14
bauzastbc: NO09:15
bauzasa prefilter just allows you to get more candidates09:15
sean-k-mooneybauzas: that is not true09:15
*** ratailor has joined #openstack-nova09:16
bauzasfilters perfomance is not an issue09:16
sean-k-mooneybauzas: if you enable a prefilter and disable a schduler filter the db outperform python09:16
bauzassean-k-mooney: again, it's wrong09:16
sean-k-mooneybauzas: in general yes09:16
bauzaslots of folks (eg. CERN) looked at the scheduler performance09:16
sean-k-mooneybauzas: we have see this form operators reporting that its faster09:16
bauzasand none of them said that the filters were outperfomed09:16
bauzasthe problem was with the DB09:17
sean-k-mooneyi acutlly tought cern said it did09:17
bauzasnot the filters09:17
*** xek has quit IRC09:17
sean-k-mooneyi guess it woudl only help09:17
sean-k-mooneyas you said if we dont return the 1000 limit on allocitons09:17
sean-k-mooneye.g. if it reduces the set below that09:17
bauzasno, the problem is that NOBODY thought about the performance for getting more than 1000 candidates09:17
bauzasI mean the API performance09:18
bauzasso we said, "ok let's limit to 1000 candidates"09:18
bauzasand then we said "oh but then shit, we won't then verify all the candidates"09:18
bauzasand then that's why we started using prefilters09:18
sean-k-mooneyya i know that i was there09:19
sean-k-mooneyand NOBODY is a strech09:19
bauzasif the Placement API was able to provide *all* candidates, the performance in between checking all of them with a filter, or instead querying the Placement *before* wouldn't be differnet09:19
sean-k-mooneywe did discuss the subseting problem before we started using placment09:19
sean-k-mooneybauzas: the different is the number of candiates pass to the schduler09:20
bauzasmeh, the ship has sailed either way09:20
sean-k-mooneyif we have 1000 nodes in 2 az of 50009:20
bauzasagain, not a problem09:20
sean-k-mooneythe az prefilter will limit the repsoce to at most 50009:20
sean-k-mooneywithout it all 1000 nodes asumming all other thing are equall would pass to the scudler09:21
sean-k-mooneyand the az filter would have to filter the 500 nodes then09:21
sean-k-mooneywhich si where ordering is importnat09:21
sean-k-mooneyand why the azfilter come before the expensive ones09:21
sean-k-mooneyanyway i can jsut drop the last line that menations performance09:22
bauzassean-k-mooney: tbc, filters aren't expensive09:23
bauzaseven the NUMATopology one, per say09:23
bauzascompared to how long it takes for getting all the candidates09:23
sean-k-mooneythat has not been my experince but i also have not seen data for large clouds09:24
sean-k-mooneyit really jsut comes down to how many candiate you need to check09:24
sean-k-mooneywell hosts09:24
sean-k-mooneybauzas: by the way we both also agree that schduling is not the slow part fo booting a vm09:25
sean-k-mooneybauzas: but what your really arguing is the previous docs were wrong09:25
bauzassean-k-mooney: don't get me wrong09:26
bauzasplacement is better for large clouds compared to what we had in the past09:26
sean-k-mooneywhen they said enabling the prefilter would improve performance09:26
bauzassean-k-mooney: true, I'm just saying 'folks, don't trust everything written or said by someone you care"09:27
bauzasand rather experiment yourselves or at least look at more than a single person09:27
* bauzas tries hard to not make a corollar with a certain molecule 09:27
sean-k-mooneyso tl;dr drop the last line and we good ?09:28
bauzasI'd say so09:28
sean-k-mooneycool09:28
sean-k-mooneythat also means i dont need to figure out how to adress gibis comment if i jsut delete the line :)09:28
*** mkrai has quit IRC09:49
*** mkrai has joined #openstack-nova09:49
*** brinzhang_ has joined #openstack-nova09:50
*** brinzhang0 has quit IRC09:54
openstackgerritLee Yarwood proposed openstack/nova stable/rocky: libvirt: Handle VIR_ERR_DEVICE_MISSING when detaching devices  https://review.opendev.org/74241709:56
openstackgerritLee Yarwood proposed openstack/nova stable/rocky: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0  https://review.opendev.org/74736109:56
*** brinzhang0 has joined #openstack-nova10:04
*** brinzhang_ has quit IRC10:07
*** Liang__ has quit IRC10:09
*** ratailor_ has joined #openstack-nova10:09
*** ratailor has quit IRC10:11
*** mnaser has quit IRC10:11
*** donnyd has quit IRC10:11
*** pas-ha has quit IRC10:11
*** fyx has quit IRC10:11
*** ttx has quit IRC10:11
*** jrosser has quit IRC10:11
*** andreykurilin has quit IRC10:11
*** corvus has quit IRC10:11
*** andreykurilin has joined #openstack-nova10:11
*** corvus has joined #openstack-nova10:11
*** pas-ha has joined #openstack-nova10:11
*** donnyd has joined #openstack-nova10:11
*** jrosser has joined #openstack-nova10:11
*** fyx has joined #openstack-nova10:16
*** mnaser has joined #openstack-nova10:16
*** ttx has joined #openstack-nova10:16
*** mnaser has quit IRC10:17
*** mnaser has joined #openstack-nova10:17
*** fyx has quit IRC10:17
*** fyx has joined #openstack-nova10:17
*** brinzhang_ has joined #openstack-nova10:17
lyarwoodrandom question for the channel but does anyone know if CODEOWNERS has ever been talked about within OpenStack? https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners & https://docs.gitlab.com/ee/user/project/code_owners.html10:17
lyarwoodI appreciate this is platform specific without us writing automation around it for opendev but it seems like a nice codeified way of keeping https://wiki.openstack.org/wiki/Nova#People up to date10:18
*** brinzhang0 has quit IRC10:20
sean-k-mooneynot familar with it10:21
*** tkajinam has quit IRC10:22
sean-k-mooneybut it does not really aligh with how we work since we dont have subsystem maintianers10:22
sean-k-mooneyif we did then yes10:22
sean-k-mooneylyarwood: i didnt even know that existed10:22
lyarwoodwe've listed contacts for areas of the code on the wiki for years now10:22
lyarwoodyeah I only just heard about it tbh10:23
sean-k-mooneyright but that is not what i tought we used10:23
sean-k-mooneyi tought we only had https://wiki.openstack.org/wiki/Nova/BugTriage10:23
sean-k-mooneythe tag owners10:23
lyarwoodwe don't use CODEOWNERS for that, I'm suggesting we do so we can keep this in the actual git tree vs a random wiki page10:23
* lyarwood adds it to the PTG pad10:24
sean-k-mooneyright but i dont think the peopel section makes sense10:24
sean-k-mooneythe tags might10:24
sean-k-mooneythe bullets in the people section dont correspond to files or subsystmes10:24
lyarwoodah sorry I was thinking about Developer Contacts10:26
lyarwoodbut anyway I'd rather this lived in tree as there appears to be tooling to do quick lookups locally10:27
lyarwoodwithout relying on a given platform10:27
sean-k-mooneyalso did not know that existed10:27
sean-k-mooneybut ya i can see it just bing a file in the git repo i guess10:28
sean-k-mooneyjust a simiple yaml file or rst in the docs under the contibutors section10:28
sean-k-mooneylyarwood: i wonder how much traffic if any the wiki gets10:29
sean-k-mooneyi have very very rearly checked any of the wikis since we started using specs10:29
sean-k-mooneyi do if google brings be there but that is very rare10:30
lyarwoodsean-k-mooney: yeah we could also render it somehow in the docs I'm sure10:30
lyarwoodsean-k-mooney: I also hardly ever go there, just had a patch against stable that needed to go to master first so went looking for docs10:31
lyarwoodsean-k-mooney: found the wiki pages and recalled reading about CODEOWNERS somewhere the other day10:31
sean-k-mooneyand the wiki was the only place?10:31
*** bbowen has joined #openstack-nova10:31
*** jangutter_ has joined #openstack-nova10:31
*** aj_mailing has quit IRC10:31
sean-k-mooneyi dont know if we even mention the wiki in the contibutors guide anymore10:31
lyarwoodsean-k-mooney: no I've linked to the contrib guide10:32
lyarwoodsean-k-mooney: it links to the wiki in places iirc10:32
sean-k-mooneythe only wiki page i check semi regularly is https://wiki.openstack.org/wiki/Meetings/Nova10:32
* lyarwood -> coffee brb10:33
brinzhang_gibi: hi10:34
brinzhang_gibi: https://review.opendev.org/#/c/729563/13/nova/tests/unit/compute/test_api.py@7685 here, what did your mean?10:34
*** jangutter has quit IRC10:35
gibibrinzhang_: I don't expect any change in your patch due to that comment10:36
gibiit is just a note why we are collecting arqs for shelve even if during shelve we dont use them10:37
brinzhang_gibi: ack, thanks, I misunderstand that words10:37
brinzhang_yes10:37
gibisorry, I could have been clearer in my words10:37
brinzhang_np, I will update later10:37
gibithanks10:39
brinzhang_gibi: https://review.opendev.org/#/c/729563/13/nova/tests/fixtures.py@2601 this can't change to arq_host_rp_list = CyborgFixture.bindings_by_instance.get(instance_uuid, [])10:39
openstackgerritStephen Finucane proposed openstack/nova stable/queens: [placement] Add status and links fields to version document at /  https://review.opendev.org/75123910:39
brinzhang_in Line2601, if the arq_host_rp_list==[], that will be an error10:39
brinzhang_so I will keep this not change yet10:39
brinzhang_if arq_host_rp_list ==[], return []10:40
openstackgerritHemanth Nakkina proposed openstack/nova master: Update pci stat pools based on PCI device changes  https://review.opendev.org/74917510:41
openstackgerritStephen Finucane proposed openstack/nova stable/pike: [placement] Add status and links fields to version document at /  https://review.opendev.org/75124010:42
gibibrinzhang_: you are right, my mistake I mixed arq_host_rp_list with bound_arq_list10:42
brinzhang_gibi: https://review.opendev.org/#/c/729563/13/nova/tests/fixtures.py@255710:42
brinzhang_gibi: ack10:42
*** brinzhang_ has quit IRC10:52
openstackgerritLee Yarwood proposed openstack/nova master: Support old & new versions of svm and vmx traits  https://review.opendev.org/66051510:56
*** k_mouza_ has joined #openstack-nova10:56
lyarwood^ kashyap / stephenfin ; fixed up some nits quickly if you can review and ack10:56
openstackgerritBalazs Gibizer proposed openstack/nova master: Follow up for I67504a37b0fe2ae5da3cba2f3122d9d0e18b9481  https://review.opendev.org/75018410:56
stephenfinlyarwood: Done. Assume kashyap will comment if he's not happy with it10:57
openstackgerritBalazs Gibizer proposed openstack/nova master: Cleanup unused code  https://review.opendev.org/75094710:58
*** k_mouza has quit IRC10:58
*** jsuchome has quit IRC10:59
openstackgerritLee Yarwood proposed openstack/nova master: trivial: Remove useless comment  https://review.opendev.org/74847511:00
lyarwoodstephenfin: ^ and that11:00
stephenfinalso done :)11:01
*** artom has joined #openstack-nova11:01
*** rcernin has joined #openstack-nova11:06
*** songwenping_ has joined #openstack-nova11:13
openstackgerritMerged openstack/nova master: Spell out 3pp in vmware undeprecation reno  https://review.opendev.org/75118811:14
sean-k-mooneystephenfin: just took another pass at https://review.opendev.org/#/c/749175/3 it now has a functional test. still want a reno but could you also take a look11:15
stephenfinsean-k-mooney: will do11:16
*** mkrai has quit IRC11:16
*** songwenping__ has quit IRC11:16
*** songwenping_ has quit IRC11:18
*** rcernin has quit IRC11:20
hemanth_nsean-k-mooney: thanks for the quick review, i will work on your comments11:21
openstackgerritBrin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support  https://review.opendev.org/72956311:23
*** songwenping_ has joined #openstack-nova11:24
sean-k-mooneyill be deploying an sriov env later this after noon so ill try and test it there too11:25
sean-k-mooneyi wont actully be able to test the device type changeing11:25
sean-k-mooneybut i can atleast assert it works fine normally11:26
openstackgerritLee Yarwood proposed openstack/nova stable/stein: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available  https://review.opendev.org/74761111:26
sean-k-mooneywhich i expect it will but i want to see if this fixes an edgecase i found when looking at sriov atach/detach11:26
openstackgerritLee Yarwood proposed openstack/nova stable/rocky: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available  https://review.opendev.org/74761211:26
openstackgerritLee Yarwood proposed openstack/nova stable/queens: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available  https://review.opendev.org/75124911:27
sean-k-mooneyhemanth_n: i assume you would like this to be backported to all maintained branches too by the way11:27
sean-k-mooneyi didnt see anything that would prevent that but we might want to updat the bug with the other branches since i did not do that when i triaged it11:28
kashyapstephenfin: Just reading the scroll, is it realted to VMX traits?11:30
kashyap(Ans: yes)11:32
lyarwoodstephenfin: https://review.opendev.org/#/c/747746/4/nova/compute/resource_tracker.py - question in here, couldn't we use `reverting` here?11:35
lyarwoodbrb11:35
*** mgariepy has quit IRC11:37
*** ttx has quit IRC11:39
*** ttx has joined #openstack-nova11:39
*** ttx has quit IRC11:40
*** ttx has joined #openstack-nova11:40
stephenfinlyarwood: replied11:43
hemanth_nsean-k-mooney: yes would like to backport,  will do11:44
openstackgerritMerged openstack/nova master: Add regression test for bug 1893284  https://review.opendev.org/74877911:44
openstackbug 1893284 in OpenStack Compute (nova) "Per-user quotas not scoped correctly when creating a server" [Low,In progress] https://launchpad.net/bugs/1893284 - Assigned to melanie witt (melwitt)11:44
*** xek has joined #openstack-nova11:45
openstackgerritFederico Ressi proposed openstack/nova master: Log stack trace when fails launching a defined domain  https://review.opendev.org/75065211:46
openstackgerritFederico Ressi proposed openstack/nova master: Log stack trace when fails launching a defined domain  https://review.opendev.org/75065211:46
*** manuvakery has joined #openstack-nova11:47
*** raildo has joined #openstack-nova11:48
*** xek has quit IRC12:12
*** mgariepy has joined #openstack-nova12:19
*** k_mouza_ has quit IRC12:23
*** k_mouza has joined #openstack-nova12:27
openstackgerritMerged openstack/nova master: Default user_id when not specified in check_num_instances_quota  https://review.opendev.org/74855012:31
openstackgerritMerged openstack/nova master: Fix an erroneous grammar in explanatory notes  https://review.opendev.org/73017612:31
*** raildo_ has joined #openstack-nova12:35
*** raildo has quit IRC12:35
*** jangutter has joined #openstack-nova12:36
*** jangutter_ has quit IRC12:39
*** ratailor_ has quit IRC12:42
openstackgerritMerged openstack/nova master: func: Remove references to attachment_id when volume_id is used  https://review.opendev.org/74966412:45
*** lbragstad_ has joined #openstack-nova12:49
*** zzzeek has quit IRC12:50
*** lbragstad_ is now known as lbragstad12:51
*** zzzeek has joined #openstack-nova12:51
*** raildo has joined #openstack-nova12:53
*** raildo_ has quit IRC12:53
*** zzzeek has quit IRC13:04
*** zzzeek has joined #openstack-nova13:07
openstackgerritStephen Finucane proposed openstack/nova master: xenapi: Remove driver and tests  https://review.opendev.org/74930413:11
openstackgerritStephen Finucane proposed openstack/nova master: virt: Remove 'is_xenapi' helper  https://review.opendev.org/74930513:11
openstackgerritStephen Finucane proposed openstack/nova master: xenapi: Remove Xen-only code paths  https://review.opendev.org/74930613:11
openstackgerritStephen Finucane proposed openstack/nova master: objects: Remove 'XenapiLiveMigrateData'  https://review.opendev.org/74930713:11
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Final cleanup  https://review.opendev.org/74930813:11
openstackgerritStephen Finucane proposed openstack/nova master: virt: Remove 'get_console_pool_info' API  https://review.opendev.org/74931113:11
openstackgerritStephen Finucane proposed openstack/nova master: virt: Remove 'post_interrupted_snapshot_cleanup' API  https://review.opendev.org/74931313:11
openstackgerritStephen Finucane proposed openstack/nova master: virt: Remove 'set_bootable' API  https://review.opendev.org/74931413:11
openstackgerritStephen Finucane proposed openstack/nova master: virt: Remove various aggregate APIs  https://review.opendev.org/74931713:11
openstackgerritStephen Finucane proposed openstack/nova master: virt: Remove 'get_per_instance_usage' API  https://review.opendev.org/74931813:11
openstackgerritStephen Finucane proposed openstack/nova master: api: Remove 'os-agents' API  https://review.opendev.org/74930913:11
openstackgerritStephen Finucane proposed openstack/nova master: objects: Remove Agent, AgentList  https://review.opendev.org/74931013:11
openstackgerritStephen Finucane proposed openstack/nova master: virt: Remove 'get_all_bw_counters' API  https://review.opendev.org/74931213:11
openstackgerritStephen Finucane proposed openstack/nova master: virt: Remove 'reset_network' API  https://review.opendev.org/74931513:11
openstackgerritStephen Finucane proposed openstack/nova master: virt: Remove 'change_instance_metadata' API  https://review.opendev.org/74931613:11
*** nweinber has joined #openstack-nova13:12
openstackgerritStephen Finucane proposed openstack/nova stable/queens: [placement] Add status and links fields to version document at /  https://review.opendev.org/75123913:16
*** jangutter_ has joined #openstack-nova13:18
openstackgerritStephen Finucane proposed openstack/nova stable/pike: [placement] Add status and links fields to version document at /  https://review.opendev.org/75124013:20
*** jangutter has quit IRC13:22
*** links has quit IRC13:27
*** jangutter has joined #openstack-nova13:31
*** jangutter_ has quit IRC13:34
*** dave-mccowan has joined #openstack-nova13:45
lyarwoodstephenfin: https://review.opendev.org/#/c/747746/4/nova/compute/resource_tracker.py - final clarification13:49
*** dave-mccowan has quit IRC13:50
stephenfinlyarwood: responded13:52
lyarwoodstephenfin: ack thanks LGTM then13:56
*** aj_mailing has joined #openstack-nova14:00
*** gmann is now known as gmann_afk14:05
openstackgerritMerged openstack/nova master: Support old & new versions of svm and vmx traits  https://review.opendev.org/66051514:10
*** jangutter_ has joined #openstack-nova14:22
*** aj_mailing has quit IRC14:22
*** aj_mailing has joined #openstack-nova14:22
openstackgerritArtom Lifshitz proposed openstack/nova master: Correctly disable greendns  https://review.opendev.org/75130214:25
*** jangutter has quit IRC14:25
artomlyarwood, ^^ after the bug call14:38
*** dklyle has joined #openstack-nova14:41
lyarwoodartom: yup already have it open14:43
openstackgerritTakashi Natsume proposed openstack/nova master: Remove six.moves  https://review.opendev.org/72722414:47
*** aj_mailing has quit IRC14:48
openstackgerritTakashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys  https://review.opendev.org/72775714:48
openstackgerritTakashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte  https://review.opendev.org/72777714:49
openstackgerritTakashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types  https://review.opendev.org/72809414:49
openstackgerritTakashi Natsume proposed openstack/nova master: Remove six.text_type (1/2)  https://review.opendev.org/72810914:50
*** songwenping__ has joined #openstack-nova14:59
openstackgerritTakashi Natsume proposed openstack/nova master: Remove six.text_type (2/2)  https://review.opendev.org/72811715:00
*** mlavalle has joined #openstack-nova15:01
*** songwenping_ has quit IRC15:02
*** jangutter has joined #openstack-nova15:02
*** jangutter has quit IRC15:03
*** jangutter has joined #openstack-nova15:03
*** xek has joined #openstack-nova15:05
*** jangutter_ has quit IRC15:06
*** mgariepy has quit IRC15:07
*** xek has quit IRC15:19
*** mkrai has joined #openstack-nova15:28
*** spatel has joined #openstack-nova15:28
*** spatel has quit IRC15:32
*** zzzeek has quit IRC15:32
*** zzzeek has joined #openstack-nova15:34
gibihave a nice weekend folks, I'm out15:40
openstackgerritRodrigo Barbieri proposed openstack/nova master: Allow tap interface with multiqueue  https://review.opendev.org/74853315:41
*** spatel has joined #openstack-nova15:43
*** spatel has quit IRC15:48
*** spatel has joined #openstack-nova15:48
*** mkrai has quit IRC15:49
*** mgariepy has joined #openstack-nova15:49
*** elod has quit IRC15:51
*** elod has joined #openstack-nova15:53
*** zzzeek has quit IRC15:55
*** zzzeek has joined #openstack-nova15:56
*** hamalq has joined #openstack-nova15:58
*** hamalq_ has joined #openstack-nova15:59
*** zzzeek has quit IRC16:03
*** hamalq has quit IRC16:03
*** zzzeek has joined #openstack-nova16:05
openstackgerritTakashi Natsume proposed openstack/nova master: doc: mark the max microversion for victoria  https://review.opendev.org/75134216:13
openstackgerritStephen Finucane proposed openstack/nova master: Remove write_to_file.  https://review.opendev.org/64508716:17
openstackgerritMerged openstack/nova stable/stein: Removed the host FQDN from the exception message  https://review.opendev.org/74961016:17
openstackgerritMerged openstack/nova master: Move revert resize under semaphore  https://review.opendev.org/74774616:17
*** derekh has quit IRC16:23
*** zzzeek has quit IRC16:26
*** zzzeek has joined #openstack-nova16:26
openstackgerritStephen Finucane proposed openstack/nova stable/ussuri: tests: Add reproducer for bug #1879878  https://review.opendev.org/75134916:32
openstackbug 1879878 in OpenStack Compute (nova) ussuri "VM become Error after confirming resize with Error info CPUUnpinningInvalid on source node " [Undecided,Confirmed] https://launchpad.net/bugs/187987816:32
openstackgerritStephen Finucane proposed openstack/nova stable/ussuri: Add generic reproducer for bug #1879878  https://review.opendev.org/75135016:32
openstackgerritStephen Finucane proposed openstack/nova stable/ussuri: functional: Don't inherit from 'ProviderUsageBaseTestCase'  https://review.opendev.org/75135116:32
openstackgerritStephen Finucane proposed openstack/nova stable/ussuri: Don't unset Instance.old_flavor, new_flavor until necessary  https://review.opendev.org/75135216:32
openstackgerritStephen Finucane proposed openstack/nova stable/ussuri: Move confirm resize under semaphore  https://review.opendev.org/75135316:32
openstackgerritStephen Finucane proposed openstack/nova stable/ussuri: Move revert resize under semaphore  https://review.opendev.org/75135416:32
openstackgerritTakashi Natsume proposed openstack/nova master: Update compute rpc version alias for victoria  https://review.opendev.org/75135516:33
*** artom has quit IRC16:35
*** artom has joined #openstack-nova16:35
openstackgerritArtom Lifshitz proposed openstack/nova master: Correctly disable greendns  https://review.opendev.org/75130216:42
*** mgariepy has quit IRC16:43
*** spatel has quit IRC16:46
*** raildo_ has joined #openstack-nova16:46
*** raildo has quit IRC16:49
*** mgariepy has joined #openstack-nova16:50
*** tobberydberg has quit IRC16:57
*** jangutter_ has joined #openstack-nova17:14
*** jangutter has quit IRC17:18
openstackgerritStephen Finucane proposed openstack/nova stable/train: Only allow one scheduler service in tests  https://review.opendev.org/75136217:20
openstackgerritStephen Finucane proposed openstack/nova stable/train: func tests: move _run_periodics() into base class  https://review.opendev.org/75136317:20
openstackgerritStephen Finucane proposed openstack/nova stable/train: Helper to start computes with different HostInfos  https://review.opendev.org/75136417:20
openstackgerritStephen Finucane proposed openstack/nova stable/train: tests: Add reproducer for bug #1879878  https://review.opendev.org/75136517:21
openstackbug 1879878 in OpenStack Compute (nova) ussuri "VM become Error after confirming resize with Error info CPUUnpinningInvalid on source node " [Undecided,In progress] https://launchpad.net/bugs/1879878 - Assigned to Stephen Finucane (stephenfinucane)17:21
openstackgerritStephen Finucane proposed openstack/nova stable/train: Add generic reproducer for bug #1879878  https://review.opendev.org/75136617:21
openstackgerritStephen Finucane proposed openstack/nova stable/train: Don't unset Instance.old_flavor, new_flavor until necessary  https://review.opendev.org/75136717:21
openstackgerritStephen Finucane proposed openstack/nova stable/train: Move confirm resize under semaphore  https://review.opendev.org/75136817:21
openstackgerritStephen Finucane proposed openstack/nova stable/train: Move revert resize under semaphore  https://review.opendev.org/75136917:21
*** gyee has joined #openstack-nova17:31
*** tesseract has quit IRC17:39
*** gmann_afk is now known as gmann17:42
*** k_mouza has quit IRC17:54
*** k_mouza has joined #openstack-nova17:54
openstackgerritElod Illes proposed openstack/nova stable/pike: DNM: zuul fix check  https://review.opendev.org/75137317:55
*** stephenfin has quit IRC17:56
*** aj_mailing has joined #openstack-nova17:59
openstackgerritElod Illes proposed openstack/nova stable/stein: DNM: zuul fix check  https://review.opendev.org/75137418:00
*** aj_mailing has quit IRC18:13
*** vishalmanchanda has quit IRC18:20
*** manuvakery has quit IRC18:27
*** ralonsoh has quit IRC18:36
*** mriedem has joined #openstack-nova18:39
*** tobberydberg has joined #openstack-nova18:58
*** aj_mailing has joined #openstack-nova19:03
*** tbachman has quit IRC19:07
*** tbachman has joined #openstack-nova19:10
*** johanssone_ has joined #openstack-nova19:13
*** johanssone has quit IRC19:15
*** xek has joined #openstack-nova19:47
*** jangutter has joined #openstack-nova19:55
*** jangutter_ has quit IRC19:58
*** tbachman_ has joined #openstack-nova19:58
*** tbachman has quit IRC19:58
*** tbachman_ is now known as tbachman19:58
*** nweinber has quit IRC20:07
*** tbachman has quit IRC20:37
*** tbachman has joined #openstack-nova20:39
*** tbachman_ has joined #openstack-nova20:42
*** tbachman has quit IRC20:44
*** tbachman_ is now known as tbachman20:44
*** raildo_ has quit IRC21:06
openstackgerritElancheran S proposed openstack/nova stable/train: Removes the delta file once image is extracted  https://review.opendev.org/74803521:14
openstackgerritsean mooney proposed openstack/nova master: Deprecate filters that have been replaced by placement filters  https://review.opendev.org/74560521:19
*** mriedem has left #openstack-nova21:28
*** xek has quit IRC21:49
openstackgerritMerged openstack/nova master: Follow up for I67504a37b0fe2ae5da3cba2f3122d9d0e18b9481  https://review.opendev.org/75018421:56
*** rcernin has joined #openstack-nova22:02
openstackgerritsean mooney proposed openstack/nova master: Deprecate filters that have been replaced by placement filters  https://review.opendev.org/74560522:02
*** rcernin has quit IRC22:03
*** rcernin has joined #openstack-nova22:03
*** bbowen has quit IRC22:09
*** mlavalle has quit IRC22:36
*** gyee has quit IRC22:46
*** hamalq has joined #openstack-nova23:13
*** hamalq_ has quit IRC23:14
*** k_mouza has quit IRC23:15
*** tosky has quit IRC23:20
*** k_mouza has joined #openstack-nova23:23
*** k_mouza has quit IRC23:28
*** k_mouza has joined #openstack-nova23:34
*** hamalq has quit IRC23:37
*** k_mouza has quit IRC23:38
*** k_mouza has joined #openstack-nova23:43
*** k_mouza has quit IRC23:46
openstackgerritMerged openstack/nova master: trivial: Remove useless comment  https://review.opendev.org/74847523:59
openstackgerritMerged openstack/nova master: xenapi: Remove driver and tests  https://review.opendev.org/74930423:59

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!