Thursday, 2019-12-05

*** brinzhang_ has joined #openstack-nova00:09
*** tosky has quit IRC00:10
openstackgerritMerged openstack/nova master: vmware: Remove vestigial nova-network support  https://review.opendev.org/69650300:11
*** brinzhang has quit IRC00:12
*** brinzhang_ has quit IRC00:14
*** tbachman has joined #openstack-nova00:14
openstackgerritsean mooney proposed openstack/nova master: Block rebuild when NUMA topology changed  https://review.opendev.org/68795700:21
openstackgerritsean mooney proposed openstack/nova master: Disable NUMATopologyFilter on rebuild  https://review.opendev.org/68986100:21
*** tbachman has quit IRC00:25
openstackgerritMerged openstack/nova master: zvm: Remove vestigial nova-network support  https://review.opendev.org/69650400:30
*** brinzhang has joined #openstack-nova00:31
*** ociuhandu has joined #openstack-nova00:39
*** brinzhang_ has joined #openstack-nova00:45
*** mdbooth has quit IRC00:48
*** ociuhandu has quit IRC00:48
*** brinzhang has quit IRC00:49
*** brinzhang has joined #openstack-nova00:50
*** brinzhang_ has quit IRC00:53
*** mdbooth has joined #openstack-nova00:59
*** tbachman has joined #openstack-nova01:05
*** Liang__ has joined #openstack-nova01:16
*** brinzhang_ has joined #openstack-nova01:21
*** brinzhang_ has quit IRC01:23
openstackgerritMerged openstack/nova master: Cache security group driver  https://review.opendev.org/69712201:24
*** brinzhang has quit IRC01:25
*** larainema has joined #openstack-nova01:27
openstackgerritjichenjc proposed openstack/nova master: libvirt: avoid cpu check at s390x arch  https://review.opendev.org/69622801:36
*** ociuhandu has joined #openstack-nova01:39
*** ociuhandu has quit IRC01:43
openstackgerritmelanie witt proposed openstack/nova stable/queens: delete consumers which no longer have allocations  https://review.opendev.org/69739801:51
*** chenhaw has joined #openstack-nova01:51
*** nicolasbock has quit IRC01:55
*** brinzhang has joined #openstack-nova01:57
*** brinzhang has quit IRC01:59
*** slaweq has joined #openstack-nova02:30
*** slaweq has quit IRC02:35
*** ociuhandu has joined #openstack-nova02:38
*** ociuhandu has quit IRC02:47
*** lbragstad has quit IRC02:48
*** mdbooth has quit IRC02:48
*** mdbooth has joined #openstack-nova02:49
*** slaweq has joined #openstack-nova02:51
*** igordc has quit IRC02:54
*** slaweq has quit IRC02:56
*** gyee has quit IRC03:03
*** brault has quit IRC03:06
*** tinwood has quit IRC03:10
*** tinwood has joined #openstack-nova03:12
*** abaindur has quit IRC03:27
*** boxiang has joined #openstack-nova03:28
*** zhanglong has joined #openstack-nova03:33
*** slaweq has joined #openstack-nova03:34
*** slaweq has quit IRC03:39
*** tkajinam is now known as tkajinam|lunch03:40
*** zhanglong has quit IRC03:44
*** udesale has joined #openstack-nova03:48
*** zhanglong has joined #openstack-nova03:49
*** slaweq has joined #openstack-nova03:58
*** lbragstad has joined #openstack-nova03:59
*** slaweq has quit IRC04:03
*** tetsuro_ has quit IRC04:06
*** tetsuro has joined #openstack-nova04:06
eanderssonmnaser what version of RabbitMQ are you running now?04:19
*** artom has quit IRC04:32
*** artom has joined #openstack-nova04:33
*** bhagyashris has joined #openstack-nova04:33
*** slaweq has joined #openstack-nova04:33
*** slaweq has quit IRC04:38
*** boxiang_ has joined #openstack-nova05:05
*** bhagyashris has quit IRC05:05
*** boxiang has quit IRC05:08
*** boxiang_ has quit IRC05:16
*** boxiang_ has joined #openstack-nova05:16
*** slaweq has joined #openstack-nova05:18
*** slaweq has quit IRC05:22
*** links has joined #openstack-nova05:23
*** threestrands has joined #openstack-nova05:24
*** slaweq has joined #openstack-nova05:25
*** bhagyashris has joined #openstack-nova05:26
*** slaweq has quit IRC05:30
*** ociuhandu has joined #openstack-nova05:30
*** ociuhandu has quit IRC05:35
*** pcaruana has joined #openstack-nova05:38
*** pcaruana has quit IRC05:42
*** slaweq has joined #openstack-nova05:45
*** slaweq has quit IRC05:54
*** slaweq has joined #openstack-nova05:58
*** slaweq has quit IRC06:03
*** Luzi has joined #openstack-nova06:03
*** rcernin has quit IRC06:09
*** slaweq has joined #openstack-nova06:12
*** udesale has quit IRC06:13
*** udesale has joined #openstack-nova06:14
*** ociuhandu has joined #openstack-nova06:16
*** slaweq has quit IRC06:18
*** ociuhandu has quit IRC06:36
*** awalende has joined #openstack-nova06:39
*** awalende has quit IRC06:44
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.opendev.org/69471706:47
*** ccamacho has quit IRC06:54
*** ralonsoh has joined #openstack-nova06:58
*** udesale has quit IRC07:02
*** ociuhandu has joined #openstack-nova07:03
*** udesale has joined #openstack-nova07:03
*** udesale has quit IRC07:06
*** udesale has joined #openstack-nova07:06
eanderssonI think I figured out the main issue we were having with instances getting stuck in BUILD. Many of our RabbitMQ Clusters got stuck into a bad state and because of how our scheduling works instances kept getting scheduled to the same computes. These computes had bad queues, but because it's almost impossible to know which compute(s) are affected.07:07
eanderssonBecause the database is not yet updated to reflect the compute hosting the instance (or at least it's not easy, or obvious to find the compute without digging deep into the database)07:08
eanderssonI am still baffled on why so many RabbitMQ Clusters broke within a short period of time. Seems to be caused by some sort of memory leak on one of the RabbitMQ servers.07:09
eanderssonBut we also saw it happen on one cluster after a network partition.07:10
eanderssonThe only thing we could really find on RabbitMQ was messages stuck un'acked (even without consumers), but unsure if that was just a display issue.07:12
*** maciejjozefczyk has joined #openstack-nova07:16
*** pcaruana has joined #openstack-nova07:17
*** avolkov has joined #openstack-nova07:19
*** ociuhandu has quit IRC07:27
*** ociuhandu has joined #openstack-nova07:27
*** damien_r has joined #openstack-nova07:54
*** zhanglong has quit IRC07:56
*** rcernin has joined #openstack-nova08:01
openstackgerritLuyao Zhong proposed openstack/nova-specs master: support live migration with virtual persistent memory  https://review.opendev.org/69586308:11
*** awalende has joined #openstack-nova08:20
openstackgerritLuyao Zhong proposed openstack/nova-specs master: support live migration with virtual persistent memory  https://review.opendev.org/69586308:20
*** tesseract has joined #openstack-nova08:21
*** yikun has quit IRC08:21
*** udesale has quit IRC08:31
*** udesale has joined #openstack-nova08:32
*** rpittau|afk is now known as rpittau08:32
*** kevinbenton has quit IRC08:34
*** NostawRm has quit IRC08:34
*** hemna_ has quit IRC08:34
*** abaindur has joined #openstack-nova08:35
*** tosky has joined #openstack-nova08:36
*** kevinbenton has joined #openstack-nova08:38
*** NostawRm has joined #openstack-nova08:38
*** hemna_ has joined #openstack-nova08:38
*** slaweq_ has joined #openstack-nova08:50
*** udesale has quit IRC08:58
*** slaweq_ has quit IRC08:59
*** lpetrut has joined #openstack-nova09:04
*** martinkennelly has joined #openstack-nova09:05
*** threestrands has quit IRC09:07
*** ociuhandu has quit IRC09:09
*** jangutter has joined #openstack-nova09:09
*** tkajinam|lunch has quit IRC09:09
*** udesale has joined #openstack-nova09:12
*** boxiang has joined #openstack-nova09:21
*** boxiang_ has quit IRC09:23
*** ccamacho has joined #openstack-nova09:42
*** derekh has joined #openstack-nova09:43
*** kevinbenton has quit IRC09:54
*** NostawRm has quit IRC09:54
*** hemna_ has quit IRC09:54
*** ccamacho has quit IRC09:54
*** boxiang has quit IRC09:54
*** lpetrut has quit IRC09:54
*** awalende has quit IRC09:54
*** avolkov has quit IRC09:54
*** tinwood has quit IRC09:54
*** amodi has quit IRC09:54
*** dpawlik has quit IRC09:54
*** sorrison has quit IRC09:54
*** jmlowe has quit IRC09:54
*** hamzy has quit IRC09:54
*** liuyulong has quit IRC09:54
*** gshippey has quit IRC09:54
*** jbernard has quit IRC09:54
*** jangutter has quit IRC09:54
*** yaawang has quit IRC09:54
*** Roamer` has quit IRC09:54
*** mugsie has quit IRC09:54
*** donnyd has quit IRC09:54
*** jdillaman has quit IRC09:54
*** rmk has quit IRC09:54
*** owalsh has quit IRC09:54
*** Li_Liu has quit IRC09:54
*** TheJulia has quit IRC09:54
*** mnasiadka has quit IRC09:54
*** rpittau has quit IRC09:54
*** gregwork has quit IRC09:54
*** toanster has quit IRC09:54
*** bcm has quit IRC09:54
*** frickler has quit IRC09:54
*** masayukig has quit IRC09:54
*** _erlon_ has quit IRC09:54
*** pas-ha has quit IRC09:54
*** eharney has quit IRC09:54
*** aloga has quit IRC09:54
*** trident has quit IRC09:54
*** gibi has quit IRC09:54
*** rnoriega_ has quit IRC09:54
*** _gryf_ has quit IRC09:54
*** d34dh0r53 has quit IRC09:54
*** openstackgerrit has quit IRC09:54
*** mtreinish has quit IRC09:54
*** antonym has quit IRC09:54
*** amorin has quit IRC09:54
*** brtknr has quit IRC09:54
*** evrardjp has quit IRC09:54
*** martinkennelly has quit IRC09:54
*** bhagyashris has quit IRC09:54
*** mdbooth has quit IRC09:54
*** larainema has quit IRC09:54
*** Liang__ has quit IRC09:54
*** tbachman has quit IRC09:54
*** df_sbr has quit IRC09:54
*** dasp has quit IRC09:54
*** andreykurilin has quit IRC09:54
*** mgoddard has quit IRC09:54
*** fnordahl has quit IRC09:54
*** jkulik has quit IRC09:54
*** StevenK has quit IRC09:54
*** cmurphy has quit IRC09:54
*** logan- has quit IRC09:54
*** adrianc has quit IRC09:54
*** dtantsur|afk has quit IRC09:54
*** mloza has quit IRC09:54
*** alex_xu has quit IRC09:54
*** markmcclain has quit IRC09:54
*** mmethot has quit IRC09:54
*** smcginnis has quit IRC09:54
*** samc-bbc has quit IRC09:54
*** haleyb has quit IRC09:54
*** yankcrime has quit IRC09:54
*** nicholas has quit IRC09:54
*** amotoki has quit IRC09:54
*** Anticimex has quit IRC09:54
*** coreycb has quit IRC09:54
*** knikolla has quit IRC09:54
*** ildikov has quit IRC09:54
*** ttx has quit IRC09:54
*** kashyap has quit IRC09:54
*** spotz has quit IRC09:54
*** tosky has quit IRC09:54
*** abaindur has quit IRC09:54
*** tesseract has quit IRC09:54
*** pcaruana has quit IRC09:54
*** maciejjozefczyk has quit IRC09:54
*** ralonsoh has quit IRC09:54
*** Luzi has quit IRC09:54
*** links has quit IRC09:54
*** tetsuro has quit IRC09:54
*** lbragstad has quit IRC09:54
*** mgariepy has quit IRC09:54
*** lennyb has quit IRC09:54
*** jamesdenton has quit IRC09:54
*** jlvillal has quit IRC09:54
*** johanssone has quit IRC09:54
*** factor has quit IRC09:54
*** Jeffrey4l has quit IRC09:54
*** elod has quit IRC09:54
*** tbarron has quit IRC09:54
*** hoonetorg has quit IRC09:55
*** irclogbot_3 has quit IRC09:55
*** bnemec has quit IRC09:55
*** benj_ has quit IRC09:55
*** klindgren_ has quit IRC09:55
*** corvus has quit IRC09:55
*** ianw has quit IRC09:55
*** sapd1_ has quit IRC09:55
*** efried has quit IRC09:55
*** dosaboy has quit IRC09:55
*** canori01 has quit IRC09:55
*** ceryx has quit IRC09:55
*** eandersson has quit IRC09:55
*** stephenfin has quit IRC09:55
*** aspiers has quit IRC09:55
*** zer0c00l has quit IRC09:55
*** szaher has quit IRC09:55
*** obre has quit IRC09:55
*** larsks has quit IRC09:55
*** Vek has quit IRC09:55
*** rcernin has quit IRC09:55
*** damien_r has quit IRC09:55
*** rouk has quit IRC09:55
*** jroll has quit IRC09:55
*** francoisp has quit IRC09:55
*** Alon_KS has quit IRC09:55
*** gouthamr has quit IRC09:55
*** bauzas has quit IRC09:55
*** pots has quit IRC09:55
*** fungi has quit IRC09:55
*** rajinir has quit IRC09:55
*** mmedvede has quit IRC09:55
*** gmann has quit IRC09:55
*** cz2 has quit IRC09:55
*** dustinc has quit IRC09:55
*** DinaBelova has quit IRC09:55
*** derekh has quit IRC09:55
*** udesale has quit IRC09:55
*** artom has quit IRC09:55
*** chenhaw has quit IRC09:55
*** jistr has quit IRC09:55
*** gryf has quit IRC09:55
*** panda has quit IRC09:55
*** mvkr has quit IRC09:55
*** adriant has quit IRC09:55
*** sean-k-mooney has quit IRC09:55
*** tonyb has quit IRC09:55
*** kaisers has quit IRC09:55
*** kukacz_ has quit IRC09:55
*** purplerbot has quit IRC09:55
*** lyarwood has quit IRC09:55
*** aarents has quit IRC09:55
*** lchabert has quit IRC09:55
*** ioni has quit IRC09:55
*** cgoncalves has quit IRC09:55
*** dansmith has quit IRC09:55
*** tobberydberg has quit IRC09:55
*** gary_perkins has quit IRC09:55
*** ericyoung has quit IRC09:55
*** tristanC has quit IRC09:55
*** arxcruz has quit IRC09:55
*** jhesketh has quit IRC09:55
*** ebbex has quit IRC09:55
*** noonedeadpunk has quit IRC09:55
*** dtruong has quit IRC09:55
*** andreaf has quit IRC09:55
*** diconico07 has quit IRC09:55
*** csatari has quit IRC09:55
*** johnsom has quit IRC09:55
*** portdirect has quit IRC09:55
*** jungleboyj has quit IRC09:55
*** zbr_ has quit IRC09:55
*** dklyle has quit IRC09:55
*** vesper11 has quit IRC09:55
*** huaqiang has quit IRC09:55
*** ircuser-1 has quit IRC09:55
*** jrosser has quit IRC09:55
*** ganso has quit IRC09:55
*** luyao has quit IRC09:55
*** mgagne has quit IRC09:55
*** lifeless has quit IRC09:55
*** zzzeek has quit IRC09:55
*** guilhermesp has quit IRC09:55
*** tonyb[m] has quit IRC09:55
*** mordred has quit IRC09:55
*** ab-a has quit IRC09:55
*** arne_wiebalck has quit IRC09:55
*** vdrok has quit IRC09:55
*** mnaser has quit IRC09:55
*** NobodyCam has quit IRC09:55
*** rm_work has quit IRC09:55
*** melwitt has quit IRC09:55
*** Hazelesque has quit IRC09:55
*** awestin1 has quit IRC09:55
*** ChanServ has quit IRC09:55
*** hemna_ has joined #openstack-nova10:04
*** NostawRm has joined #openstack-nova10:04
*** kevinbenton has joined #openstack-nova10:04
*** jbernard has joined #openstack-nova10:04
*** gshippey has joined #openstack-nova10:04
*** liuyulong has joined #openstack-nova10:04
*** hamzy has joined #openstack-nova10:04
*** jmlowe has joined #openstack-nova10:04
*** sorrison has joined #openstack-nova10:04
*** dpawlik has joined #openstack-nova10:04
*** amodi has joined #openstack-nova10:04
*** tinwood has joined #openstack-nova10:04
*** avolkov has joined #openstack-nova10:04
*** awalende has joined #openstack-nova10:04
*** lpetrut has joined #openstack-nova10:04
*** boxiang has joined #openstack-nova10:04
*** ccamacho has joined #openstack-nova10:04
*** pas-ha has joined #openstack-nova10:04
*** _erlon_ has joined #openstack-nova10:04
*** masayukig has joined #openstack-nova10:04
*** frickler has joined #openstack-nova10:04
*** bcm has joined #openstack-nova10:04
*** toanster has joined #openstack-nova10:04
*** gregwork has joined #openstack-nova10:04
*** rpittau has joined #openstack-nova10:04
*** mnasiadka has joined #openstack-nova10:04
*** Li_Liu has joined #openstack-nova10:04
*** TheJulia has joined #openstack-nova10:04
*** owalsh has joined #openstack-nova10:04
*** rmk has joined #openstack-nova10:04
*** jdillaman has joined #openstack-nova10:04
*** donnyd has joined #openstack-nova10:04
*** mugsie has joined #openstack-nova10:04
*** Roamer` has joined #openstack-nova10:04
*** yaawang has joined #openstack-nova10:04
*** jangutter has joined #openstack-nova10:04
*** evrardjp has joined #openstack-nova10:04
*** brtknr has joined #openstack-nova10:04
*** amorin has joined #openstack-nova10:04
*** antonym has joined #openstack-nova10:04
*** mtreinish has joined #openstack-nova10:04
*** openstackgerrit has joined #openstack-nova10:04
*** d34dh0r53 has joined #openstack-nova10:04
*** _gryf_ has joined #openstack-nova10:04
*** rnoriega_ has joined #openstack-nova10:04
*** gibi has joined #openstack-nova10:04
*** trident has joined #openstack-nova10:04
*** aloga has joined #openstack-nova10:04
*** eharney has joined #openstack-nova10:04
*** derekh has joined #openstack-nova10:04
*** tosky has joined #openstack-nova10:04
*** tesseract has joined #openstack-nova10:04
*** rcernin has joined #openstack-nova10:04
*** damien_r has joined #openstack-nova10:04
*** pcaruana has joined #openstack-nova10:04
*** maciejjozefczyk has joined #openstack-nova10:04
*** ralonsoh has joined #openstack-nova10:04
*** Luzi has joined #openstack-nova10:04
*** links has joined #openstack-nova10:04
*** artom has joined #openstack-nova10:04
*** tetsuro has joined #openstack-nova10:04
*** lbragstad has joined #openstack-nova10:04
*** chenhaw has joined #openstack-nova10:04
*** mgariepy has joined #openstack-nova10:04
*** lennyb has joined #openstack-nova10:04
*** spotz has joined #openstack-nova10:04
*** kashyap has joined #openstack-nova10:04
*** zbr_ has joined #openstack-nova10:04
*** dklyle has joined #openstack-nova10:04
*** francoisp has joined #openstack-nova10:04
*** vesper11 has joined #openstack-nova10:04
*** jamesdenton has joined #openstack-nova10:04
*** jhesketh has joined #openstack-nova10:04
*** jlvillal has joined #openstack-nova10:04
*** huaqiang has joined #openstack-nova10:04
*** johanssone has joined #openstack-nova10:04
*** ircuser-1 has joined #openstack-nova10:04
*** jistr has joined #openstack-nova10:04
*** factor has joined #openstack-nova10:04
*** gryf has joined #openstack-nova10:04
*** Jeffrey4l has joined #openstack-nova10:04
*** jrosser has joined #openstack-nova10:04
*** rouk has joined #openstack-nova10:04
*** jroll has joined #openstack-nova10:04
*** ganso has joined #openstack-nova10:04
*** tbarron has joined #openstack-nova10:04
*** elod has joined #openstack-nova10:04
*** panda has joined #openstack-nova10:04
*** mvkr has joined #openstack-nova10:04
*** luyao has joined #openstack-nova10:04
*** hoonetorg has joined #openstack-nova10:04
*** Alon_KS has joined #openstack-nova10:04
*** gouthamr has joined #openstack-nova10:04
*** bauzas has joined #openstack-nova10:04
*** adriant has joined #openstack-nova10:04
*** sean-k-mooney has joined #openstack-nova10:04
*** tonyb has joined #openstack-nova10:04
*** kaisers has joined #openstack-nova10:04
*** irclogbot_3 has joined #openstack-nova10:04
*** bnemec has joined #openstack-nova10:04
*** benj_ has joined #openstack-nova10:04
*** klindgren_ has joined #openstack-nova10:04
*** pots has joined #openstack-nova10:04
*** corvus has joined #openstack-nova10:04
*** mgagne has joined #openstack-nova10:04
*** kukacz_ has joined #openstack-nova10:04
*** ianw has joined #openstack-nova10:04
*** tonyb[m] has joined #openstack-nova10:04
*** sapd1_ has joined #openstack-nova10:04
*** lifeless has joined #openstack-nova10:04
*** efried has joined #openstack-nova10:04
*** zzzeek has joined #openstack-nova10:04
*** guilhermesp has joined #openstack-nova10:04
*** purplerbot has joined #openstack-nova10:04
*** dosaboy has joined #openstack-nova10:04
*** canori01 has joined #openstack-nova10:04
*** eandersson has joined #openstack-nova10:04
*** ceryx has joined #openstack-nova10:04
*** lyarwood has joined #openstack-nova10:04
*** ioni has joined #openstack-nova10:04
*** fungi has joined #openstack-nova10:04
*** aarents has joined #openstack-nova10:04
*** rajinir has joined #openstack-nova10:04
*** mordred has joined #openstack-nova10:04
*** stephenfin has joined #openstack-nova10:04
*** lchabert has joined #openstack-nova10:04
*** ab-a has joined #openstack-nova10:04
*** aspiers has joined #openstack-nova10:04
*** cgoncalves has joined #openstack-nova10:04
*** dtruong has joined #openstack-nova10:04
*** zer0c00l has joined #openstack-nova10:04
*** szaher has joined #openstack-nova10:04
*** andreaf has joined #openstack-nova10:04
*** obre has joined #openstack-nova10:04
*** dansmith has joined #openstack-nova10:04
*** larsks has joined #openstack-nova10:04
*** Vek has joined #openstack-nova10:04
*** tobberydberg has joined #openstack-nova10:04
*** diconico07 has joined #openstack-nova10:04
*** noonedeadpunk has joined #openstack-nova10:04
*** gary_perkins has joined #openstack-nova10:04
*** mmedvede has joined #openstack-nova10:04
*** ericyoung has joined #openstack-nova10:04
*** tristanC has joined #openstack-nova10:04
*** arxcruz has joined #openstack-nova10:04
*** cz2 has joined #openstack-nova10:04
*** gmann has joined #openstack-nova10:04
*** dustinc has joined #openstack-nova10:04
*** DinaBelova has joined #openstack-nova10:04
*** csatari has joined #openstack-nova10:04
*** johnsom has joined #openstack-nova10:04
*** portdirect has joined #openstack-nova10:04
*** ebbex has joined #openstack-nova10:04
*** jungleboyj has joined #openstack-nova10:04
*** awestin1 has joined #openstack-nova10:04
*** Hazelesque has joined #openstack-nova10:04
*** melwitt has joined #openstack-nova10:04
*** rm_work has joined #openstack-nova10:04
*** mnaser has joined #openstack-nova10:04
*** NobodyCam has joined #openstack-nova10:04
*** vdrok has joined #openstack-nova10:04
*** arne_wiebalck has joined #openstack-nova10:04
*** ChanServ has joined #openstack-nova10:04
*** orwell.freenode.net sets mode: +o ChanServ10:04
*** ttx has joined #openstack-nova10:04
*** martinkennelly has joined #openstack-nova10:04
*** bhagyashris has joined #openstack-nova10:04
*** mdbooth has joined #openstack-nova10:04
*** larainema has joined #openstack-nova10:04
*** Liang__ has joined #openstack-nova10:04
*** tbachman has joined #openstack-nova10:04
*** df_sbr has joined #openstack-nova10:04
*** dasp has joined #openstack-nova10:04
*** andreykurilin has joined #openstack-nova10:04
*** mgoddard has joined #openstack-nova10:04
*** jkulik has joined #openstack-nova10:04
*** fnordahl has joined #openstack-nova10:04
*** cmurphy has joined #openstack-nova10:04
*** StevenK has joined #openstack-nova10:04
*** logan- has joined #openstack-nova10:04
*** adrianc has joined #openstack-nova10:04
*** dtantsur|afk has joined #openstack-nova10:04
*** alex_xu has joined #openstack-nova10:04
*** mloza has joined #openstack-nova10:04
*** markmcclain has joined #openstack-nova10:04
*** mmethot has joined #openstack-nova10:04
*** smcginnis has joined #openstack-nova10:04
*** samc-bbc has joined #openstack-nova10:04
*** haleyb has joined #openstack-nova10:04
*** yankcrime has joined #openstack-nova10:04
*** nicholas has joined #openstack-nova10:04
*** amotoki has joined #openstack-nova10:04
*** Anticimex has joined #openstack-nova10:04
*** coreycb has joined #openstack-nova10:04
*** knikolla has joined #openstack-nova10:04
*** ildikov has joined #openstack-nova10:04
*** logan- has quit IRC10:05
*** dtantsur|afk is now known as dtantsur10:06
*** logan- has joined #openstack-nova10:06
stephenfinbauzas: Can you send this on its way? https://review.opendev.org/#/c/665518/10:08
bauzasstephenfin: sure10:09
stephenfinThanks10:10
bauzasstephenfin: any chance to add a new tox target that would wrap pre-commit ?10:14
stephenfinI could, but it doesn't really make sense. They're really different tools10:14
stephenfinpre-commit "installs" things by cloning git repos and creating its own venv. It wouldn't benefit from tox10:15
zbr_stephenfin: it makes a lot of sense, haptics being one of them10:16
stephenfinhaptics?10:16
zbr_yeah, or habbits10:16
zbr_most devs are used to call "tox -e linters", as most projects in opestack have them.10:16
bauzasstephenfin: i have to admit I'm new to pre-commit, but I'll give it a try10:16
stephenfinzbr_: to be clear, this doesn't replace that - you can still call that10:17
bauzasstephenfin: that said, I'm maybe a bit torn about saying we *could* deprecate fast810:17
zbr_mainly you change how we orchestrate linters without changing the user interface10:17
stephenfinall that pre-commit adds is a way to call it automatically10:17
zbr_tox -> linters -> pre-commit -> ....10:17
stephenfinno, it's more like10:17
zbr_mainly the only dependency listed in linters environment of tox would be "pre-commit"10:17
stephenfintox -> flake8 (hacking)10:17
stephenfinpre-commit -> flake8 (hacking)10:17
stephenfinit's two different ways to achieve the same end goal10:18
bauzasstephenfin: anyway, sent to the gate10:18
stephenfinbauzas: ta10:18
zbr_so you adopt the tool without changing how user is calling it, and more important without asking them to install a new tool.10:18
*** Liang__ has quit IRC10:18
* bauzas needs to drop for a while10:18
zbr_just having python and tox is usually enough10:18
stephenfinI have flake8 integrated into my editor because it makes my life easier10:19
stephenfinbut I don't force people to do the same10:19
stephenfinditto for pre-commit10:19
stephenfinso long as flake8 is passing, I don't care how people check things10:19
stephenfinpre-commit is just another way to do that check10:19
stephenfinon top of 'tox -e pep8', 'tox -e flake8', random editor integration, etc.10:20
zbr_stephenfin: yeah, another way. who said you would loose ability to use flake8 in your editor10:20
stephenfinno one did10:21
zbr_huh.... :D10:21
stephenfinI don't get the point?10:21
zbr_I was explaining why I find useful to call pre-commit from tox, also makes very easy to integrate with zuul, without gaving to add any extra configs.10:22
zbr_in fact tox itself is doing it, https://github.com/tox-dev/tox/blob/master/tox.ini#L68-L7010:24
stephenfincould be one to do in the future so, once it's bedded in10:24
stephenfinfor now though, I've had a hard enough time getting _something_ in that I'll take the win I have and be happy with it :)10:25
zbr_stephenfin: i empathise with your pain... i know exactly what you mean.10:26
*** salmankhan has joined #openstack-nova10:27
*** salmankhan has quit IRC10:28
*** salmankhan has joined #openstack-nova10:29
openstackgerritEric Xie proposed openstack/nova master: Host CPU compatibility check does not make sense on AArch64; skip CPU comparison  https://review.opendev.org/58976910:29
*** chenhaw has quit IRC10:33
*** lbragstad has quit IRC10:36
*** zhanglong has joined #openstack-nova10:37
*** ociuhandu has joined #openstack-nova10:39
*** pcaruana has quit IRC10:42
openstackgerritEric Xie proposed openstack/nova master: Skip cpu comparison on AArch64  https://review.opendev.org/58976910:48
sean-k-mooneyzbr_: for what its worth i still dont know of any project that i have worked on that has tox -e linters11:07
*** jdillaman has quit IRC11:08
sean-k-mooneyzbr_: i may have but if i did i was not aware of it. its not part of the pti https://github.com/openstack/governance/tree/master/reference/pti so its not require for project to implement and you are the first person who has raised it as a thing11:09
sean-k-mooneymost project i was aware of if they ran addtional linters added them to the pep8 tox enve11:10
zbr_sean-k-mooney: http://codesearch.openstack.org/?q=testenv%3Alinters&i=nope&files=tox.ini&repos= reports 201 repos, and is one of the few officially supported environments: https://docs.openstack.org/infra/openstack-zuul-jobs/jobs.html#job-openstack-tox-linters11:11
sean-k-mooneyi belive neutron still adds extra linters to there pep8 tox env11:11
zbr_mainly is seen as a replacement for the older pep811:11
zbr_sean-k-mooney: that's a perfect alternative, mainly is the same, just a different name to avoid confusions.11:11
zbr_i used pep8 too in several places, for convenience.11:12
zbr_even if is wasn't really pep8, it was flake8 + 10 others inside.11:12
sean-k-mooney looking at that is mainly openstack ansible and windmil + some test projects11:13
zbr_on at least one project I made an "alias" to point pep8 -> linters, just to keep it working for the user. i hate to force people to change habits, especially good ones.11:13
sean-k-mooneynon of the core project or service project actully use linters which is why im not familar with it11:14
zigoeandersson: What package do you need to be updated in Debian?11:14
zbr_sean-k-mooney: if you have pep8, keep using it and just document its more broad goal.11:14
sean-k-mooneyzbr_: sure im not really against tox -e linters11:14
zbr_sean-k-mooney: in fact I was considering altering the zuul jobs to introspect tox file and to run whatever they find linters or pep8. If I do this it would be much easier to projects to migrate to the more generic one. Now is almost impossible to remove openstack-tox-pep8 job as is deep inside defaults.11:17
sean-k-mooneyits just 200 project repos which is about 50% openstack stack ansible does not mean its a common practice in openstack when we have thoasands of repos in opendev orgs11:17
zbr_i already talked with infra about this about but i did not had time to go into action yet.11:17
zbr_true11:17
sean-k-mooneyzbr_: to do that you would need to modify the project testing interface11:18
sean-k-mooneywhich require a tc vote11:18
zbr_that is why i was considering the smart linter idea, because it could allow people to adopt newer method without breaking existing users.11:18
sean-k-mooneyzbr_: no11:19
sean-k-mooneythat would be still a violation of the testing interface11:19
sean-k-mooneyif you want to change it then you shoudl propose a governace motion via a patch11:19
zbr_i know, but before working on such a change i wanted to get some feedback.11:20
sean-k-mooneymin would be we should keep pep8 and have an optional seperate linters target11:20
sean-k-mooney*mine11:20
zbr_we already have this, probably is more than an year old11:21
sean-k-mooneyexitsting project that run addtionall linters in pep8 shoudl move them11:21
sean-k-mooneyto the linter target11:21
*** artom has quit IRC11:21
*** brinzhang has joined #openstack-nova11:30
*** ociuhandu has quit IRC11:31
*** derekh has quit IRC11:33
*** tbachman has quit IRC11:36
*** ociuhandu has joined #openstack-nova11:44
*** dviroel has joined #openstack-nova11:46
*** zhanglong has quit IRC11:49
*** shilpasd has joined #openstack-nova11:51
*** udesale has joined #openstack-nova11:51
openstackgerritStephen Finucane proposed openstack/nova master: functional: Unify '_wait_until_deleted' implementations  https://review.opendev.org/68918111:53
openstackgerritStephen Finucane proposed openstack/nova master: functional: Unify '_build_minimal_create_server_request' implementations  https://review.opendev.org/69502411:53
openstackgerritStephen Finucane proposed openstack/nova master: functional: Make '_IntegratedTestBase' subclass 'InstanceHelperMixin'  https://review.opendev.org/68918211:53
*** jraju__ has joined #openstack-nova11:53
*** links has quit IRC11:53
openstackgerritStephen Finucane proposed openstack/nova master: functional: Remove 'get_invalid_image'  https://review.opendev.org/69745411:58
openstackgerritStephen Finucane proposed openstack/nova master: functional: Unify '_wait_until_deleted' implementations  https://review.opendev.org/68918112:00
openstackgerritStephen Finucane proposed openstack/nova master: functional: Unify '_build_minimal_create_server_request' implementations  https://review.opendev.org/69502412:00
openstackgerritStephen Finucane proposed openstack/nova master: functional: Make '_IntegratedTestBase' subclass 'InstanceHelperMixin'  https://review.opendev.org/68918212:00
openstackgerritStephen Finucane proposed openstack/nova master: functional: Remove 'get_invalid_image'  https://review.opendev.org/69745412:00
*** artom has joined #openstack-nova12:08
*** zbr_ is now known as zbr|out12:13
*** shilpasd has quit IRC12:20
*** yan0s has joined #openstack-nova12:25
*** larainema has quit IRC12:26
openstackgerritMerged openstack/nova master: Integrate 'pre-commit'  https://review.opendev.org/66551812:26
*** brinzhang has quit IRC12:33
*** rcernin has quit IRC12:52
*** pcaruana has joined #openstack-nova12:54
*** ociuhandu has quit IRC13:06
*** eharney has quit IRC13:06
*** tbachman has joined #openstack-nova13:22
*** belmoreira has joined #openstack-nova13:25
*** bhagyashris has quit IRC13:27
*** ociuhandu has joined #openstack-nova13:27
*** udesale has quit IRC13:30
*** abhishekk has joined #openstack-nova13:35
*** udesale has joined #openstack-nova13:39
*** mriedem has joined #openstack-nova13:54
*** derekh has joined #openstack-nova13:56
openstackgerritMatt Riedemann proposed openstack/nova stable/train: Cache security group driver  https://review.opendev.org/69747514:02
*** shilpasd has joined #openstack-nova14:07
shilpasdmriedem: requesting to review 'https://review.opendev.org/#/c/612626/14:09
*** eharney has joined #openstack-nova14:09
openstackgerritsean mooney proposed openstack/nova master: support pci numa affinity policies in flavor and image  https://review.opendev.org/67407214:09
shilpasdbauzas: requesting you to review 'https://review.opendev.org/#/c/650188/8 Allow compute nodes to use DISK_GB from shared storage RP' as per your priorities14:10
bauzasshilpasd: ack14:10
shilpasdbauzas: thank you14:10
shilpasd@nova core: please review 'https://review.opendev.org/#/c/650188/8 Allow compute nodes to use DISK_GB from shared storage RP'14:11
*** cgoncalves has quit IRC14:13
*** mkrai has joined #openstack-nova14:16
*** ociuhandu has quit IRC14:16
stephenfinIs anyone else seeing this message when running 'tox -e py36 <whatever>' http://paste.openstack.org/show/787167/ ?14:17
mnasereandersson: im just about ready to give up on rabbit clusters14:19
*** belmoreira has quit IRC14:21
mriedemshilpasd: comments on https://review.opendev.org/#/c/694462/14:24
*** belmoreira has joined #openstack-nova14:24
shilpasdmriedem: thank you for review, will work on them14:25
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: Cache security group driver  https://review.opendev.org/69748014:25
mriedemstephenfin: trying it14:26
*** Luzi has quit IRC14:28
mriedemefried: this is why that multiple possible networks found thing is intermittent https://bugs.launchpad.net/nova/+bug/183104814:29
openstackLaunchpad bug 1844568 in tempest "duplicate for #1831048 [compute] "create_test_server" if networks is undefined and more than one network is present" [Medium,In progress] - Assigned to Rodolfo Alonso (rodolfo-alonso-hernandez)14:29
mriedemneutron tests create a new network at the same time we're creating a server and kaboom14:29
mriedemthough we shouldn't be running tempest.api.network tests in nova-next, but it could fail during scenario runs or something i guess?14:30
mriedemstephenfin: i didn't get that problem14:30
mriedemi'm on ubuntu bionic14:30
stephenfinokay, it's a Fedora issue14:30
stephenfinmriedem++ thanks14:30
*** zbr|out has quit IRC14:33
*** zbr has joined #openstack-nova14:34
*** pcaruana has quit IRC14:38
mriedemi wonder if the nova-next job hits that more often because of how we do concurrency differently in that job from others?14:40
*** ociuhandu has joined #openstack-nova14:40
mriedemi think tempest-full runs api tests concurrently and scenario tests serially, and i think nova-next runs them all concurretly14:40
*** zbr_ has joined #openstack-nova14:41
mriedemyeah so i bet a network scenario test creates a network while the compute api tests are running and it blows up14:43
sean-k-mooneymriedem: for the senario test do we not run those with concurancy 114:44
*** zbr has quit IRC14:44
*** nweinber has joined #openstack-nova14:44
sean-k-mooneybut yes i can see how it could fial in if a neutron test created a network in paralle with a compute test although ideally it would do that in a different tenant so that it could not affect other tests14:45
mriedemthere is probably a scenario test that creates a public network14:46
mriedemand yes we run all tests, api and scenario, concurrently in nova-next14:46
sean-k-mooneyah ok so we dont do the two pahse execution we do in tempst full wehre it runs all test excpet the senario tests first with concurrency and then runs the senario tests serially14:47
sean-k-mooneyya that would proably make it more likely to fail in nova-next14:47
*** zbr_ has quit IRC14:49
mriedemall things i just said above14:49
mriedemefried: gibi: so i think it was actually enabling tempest.scenario.test_minbw_allocation_placement.MinBwAllocationPlacementTest.test_qos_min_bw_allocation_basic in nova-next again that is making this worse14:50
mriedemthat test creates a public network14:51
mriedemand everything that failed in this job was right around the time that test was running https://zuul.opendev.org/t/openstack/build/b7872ac3886e4cc7a060c1376a2a3395/log/job-output.txt#7912414:51
mriedemwhich is also our spike in failures http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Details%3A%20%7B'code'%3A%20409%2C%20'message'%3A%20'Multiple%20possible%20networks%20found%2C%20use%20a%20Network%20ID%20to%20be%20more%20specific.'%7D%5C%22%20AND%20tags%3A%5C%22console%5C%22%20AND%20voting%3A1&from=7d14:51
*** lbragstad has joined #openstack-nova14:52
*** zbr has joined #openstack-nova14:56
*** ociuhandu has quit IRC14:57
*** cgoncalves has joined #openstack-nova14:57
*** ociuhandu has joined #openstack-nova14:57
openstackgerritMatt Riedemann proposed openstack/nova master: Skip test_minbw_allocation_placement in nova-next job  https://review.opendev.org/69749114:57
mriedemgibi: efried: stephenfin: dansmith: if you wanna help the gate and love patriotism ^14:57
stephenfinI hate patriotism, but I do have the feels for the gate14:58
stephenfinyuuup, +214:59
stephenfinlyarwood: What was the reason for the request for configurables here? https://review.opendev.org/#/c/669674/3/nova/compute/manager.py@261315:01
stephenfinI want to -1 that patch _because_ of the seemingly extraneous knob but I could missing something15:01
*** rpittau is now known as rpittau|afk15:03
lyarwoodstephenfin: looking15:07
*** jraju__ has quit IRC15:08
gibimriedem: you are right. at the same time of the skip the test we need to keep an eye on the real fix https://review.opendev.org/#/c/68296415:10
mriedemlyarwood: stephenfin: i left several comments in there too,15:11
mriedemblindly retrying on *any* exception when doing something like deleting an attachment makes me nervous because if cinder returns a 404 because the original request completed and then we keep retrying, we're just going to keep retrying and getting 404s until we exhaust our retries15:12
stephenfingood point15:12
lyarwoodstephenfin: no specific reason other than a personal dislike of hardcoded retry values tbh15:12
mriedemiow, can we scope those retries to just 500 errors?15:12
stephenfinand I have a personal dislike of knobs that no one ever touches15:12
stephenfinUnstoppable force, meet immovable object :)15:13
lyarwoodMeets the guy with +2 over the guy with +1 so you win15:13
lyarwoodmriedem: yeah that's fair, I think a previous PS did previously and I've likely missed that cinder_exception.ClientException covers both?15:13
lyarwoodyeah it's the base, gah my bad.15:14
mriedemright so https://github.com/openstack/python-cinderclient/blob/5.0.0/cinderclient/exceptions.py#L242 for 404 will create a NotFound(ClientException) and we'd retry on that15:15
mriedemso we need to scope the retries to ClientExceptions with code==500 only imo15:15
*** dpawlik has quit IRC15:16
mriedemwould also be nice to eventually add long_rpc_timeout to cinder for apis like this that have been known to take awhile15:16
mriedemi know we've seen attachment delete timeout in the gate15:16
mriedemsurely red hat has some people working on cinder yeah?15:16
lyarwoodyup we do, I can follow up with them about that15:17
mriedemhttps://bugs.launchpad.net/cinder/+bug/1763712 is where i suggested using long_rpc_timeout in cinder15:17
openstackLaunchpad bug 1763712 in Cinder "Unable to update the attachment.: MessagingTimeout" [Medium,Confirmed]15:17
mriedemit shouldn't be that hard to add since the cinder rpc stuff is likely very similar to nova's15:17
*** ociuhandu has quit IRC15:21
*** ociuhandu has joined #openstack-nova15:22
mriedemgibi: on https://review.opendev.org/#/c/693248/ can't we just hard-code the server created to use the net0 network - that's the one that has the port with the qos allocations15:32
mriedemoh well i guess shared != external15:34
mriedemso we got the shared public external network and nova puked because by default non-admins can't create servers on external networks15:35
*** udesale has quit IRC15:37
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Cache security group driver  https://review.opendev.org/69750515:39
*** abhishekk has quit IRC15:41
*** mkrai has quit IRC15:42
*** mkrai_ has joined #openstack-nova15:42
openstackgerritMatt Riedemann proposed openstack/nova master: Regression test for bug 1849657  https://review.opendev.org/69324815:44
openstackbug 1849657 in OpenStack Compute (nova) train " allocation key is missing from the binding:profile of the neutron qos port when the server is created by a non-admin user" [Medium,Fix committed] https://launchpad.net/bugs/1849657 - Assigned to Balazs Gibizer (balazs-gibizer)15:44
*** belmoreira has quit IRC15:47
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: log when loading security group driver  https://review.opendev.org/65278315:49
*** belmoreira has joined #openstack-nova15:51
efriedmriedem: ack, thanks for that.15:53
efriedmy cold has decided to kill me, so I may be sparse today.15:54
mriedemRIP15:54
mriedemefried: fyi i've got a thing in the nova meeting agenda for today but likely not going to be able to attend, it's just a question about if a spec is needed for that filter i'm working on15:55
efriedack15:56
mriedemalso, the price is right is on in 4 minutes15:56
mriedemgood time to get on the couch with a wubby15:56
*** mlavalle has joined #openstack-nova15:57
efriedstephenfin: https://review.opendev.org/#/c/696507/ if you want to update that commit message quick I'll fast approve, no need to rebase anything.16:00
efrieddo we still say things like $show is on at $time? Who watches anything on a broadcast schedule?16:01
stephenfinokidok16:02
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove SG tests that don't apply to neutron  https://review.opendev.org/69650716:03
stephenfinefried: ^16:03
efriedstephenfin: +A, thanks for accommodating my pickiness.16:04
stephenfinthanks for the review (y)16:04
mordredefried: I think we maybe contracted the same plague16:04
mordredmriedem: do we need an sdk release for the ironic patch we landed for you?16:05
efriedI suspect I actually got two things back to back. Which isn't supposed to happen.16:05
*** mkrai_ has quit IRC16:05
mriedemmordred: at some point, but the nova blueprint that needs it isn't approved yet so it's not a rush16:07
mriedemefried: i wouldn't dvr the price is right because i only watch it like twice a year16:07
*** jdillaman has joined #openstack-nova16:07
mriedemabout the same amount of times i buy bacon - usually in the same week16:08
*** dave-mccowan has joined #openstack-nova16:08
mriedemah the holidays, a time for bacon and day time tv game shows16:08
*** yan0s has quit IRC16:10
efriedwow, I buy bacon weekly, in multiple pounds.16:10
mordredmy biggest issue with bacon is that it's never possible to buy enough16:12
*** bbowen has joined #openstack-nova16:12
mriedemno one else in my house will eat it (now that mortimer the dog is dead anyway) so i'm not making that smelly mess for myself every day16:13
efriedI make the whole mess at once and bag it. Bacon is a food that loses almost nothing reheated.16:14
*** maciejjozefczyk has quit IRC16:14
mriedemreheated in the microwave?16:14
mriedemblasphemy16:14
*** ociuhandu has quit IRC16:23
*** jangutter has quit IRC16:25
*** igordc has joined #openstack-nova16:27
*** shilpasd has quit IRC16:30
dansmithnot blasphemy, plus it means bacon more often16:31
dansmithBACON AT EVERY MEAL (tm)16:31
efriedOkay, I updated the meeting agenda. I'm going to go be miserable somewhere for a while.16:32
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: Improve metadata server performance with large security groups  https://review.opendev.org/69592516:34
*** tesseract has quit IRC16:39
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Improve metadata server performance with large security groups  https://review.opendev.org/69751716:40
*** gyee has joined #openstack-nova16:41
melwittmriedem: I went out on a limb and proposed a backport to queens that landed in rocky, for cleaning up placement consumer records that no longer have allocations https://review.opendev.org/697398 I had to modify it a lot because the code was in different files in queens16:41
*** slaweq_ has joined #openstack-nova16:41
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Improve metadata server performance with large security groups  https://review.opendev.org/69751816:42
*** openstackgerrit has quit IRC16:42
melwittnot sure whether that's going to fly but I figured I'd try because it's a pretty big unbounded table grow that otherwise has to be cleaned up manually16:43
*** nicolasbock has joined #openstack-nova16:44
mriedemqueens is EM so that's pretty low priority to address but fine to post if you're doing it for downstream anyway and if others want to use it16:45
melwittah, I forgot it's EM. ok16:46
melwittunderstood16:46
*** damien_r has quit IRC16:49
mriedemdansmith: how do you feel about backporting this? https://review.opendev.org/#/c/696953/16:52
mriedemseems ok since it's tied to a long-standing bug but wanted a 2nd opinion16:52
dansmithmriedem: yeah, I mean... as long as the old version has a sufficient oslo.messaging and the config knob, I can't really think of any reason not to do tose16:53
dansmith*those16:53
*** openstackgerrit has joined #openstack-nova16:53
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Improve metadata server performance with large security groups  https://review.opendev.org/69752316:53
mriedemlyarwood: i'd be +2 on this https://review.opendev.org/#/c/696953/ but i'm not sure how pedantic you want to be about using the correct source commit hash16:56
mriedemgiven train and stein had conflicts which you've already resolved i'm assuming you don't want to redo all of that16:57
*** dave-mccowan has quit IRC16:58
mriedemnvm i just +2ed16:59
*** dave-mccowan has joined #openstack-nova17:00
mriedemdansmith: you want to get this https://review.opendev.org/#/c/696339/ and this then https://review.opendev.org/69695317:01
*** slaweq_ has quit IRC17:02
*** ociuhandu has joined #openstack-nova17:02
dansmithyar17:03
lyarwoodmriedem: sorry was on a call, I really don't care about the source commit hash with backports given the Change IDs tbh. That said I have no idea how I ended up using the wrong one for this.17:06
lyarwoodmriedem: and thanks17:06
*** dave-mccowan has quit IRC17:10
*** salmankhan has quit IRC17:13
*** dtantsur is now known as dtantsur|afk17:19
*** lennyb has quit IRC17:20
*** lennyb has joined #openstack-nova17:26
*** lennyb has quit IRC17:32
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: rt: only map compute node if we created it  https://review.opendev.org/67646317:33
*** lennyb has joined #openstack-nova17:36
*** lpetrut has quit IRC17:40
*** lennyb has quit IRC17:41
*** ccamacho has quit IRC17:46
*** lennyb has joined #openstack-nova17:46
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Add functional regression test for bug 1849409  https://review.opendev.org/69074617:54
openstackbug 1849409 in OpenStack Compute (nova) queens "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem)17:54
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Fix listing deleted servers with a marker  https://review.opendev.org/69074717:54
openstackgerritStephen Finucane proposed openstack/nova master: functional: Make '_IntegratedTestBase' subclass 'InstanceHelperMixin'  https://review.opendev.org/68918217:57
openstackgerritStephen Finucane proposed openstack/nova master: functional: Remove 'get_invalid_image'  https://review.opendev.org/69745417:57
openstackgerritStephen Finucane proposed openstack/nova master: functional: Add unified '_build_server' helper function  https://review.opendev.org/69753717:57
openstackgerritStephen Finucane proposed openstack/nova master: functional: Add unified '_(build|create)_flavor' helper functions  https://review.opendev.org/69753817:57
openstackgerritStephen Finucane proposed openstack/nova master: functional: Add '_delete_server' to 'InstanceHelperMixin'  https://review.opendev.org/69753917:57
openstackgerritStephen Finucane proposed openstack/nova master: functional: Add '_create_server' helper  https://review.opendev.org/69754017:57
*** derekh has quit IRC18:00
stephenfinsean-k-mooney: One comment on https://review.opendev.org/#/c/674072/18:05
*** lennyb has quit IRC18:06
*** lpetrut has joined #openstack-nova18:07
stephenfinsean-k-mooney: also on https://review.opendev.org/#/c/687957/18:16
stephenfinand with that...18:16
* stephenfin -> 🏠18:16
*** ociuhandu has quit IRC18:17
*** awalende has quit IRC18:20
*** awalende has joined #openstack-nova18:21
*** awalende has quit IRC18:25
*** jdillaman has quit IRC18:34
*** artom has quit IRC18:35
*** artom has joined #openstack-nova18:35
sean-k-mooneystephenfin: yep ill take a look and adress them today. just back form dinner so ill work on that shortly18:35
*** ralonsoh has quit IRC18:37
*** dave-mccowan has joined #openstack-nova18:38
*** jdillaman has joined #openstack-nova18:48
*** martinkennelly has quit IRC18:50
openstackgerritMatt Riedemann proposed openstack/nova master: Add resource provider allocation unset example to troubleshooting doc  https://review.opendev.org/69658218:50
*** lennyb has joined #openstack-nova18:53
*** ociuhandu has joined #openstack-nova19:07
efriedmriedem, smcginnis, jungleboyj: Any ideas on how to make sure we've tested https://review.opendev.org/697134 through os-brick?19:17
*** jmlowe has quit IRC19:19
mriedemefried: i commented but tl;dr i can't think of anything19:19
mriedemthis is assuming that all of os-brick is using privsep and not any rootwrap helpers for some of the backends in os-brick somewhere19:20
mriedemwe can see that it's fine for lvm and rbd though19:20
mriedemthe libvirt driver uses os-brick, the ceph job uses the rbd volume type and backend, and all other jobs are using lvm19:20
mriedemif there is something else in brick not using privsep then we likely won't see it here, but once we land this i'd think 3rd party CIs in cinder would be failing and alerting to cinder team19:21
mriedemthough if cinder's 3rd party ci's are anything like ours, there are probably many that fail often so people don't pay attention19:21
mriedemthe quobyte ci might be a decent indication and it failed but it also didn't report anything19:22
mriedemkaisers: ^19:22
efriedthanks for the look mriedem19:24
mriedemlooking at the hyperv logs, and i noticed this the other day as well, they seem to hit https://review.opendev.org/#/q/topic:bug/1748697+(status:open+OR+status:merged) quite a bit19:25
mriedemdoing cold migration testing and the scheduler picks the current host which triggers a reschedule19:26
efriedttx: If you concur, my philosophy is that we should merge this now to give some soak time for 3pCIs to uncover issues. We can always revert if we need to.19:26
mriedem2019-12-05 11:43:22.672 2556 2225548250976 MainThread ERROR nova.compute.manager [req-3e316bf9-f63f-4388-8da2-108d7781d6f8 14ffcc57bb934290aa4ec7ba0ff79a98 6fec721e534d443f9aa4123acb1d9d20 - default default] [instance: ebcd3e84-9ee7-455d-be7a-849338e9781e] Error: Instance rollback performed due to: Unable to migrate instance (ebcd3e84-9ee7-455d-be7a-849338e9781e) to current host (n-h1-697134-2).: nova.exception.InstanceFaultR19:26
mriedemack: Instance rollback performed due to: Unable to migrate instance (ebcd3e84-9ee7-455d-be7a-849338e9781e) to current host (n-h1-697134-2).19:26
jungleboyjefried:  I concur with mriedem 's analysis.19:28
efriedthanks jungleboyj19:28
jungleboyjUnfortunately the CIs aren't super reliable so what kind of flags it will send up I don't know.19:28
jungleboyjIt should be covered by the check for RBD and LVM though.19:29
jungleboyjSo, that is a good start.19:29
mriedemhemna_ should be able to just tell us if there is anything in os-brick that isn't using privsep19:29
jungleboyjI feel like we got things moved over.19:30
*** ociuhandu has quit IRC19:31
hemna_I don't think any of the connectors were using rootwrap, but I'd have to look closely.19:31
*** mvkr has quit IRC19:31
*** dave-mccowan has quit IRC19:36
jungleboyjhemna_:  Have I mentioned it is good to have you back?19:37
efriedmriedem: I don't know if we're talking about different things that could (n)ever work by filtering GET /a_c with (required or forbidden) COMPUTE_SAME_HOST_COLD_MIGRATE. I tried to explain myself again.19:39
efriedWe don't have "conditional traits". We can't say "exclude if rp_uuid=$self and !COMPUTE_SAME_HOST_COLD_MIGRATE". Without that conditional, saying required=!COMPUTE_SAME_HOST_COLD_MIGRATE will never, ever make sense. Ever.19:40
mriedemi replied, think i got what you're saying19:43
hemna_:)  thanks man19:43
sean-k-mooneyefried: well required=!COMPUTE_SAME_HOST_COLD_MIGRATE litrally would mean a host that cant support same host cold migration but ya we can tmake that require ment condionaly only if rp_uuid is a specific value and ignore it otherwsie for the same allocation candiate19:46
efriedright19:46
hemna_from what I can tell, most stuff works through the Executor, which defaults to privsep, or the privileged/rootwrap.execute, which also defaults to privsep19:46
efriedmriedem: before I dive back into the nova side, is it using the trait?19:47
efried(for the post-filter, that is)19:47
mriedemhttps://review.opendev.org/#/c/676022/ is a backportable workaround that is not, but if we don't care about backportability for the fix, then i will abandon. the patch that comes after that https://review.opendev.org/#/c/695220/ uses the trait yes and has a functional test to show it working.19:48
mriedemwell, it's using a custom trait for now since the os-traits patch isn't released19:48
mriedemsame idea19:48
efriedokay, I didn't see a Depends-On at the top so that's what I was asking. Will dig in.19:49
mriedemi left a draft proposed comment for the os-traits change if it will work for you19:49
efriedI'm responding. But maybe easier to discuss here...19:50
efriedThat's a reasonable walkthrough of a specific scenario, but my reasoning was much simpler than that.19:50
mriedemjust tell me what to write19:50
efriedlet me talk this out first19:51
efriedIf I'm on a host and doing a cold migrate, the condition I want to express is "if SHCM is not supported by $me, exclude $me from the results".19:51
mriedemcorrect19:52
efriedbut we don't have any placement way to say that19:52
mriedemnot unless there is some !in_tree logic in GET /a_c i'm not aware of19:52
sean-k-mooneyright we would have to do too quires. one to determin if we support X and then a second with either this host included or excluded19:52
efriedExactly, point being that the GET /a_c query wouldn't involve the SHCM trait; it would involve the host (RP (tree)) we want to exclude.19:53
efried...assuming !in_tree was a thing19:54
* efried looks19:54
*** damien_r has joined #openstack-nova19:54
sean-k-mooneyi think that was aded but the in paramter of traits was not19:54
efriedwe were going to do something like that for CERNs server groups case, right?19:54
mriedemi don't remember19:55
sean-k-mooneyfor anti affinity i think19:55
mriedemthere is no !in_tree semantics documented in the GET /a_c reference19:55
efriednor in the gabbits, so yeah, I don't think that's a thing.19:55
sean-k-mooneywe may have drop the propsoal since we did not need it in train19:56
efriedregardless, the point is that we don't expect that trait to ever make sense in a GET /a_c query because we have no way to express the SAME_HOST part of it in GET /a_c19:57
mriedemas for anti-affinity....there is a member_of=!in, so for server groups you could create a provider aggregate for any host a member of that group is on and then when scheduling others you'd say !in for any other provider aggregate that represents other hosts in the group that has members, maybe - or not, it's all confusing and a tangent to what i'm trying to solve19:57
efriedit's, like, a tautological impossibility.19:57
mriedemwell, saying "no never because we can't ever" and "because GET /a_c doesn't have some semantics today" is a conflict, but sure19:58
mriedemi agree today it's not possible19:58
mriedemanyway, that's kind of why i didn't really want to put a bunch of words in a comment in os-traits about the state of how things are done between nova and placement today19:59
mriedemit's super duper tight coupling19:59
efriedit's tight coupling between os-traits and placement, not anything and nova.20:00
mriedemefried: if you're asking for a comment about why the trait can't be used in GET /a_c then that seems like something that should go in the nova change that's using the trait20:00
mriedembecause that is the code that would eventually be changed, not where the trait lives in os-traits20:00
mriedemi mean, if you're asking about a reminder to yourself if someone tries to do this some day20:00
efriedthat would also be good.20:01
eanderssonThis RabbitMQ issue is so weird.20:02
eanderssonThe compute queues only are getting messages, but they are staying as un'ackd.20:02
sean-k-mooneyeandersson: have you restarted the nova compute agent20:02
*** rcernin has joined #openstack-nova20:02
eanderssonWe suspect RabbitMQ still, but it's weird that it is only affectin compute.20:03
eanderssonYea sean-k-mooney they are kept as unac'kd even after restarting the agent. So for sure a RabbitMQ issue, but don't know what triggers it.20:03
efriedmriedem: responded x220:03
mriedemefried: so to recap, what i'd like to do is (1) remove the mention from the os-traits commit message about eventually doing GET /a_c or whatever, (2) *not* leave a coupled comment next to the trait, and instead (3) put a comment about that in the nova patch https://review.opendev.org/#/c/695220/20:03
efriedmriedem: (2) do leave a comment, (3) do put a comment.20:03
efriedThey're different comments20:04
sean-k-mooneyeandersson: oh ok so its not the issue where the compute agent does not recreate the topic queue if the rabbmq server goes offline20:04
*** dosaboy has quit IRC20:04
sean-k-mooneyeandersson: im not sure what that issue is then20:05
*** igordc has quit IRC20:05
efried(2) I suggested in the os-traits patch, just talks about GET /a_c. (3) should have your specific libvirt scenario described20:05
openstackgerritMatt Riedemann proposed openstack/os-traits master: Add COMPUTE_SAME_HOST_COLD_MIGRATE trait  https://review.opendev.org/66660420:07
mriedemdone20:08
mriedemartom: dansmith: shall i just abandon the backportable workaround option that is https://review.opendev.org/#/c/676022/ and focus on the trait-based master-only solution to the extremely latent bug?20:09
mriedemif it's this latent people obviously don't care enough so backports are a meh to me at this point20:09
efriedmriedem: +2, thanks for bearing with me.20:09
mriedemsame20:10
*** jmlowe has joined #openstack-nova20:12
dansmithmriedem: sounds like that would sidestep the whole argument, so.. yes?20:12
efriedIt's got a bug heat of 32 fwiw.20:13
efriedMultiple people commented on it over the almost-2y it's been open.20:13
efriedso having a backportable solution doesn't seem like a terrible idea to me.20:14
efriedbut I need to relook at how ugly it is.20:14
mriedemi've abandoned20:14
mriedempeople can cherry pick it downstream if they really need it20:14
efriedokay20:14
*** ceryx is now known as ccstone20:15
efriedmriedem: I guess you're going to rebase and clean up the other one now, so I should wait to review?20:15
efriedyeah20:16
mriedemi'm not going to get to that today but it won't be much different20:16
mriedemprobably crank it out in the wee morn20:16
efriedI didn't open any of the files, but the commit message at least needs an overhaul.20:16
mriedemubetcha20:17
efriedokay, let's see if I can think clearly enough to get back to rg/rp mappings...20:17
*** rcernin has quit IRC20:18
efriedmriedem: if you're feeling magnanimous, you could hit the bottom three patches here, fairly simple: https://review.opendev.org/#/c/696418/20:18
mriedemoh god request groups20:19
mriedemi think melwitt said she wanted to learn more about this part of the code20:19
efriedhahaha20:19
efriedIn my IRC client, melwitt shows up a nice grasshopper green. Coincidence?20:20
mriedemas in the kung fu grasshopper pupil green?20:21
*** awalende has joined #openstack-nova20:21
mriedemor the "oh god request groups, i think i'm going to be sick" green20:21
*** awalende has quit IRC20:25
mriedemgibi: a few nits in https://review.opendev.org/#/c/696245/ if you want to start a FUP patch for that series20:32
openstackgerritEric Fried proposed openstack/nova master: WIP: Use provider mappings from Placement (mostly)  https://review.opendev.org/69699220:34
*** damien_r has quit IRC20:34
*** bbowen has quit IRC20:42
efriedmeeting now21:00
*** jdillaman has quit IRC21:01
*** eharney has quit IRC21:06
*** dosaboy has joined #openstack-nova21:09
efriedmriedem: The vote in the meeting was unanimous: do a spec for the ironic owner filter21:14
efried(unanimous: both of us)21:14
sean-k-mooneyim not sure i normally get a vote so it was also unilatreral21:14
sean-k-mooneybut close enough21:15
*** jdillaman has joined #openstack-nova21:15
efriedWe don't really vote officially on things like that anyway, so it was all opinion21:16
efriedand you know what they say about opinions.21:16
sean-k-mooney:) as an fyi i will be going on vacation in about a week so im going to rebase the two feature im trying to get merged into a chain since they will conflict if i dont21:17
sean-k-mooneyfor what its worth of the two option i kind of prefer the tradtional filter for filtering nodes by owner21:20
sean-k-mooneyif we did it a placemnt way i woudl thing an aggreate feel more natual for grouping RP the a custom trait21:21
*** jaosorior has joined #openstack-nova21:22
sean-k-mooneyanyway that is just my gut reaction so i have not really tought about it much21:22
efriedsean-k-mooney: we've already established that a trait won't work without hackitude.21:27
sean-k-mooneyack ya noticed the comment form mriedem after i wrote that21:27
*** ociuhandu has joined #openstack-nova21:41
*** stephenfin has quit IRC21:50
efrieddansmith: TypeError: Object of type 'datetime' is not JSON serializable21:52
*** dasp_ has joined #openstack-nova21:52
efriedThis on a cast of prep_resize21:52
efriedThe payload appears to have datetime.datetime objects in the image meta21:53
efriedwhat's odd is, this wasn't failing before I changed something in a totally different place21:53
*** jaosorior has quit IRC21:53
dansmithefried: nothing obvious, other than that oslo.serialization has some quiet handling of that21:54
*** dasp has quit IRC21:55
dansmithand there was also some proposed change recently specifically about that, but I think we determined it wasn't going to be a problem21:55
*** stephenfin has joined #openstack-nova21:57
efriedthat's pretty bizarre21:57
dansmithwhat is?21:57
efriedIs it possible this stack trace is a red herring and the problem is really somewhere else?21:58
dansmithisn't that pretty much always a true statement? :)21:58
melwittmriedem, efried: lol, you rascals (just read the mentions about request groups)22:08
efriedyou got ToRed22:09
efriedas my father would say22:09
efried(Transfer of Responsibility)22:09
*** nweinber has quit IRC22:09
efrieddansmith: I think my venv might have just been out of date. Repro'd same result against master. Rebuilding...22:14
efriedyuh, that cleared it up.22:16
sean-k-mooneystale pyc files22:17
sean-k-mooneymaybe22:17
sean-k-mooneystill an odd error22:18
efriedno, don't we get bad magic number when that happens? In this case I'm betting it was a downrev of oslo.serialization or similar.22:18
efrieddon't care enough to track it down exactly.22:18
*** bbowen has joined #openstack-nova22:19
*** ociuhandu has quit IRC22:20
*** rcernin has joined #openstack-nova22:28
openstackgerritEric Fried proposed openstack/nova master: Use provider mappings from Placement (mostly)  https://review.opendev.org/69699222:43
efriedmriedem, dansmith, sean-k-mooney, gibi, stephenfin: I think this is done ^22:44
efriedIt uses placement 1.34 to get rid of the hack where we were calculating mappings based on allocations, in all code paths except revert_resize.22:44
efriedand knocks down another two microversions in the parity gap.22:45
sean-k-mooneyoh thi si sremoving the code we had on the nova side to compute that and using the info returned form placement correct22:53
efriedyeah, unfortunately it doesn't remove the code yet, cause it's still needed by revert_resize. Haven't figure out how we're going to fix that yet. Maybe I leave that to gibi :P22:54
sean-k-mooneywill we need to keep it for upgrade too?22:54
sean-k-mooneyfor n-122:54
efriedI wouldn't have thought so. The data is identical (for existing use cases), just the method of computing that changed.22:55
efriedWe no longer support downrev placements, you gotta upgrade placement first.22:55
sean-k-mooneysure but did we ever need to look this up on the compute node or was it only used in the schduler22:56
efriedum22:56
efriedyeah, I think we wanted to be able to use it from the compute22:56
efriedbut hadn't yet.22:56
sean-k-mooneyok if we havent yet its not an upgrade issue :)22:57
sean-k-mooneywe will have the placment native way before we do22:57
efriedLike, to correlate VGPUs with the proper mdevs, and eventually to make sure we pin CPUs from the right NUMA nodes.22:57
sean-k-mooneyyep that is what i was thinking of22:57
sean-k-mooneywe will need to use the mapping when doing the physical assignment on the compute node22:58
sean-k-mooneyto make sure we assign the resocue that corralates with the allocation22:58
efriedactually, I guess bw gets around that today because the mapping is stuffed into the binding profile, so (I'm guessing) the compute pulls it from there.22:58
efriedto... figure out which NIC to assign?22:58
efriedstart talking network stuff, I hear woh woh woh22:58
sean-k-mooneyin the sriov case maybe22:58
sean-k-mooneyin the ovs case it does not matter22:58
sean-k-mooneyactully in the sriov cas ethe port request has the pf netdev name and we add that to the pci_request object23:00
sean-k-mooneyso again stuffing stuff in the vif object to avoid the problem23:00
efriedI think ima rebase on the "skip the bw test" patch, and put one on top unskipping to prove it works.23:01
efriedso I don't have to recheck a zillion times on that stupid multi-network bug.23:01
*** tkajinam has joined #openstack-nova23:07
openstackgerritEric Fried proposed openstack/nova master: Use Placement 1.34 (string suffixes & mappings)  https://review.opendev.org/69641823:08
openstackgerritEric Fried proposed openstack/nova master: refactor: RequestGroup.is_empty() and .strip_zeros()  https://review.opendev.org/69699123:08
openstackgerritEric Fried proposed openstack/nova master: Tie requester_id to RequestGroup suffix  https://review.opendev.org/69694623:08
openstackgerritEric Fried proposed openstack/nova master: Use provider mappings from Placement (mostly)  https://review.opendev.org/69699223:08
openstackgerritEric Fried proposed openstack/nova master: Restore test_minbw_allocation_placement in nova-next job  https://review.opendev.org/69758623:08
efriedmriedem, gibi:  FYI, re-restoring that minbw test via https://review.opendev.org/#/c/697586/23:13
efriedIt's at the top of that series for now so I can prove the top patch works, can move it later if necessary.23:14
efriedand with that, I'm outta here o/23:15
openstackgerritsean mooney proposed openstack/nova master: Block rebuild when NUMA topology changed  https://review.opendev.org/68795723:21
openstackgerritsean mooney proposed openstack/nova master: Disable NUMATopologyFilter on rebuild  https://review.opendev.org/68986123:21
openstackgerritsean mooney proposed openstack/nova master: support pci numa affinity policies in flavor and image  https://review.opendev.org/67407223:22
sean-k-mooneyalso rebased that on top of the same skip patch to avoid the rechecks23:23
*** avolkov has quit IRC23:26
*** nicolasbock has quit IRC23:29
*** ociuhandu has joined #openstack-nova23:30
*** dviroel has quit IRC23:35
*** ociuhandu has quit IRC23:35
*** tosky has quit IRC23:38
*** abaindur has joined #openstack-nova23:40
*** abaindur has quit IRC23:44
*** abaindur has joined #openstack-nova23:45
*** mdbooth has quit IRC23:49
*** mdbooth has joined #openstack-nova23:50
*** mvkr has joined #openstack-nova23:55

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