Monday, 2018-10-22

openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9)  https://review.openstack.org/57558100:07
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10)  https://review.openstack.org/57601700:07
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11)  https://review.openstack.org/57601800:07
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12)  https://review.openstack.org/57601900:08
*** dave-mccowan has quit IRC00:14
*** zhanglong has joined #openstack-nova00:17
*** dave-mccowan has joined #openstack-nova00:32
*** dave-mccowan has quit IRC01:10
*** medberry has quit IRC01:14
*** tetsuro has joined #openstack-nova01:18
*** mrsoul has joined #openstack-nova01:19
*** imacdonn has quit IRC01:23
*** imacdonn has joined #openstack-nova01:23
*** dave-mccowan has joined #openstack-nova01:24
*** mhen has quit IRC01:44
*** TuanDA has joined #openstack-nova01:44
*** mhen has joined #openstack-nova01:46
*** tiendc has joined #openstack-nova01:47
*** lei-zh has joined #openstack-nova01:58
*** annp has joined #openstack-nova01:58
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13)  https://review.openstack.org/57602002:03
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14)  https://review.openstack.org/57602702:04
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15)  https://review.openstack.org/57603102:04
openstackgerritTakashi NATSUME proposed openstack/nova master: Add API ref guideline for body text  https://review.openstack.org/60562802:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py  https://review.openstack.org/56615302:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Add API ref guideline for examples  https://review.openstack.org/60406002:06
*** hongbin has joined #openstack-nova02:19
*** brinzhang has joined #openstack-nova02:22
*** hongbin has quit IRC02:35
*** hongbin has joined #openstack-nova02:35
*** rnoriega has quit IRC02:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7)  https://review.openstack.org/57199202:41
*** hongbin_ has joined #openstack-nova02:41
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8)  https://review.openstack.org/57199302:41
*** hongbin has quit IRC02:43
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16)  https://review.openstack.org/57629902:46
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17)  https://review.openstack.org/57634402:47
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18)  https://review.openstack.org/57667302:47
*** psachin has joined #openstack-nova02:54
*** dave-mccowan has quit IRC03:11
*** zhanglong has quit IRC03:30
*** takashin has quit IRC03:33
*** takashin has joined #openstack-nova03:51
*** udesale has joined #openstack-nova03:55
*** udesale has quit IRC03:55
*** udesale has joined #openstack-nova03:56
*** tetsuro has quit IRC03:57
*** kevinbenton has quit IRC04:00
*** kevinbenton has joined #openstack-nova04:00
openstackgerritAndreas Jaeger proposed openstack/nova master: Convert legacy-tempest-dsvm-neutron-src-oslo.versionedobjects job  https://review.openstack.org/61027104:06
openstackgerritAndreas Jaeger proposed openstack/nova master: Rename tempest-nova job to follow conventions  https://review.openstack.org/61223004:08
openstackgerritSam Morrison proposed openstack/nova master: Fix up compute rpcapi version for pike release  https://review.openstack.org/61223104:17
*** hongbin_ has quit IRC04:22
*** hongbin has joined #openstack-nova04:24
*** hongbin has quit IRC04:38
*** lei-zh has quit IRC04:38
*** hoonetorg has quit IRC04:48
*** bhagyashris has joined #openstack-nova04:50
*** hoonetorg has joined #openstack-nova05:01
*** lei-zh has joined #openstack-nova05:03
*** lei-zh has quit IRC05:07
*** lei-zh has joined #openstack-nova05:07
*** hoonetorg has quit IRC05:12
*** hoonetorg has joined #openstack-nova05:33
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix best_match() deprecation warning  https://review.openstack.org/61120405:37
*** bhagyashris has quit IRC05:38
*** ratailor has joined #openstack-nova05:50
*** Luzi has joined #openstack-nova05:52
*** pooja_jadhav has joined #openstack-nova06:02
*** phillu has joined #openstack-nova06:03
*** maciejjozefczyk has joined #openstack-nova06:09
*** dpawlik has joined #openstack-nova06:20
*** pcaruana has joined #openstack-nova06:21
*** hamdyk has joined #openstack-nova06:26
*** adrianc has joined #openstack-nova06:26
*** adrianc_ has joined #openstack-nova06:27
*** psachin has quit IRC06:29
*** janki has joined #openstack-nova06:30
*** adrianc has quit IRC06:31
*** adrianc has joined #openstack-nova06:31
openstackgerritYongli He proposed openstack/nova-specs master: add 'show-server-group' spec  https://review.openstack.org/61225506:37
openstackgerritYongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.openstack.org/61225606:37
*** bzhao__ has joined #openstack-nova06:42
*** sahid has joined #openstack-nova06:48
*** lei-zh has quit IRC06:49
*** lei-zh has joined #openstack-nova06:49
*** pcaruana has quit IRC06:58
*** jangutter has joined #openstack-nova06:59
*** rcernin has quit IRC06:59
*** ccamacho has joined #openstack-nova07:03
*** rcernin has joined #openstack-nova07:05
*** lei-zh has quit IRC07:06
*** lei-zh has joined #openstack-nova07:06
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19)  https://review.openstack.org/57667607:07
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20)  https://review.openstack.org/57668907:07
*** rcernin has quit IRC07:07
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21)  https://review.openstack.org/57670907:07
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22)  https://review.openstack.org/57671207:08
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Replace MB with MiB  https://review.openstack.org/60880707:09
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Replace MB with MiB  https://review.openstack.org/60880707:11
*** pcaruana has joined #openstack-nova07:13
*** moshele has joined #openstack-nova07:16
*** helenafm has joined #openstack-nova07:20
*** adrianc has quit IRC07:25
*** adrianc has joined #openstack-nova07:26
*** adrianc__ has joined #openstack-nova07:26
*** lei-zh has quit IRC07:28
*** alexchadin has joined #openstack-nova07:28
*** TuanDA has quit IRC07:29
*** adrianc__ has quit IRC07:29
*** TuanDA has joined #openstack-nova07:29
*** ralonsoh has joined #openstack-nova07:29
*** sahid has quit IRC07:30
*** sahid has joined #openstack-nova07:30
*** moshele has quit IRC07:30
*** pooja_jadhav has quit IRC07:32
*** jpena|off is now known as jpena07:32
*** TuanDA has quit IRC07:33
*** udesale has quit IRC07:33
*** TuanDA has joined #openstack-nova07:33
*** udesale has joined #openstack-nova07:33
*** pooja_jadhav has joined #openstack-nova07:35
jaosoriorHey folks! Could I get a review for this https://review.openstack.org/#/c/609591/ ? removing dead code :D07:42
*** lei-zh has joined #openstack-nova07:42
*** phillu has quit IRC07:44
*** lpetrut has joined #openstack-nova07:50
*** fanzhang has joined #openstack-nova07:50
openstackgerritFan Zhang proposed openstack/nova master: Retry after hitting libvirt error VIR_ERR_OPERATION_INVALID in live migration.  https://review.openstack.org/61227207:51
*** kashyap has quit IRC07:55
*** alexchadin has quit IRC07:58
*** rpittau has joined #openstack-nova08:02
*** phillu has joined #openstack-nova08:03
*** tetsuro has joined #openstack-nova08:11
*** tetsuro has quit IRC08:12
*** tetsuro_ has joined #openstack-nova08:12
*** k_mouza has joined #openstack-nova08:13
*** panda has joined #openstack-nova08:13
*** finucannot is now known as stephenfin08:20
*** ttsiouts has joined #openstack-nova08:25
*** ttsiouts has quit IRC08:29
*** adrianc_ has quit IRC08:39
*** adrianc has quit IRC08:39
*** ShilpaSD has joined #openstack-nova08:47
*** lpetrut has quit IRC09:06
*** slaweq has joined #openstack-nova09:08
*** moshele has joined #openstack-nova09:10
*** derekh has joined #openstack-nova09:11
*** ttsiouts has joined #openstack-nova09:17
*** alexchadin has joined #openstack-nova09:18
*** ttsiouts has quit IRC09:23
*** lei-zh has quit IRC09:27
*** alexchadin has quit IRC09:30
*** dtantsur|afk is now known as dtantsur09:30
*** sambetts|afk is now known as sambetts09:30
*** mvkr has quit IRC09:35
*** zzzeek_ has quit IRC09:37
*** k_mouza has quit IRC09:38
*** ociuhandu has joined #openstack-nova09:39
*** zzzeek has joined #openstack-nova09:40
*** ociuhandu has quit IRC09:40
openstackgerritRadoslav Gerganov proposed openstack/nova stable/rocky: Update resources once in update_available_resource  https://review.openstack.org/61229309:43
*** ttsiouts has joined #openstack-nova09:43
openstackgerritRadoslav Gerganov proposed openstack/nova stable/queens: Update resources once in update_available_resource  https://review.openstack.org/61229409:44
openstackgerritRadoslav Gerganov proposed openstack/nova stable/pike: Update resources once in update_available_resource  https://review.openstack.org/61229509:45
*** bnemec has joined #openstack-nova09:45
*** hoonetorg has quit IRC09:51
*** bnemec has quit IRC09:53
*** psachin has joined #openstack-nova09:56
*** mvkr has joined #openstack-nova09:59
*** k_mouza has joined #openstack-nova10:01
*** sean-k-mooney has joined #openstack-nova10:03
*** psachin has quit IRC10:06
*** TuanDA has quit IRC10:07
*** hoonetorg has joined #openstack-nova10:08
*** alexchadin has joined #openstack-nova10:11
*** FlorianFa has joined #openstack-nova10:12
*** dave-mccowan has joined #openstack-nova10:22
*** s10 has joined #openstack-nova10:25
*** cdent has joined #openstack-nova10:30
*** adrianc has joined #openstack-nova10:32
*** adrianc_ has joined #openstack-nova10:32
adriancsean-k-mooney: Hi, id like to collaborate on https://blueprints.launchpad.net/nova/+spec/libvirt-neutron-sriov-livemigration can we have a blue-jeans meeting to discuss ?10:37
sean-k-mooneyadrianc: sure we could that said it is best to keep as much of these discussion on irc/ml/gerrit when it comes to the design side10:38
*** Dinesh_Bhor has joined #openstack-nova10:40
openstackgerritAdam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances  https://review.openstack.org/60977910:41
sean-k-mooneyadrianc: have you reviewed the spec i have published yet?10:41
sean-k-mooneyah i see you have10:41
Dinesh_BhorHi All, Is there a recommended/official API rate limiter for Nova?10:41
adriancsean-k-mooney: agree(also commented on your spec), I have also a WIP POC, myself and moshele would like to have a short sync on the topic10:42
sean-k-mooneyadrianc: did you have a time in mind. i would like th discuss the spec/blueprint at the next nova team meeting to see if we can at least get the blueprint approved10:44
sean-k-mooneyregarding the poc how far have you gotten10:44
sean-k-mooneyi had started on the same but i have been pulled into too other activities also10:44
sean-k-mooneyi had hoped to work on adding pci address claiming this week/next10:45
adriancsean-k-mooney: for indirect SRIOV with macvtap its working apart of the fact that no multiple port binding implementation in this mech driver10:45
adriancsean-k-mooney: including the pci resource claim for the migrated instance10:46
sean-k-mooneyadrianc: well wiht what i have published where im not doing the claim macvtap works10:47
sean-k-mooneyhttps://review.openstack.org/#/c/607365/ crrects the multiple port binding issue10:48
*** moshele has quit IRC10:48
sean-k-mooneythen when i comment out this exception https://review.openstack.org/#/c/608788/1/nova/network/neutronv2/api.py10:48
sean-k-mooneythe migration succeeds. i was planning on trying to get the claim to work.10:48
sean-k-mooneylater this week10:48
adriancsean-k-mooney: was the neutron port in ACTIVE state ?10:49
adriancafter migration that is10:49
sean-k-mooneyi did not check but i could ssh to the vm.10:49
*** annp has quit IRC10:49
sean-k-mooneyi belive it should be but we may need to modify the ml2 driver on the neutron side.10:50
*** cdent has quit IRC10:50
sean-k-mooneyit has been almost 2 weeks since i worked on this since i was on training most of last week10:50
*** jamesdenton has joined #openstack-nova10:52
sean-k-mooneyadrianc: do you have any review pushed to gerrit10:52
adriancsean-k-mooney: not yet, its on local branch.10:54
*** moshele has joined #openstack-nova10:54
*** Dinesh_Bhor has quit IRC11:03
*** erlon has joined #openstack-nova11:04
sean-k-mooneyadrianc: did you have a time in mind to chat about this futher.11:04
*** Dinesh_Bhor has joined #openstack-nova11:05
adriancsean-k-mooney: im on PTO tomorrow but Wednesday would be great11:06
sean-k-mooneysure that should work i have one or two meetsing around 4pm IST(UTC+1)11:08
*** erlon has quit IRC11:11
adriancsean-k-mooney: great, can you set something at around 11AM (UTC+1)11:13
*** owalsh has quit IRC11:13
*** erlon has joined #openstack-nova11:14
sean-k-mooneythat works for me. im not sure if there are any US based folks that should join but assumeing not then yes11:14
adriancgreat, no US folks from my side.11:16
sean-k-mooneycool we may want to check in with artom who will not be online at that time but i can sync with him later11:17
artomI'm here, barely11:18
sean-k-mooneyartom: its 6am for you now right?11:19
artom7:2011:19
*** priteau has joined #openstack-nova11:19
aspiersouch11:19
sean-k-mooneyartom: why? why would you be awake and on irc at this time?11:20
*** k_mouza has quit IRC11:20
artomHeh, I woke up on my own, 6-7 hours of sleep is usually enough for me11:21
*** k_mouza has joined #openstack-nova11:21
artomAnd I wanted to get a downstream escalation out of the way, hence laptop and IRC11:21
*** Dinesh_Bhor has quit IRC11:21
sean-k-mooneyartom: i was just talking to adrianc about having a call about sriov livemigtion at 11:00 utc+1 on wednesday. is that something you want to join i was sugessting it might be a bit too early for you but not sure you needed to attend either11:22
*** markvoelker has joined #openstack-nova11:22
artomAlso, I need to literally go buy milk, otherwise 1. kids will complain that I'm a bad parent and 2. I'll have nothing to put in my coffee11:22
artomsean-k-mooney, err, can you give me a time delta for the meeting? Like, "in 3 hours"?11:22
sean-k-mooneyartom: hehe no worres coff.. childern are importent11:22
sean-k-mooneyartom: it 12:22 now so it would be your 6 am on wednesday11:23
*** jpena is now known as jpena|lunch11:23
artomsean-k-mooney, hah, no.11:23
artom:)11:23
* artom -> milk11:24
*** pcaruana has quit IRC11:26
*** markvoelker has quit IRC11:26
*** phillu has quit IRC11:28
*** udesale has quit IRC11:30
janguttersean-k-mooney: is https://review.openstack.org/#/c/608693/ now more or less in line with your idea of the "generic representor" thingy? If so, I'll update the spec again to reflect this.11:34
*** dtantsur is now known as dtantsur|brb11:37
*** owalsh has joined #openstack-nova11:41
openstackgerritJie Li proposed openstack/nova-specs master: Support volume-backed server rebuild  https://review.openstack.org/53240711:46
*** pcaruana has joined #openstack-nova11:48
artomsean-k-mooney, thanks for thinking of including me, but 6:00 is too damn early for a meeting11:50
artomI'll happily review the specs/code in the normal fashion11:50
*** cdent has joined #openstack-nova11:52
*** maciejjozefczyk has left #openstack-nova11:53
*** maciejjozefczyk has joined #openstack-nova11:53
*** panda is now known as panda|lunch11:55
*** ratailor has quit IRC12:00
sean-k-mooneyartom: no worries ill can give you the clip notes if it impacts the numa aware migration in anyway12:03
*** tiendc has quit IRC12:03
sean-k-mooneyjangutter: sorry spinning a few plates currently ill comment on the review later today12:08
janguttersean-k-mooney: no rush at all, crockery comes first12:09
*** k_mouza has quit IRC12:09
sean-k-mooneyjangutter: if i remember correctly it did but ill have to check it later to confirm12:09
*** janki has quit IRC12:10
*** pcaruana has quit IRC12:12
*** pcaruana has joined #openstack-nova12:12
*** efried_pto is now known as efried12:14
*** brinzhang has quit IRC12:15
*** rnoriega has joined #openstack-nova12:19
*** icey has quit IRC12:19
*** tbachman has joined #openstack-nova12:21
*** jpena|lunch is now known as jpena12:23
*** smcginnis_vaca is now known as smcginnis12:26
*** jaypipes has joined #openstack-nova12:27
*** k_mouza has joined #openstack-nova12:29
jaosoriorhey folks, when configuring a vendordata plugin, from where is the vendordata plugin called? is it from nova-metadata? or from nova-api? or the compute itself?12:29
*** tetsuro_ has quit IRC12:33
jaypipesaspiers: ok, sorry for delay. https://review.openstack.org/#/c/609779/ is now reviewed.12:38
*** alexchadin has quit IRC12:38
aspiersthanks!12:38
jaypipesnp! :)12:38
jaypipesaspiers: sorry, was out at a $work thing all last week :(12:39
aspiersnp at all12:39
aspiersI had enough other feedback to keep me busy ;-)12:39
jaypipesrock on :)12:39
sapd1Hi everyone, I get this errror when live migrate instance. http://paste.openstack.org/show/732581/12:40
sapd1I upgraded from queens to rocky12:40
sapd1libvirt: 3.6 and qemu 2.1112:40
*** owalsh_ has joined #openstack-nova12:48
*** mvkr has quit IRC12:48
*** icey has joined #openstack-nova12:50
*** owalsh has quit IRC12:51
*** alexchadin has joined #openstack-nova12:51
*** jroll has quit IRC12:53
*** jroll has joined #openstack-nova12:54
*** edleafe has joined #openstack-nova12:55
*** panda|lunch is now known as panda12:56
*** owalsh_ is now known as owalsh12:57
*** owalsh has quit IRC12:58
*** mvkr has joined #openstack-nova13:03
*** dtantsur|brb is now known as dtantsur13:04
*** boden has joined #openstack-nova13:04
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform compute_task notifications  https://review.openstack.org/48262913:08
*** alexchadin has quit IRC13:10
*** mriedem has joined #openstack-nova13:11
*** tetsuro has joined #openstack-nova13:12
*** owalsh has joined #openstack-nova13:13
*** mvkr has quit IRC13:23
*** mvkr has joined #openstack-nova13:23
*** eharney has joined #openstack-nova13:27
openstackgerritTakashi NATSUME proposed openstack/nova stable/rocky: Remove unnecessary redirect  https://review.openstack.org/60740013:29
openstackgerritTakashi NATSUME proposed openstack/nova stable/queens: Remove unnecessary redirect  https://review.openstack.org/60740213:29
openstackgerritAdam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances  https://review.openstack.org/60977913:30
aspiersjaypipes: ^^^13:31
*** adrianc_ has quit IRC13:31
*** adrianc has quit IRC13:31
*** munimeha1 has joined #openstack-nova13:33
*** tbachman has quit IRC13:33
*** awaugama has joined #openstack-nova13:36
*** k_mouza_ has joined #openstack-nova13:37
*** boden has left #openstack-nova13:37
*** takashin has quit IRC13:37
openstackgerritMatthew Edmonds proposed openstack/nova master: Use tempfile for powervm config drive  https://review.openstack.org/61017413:39
*** mrch has joined #openstack-nova13:39
*** k_mouza has quit IRC13:39
*** elbragstad is now known as lbragstad13:39
jaypipesaspiers: rock on.13:40
*** takashin has joined #openstack-nova13:42
openstackgerritMarkus Hentsch proposed openstack/nova-specs master: Spec for the Nova part of Image Encryption  https://review.openstack.org/60869613:43
*** k_mouza_ has quit IRC13:46
*** ratailor has joined #openstack-nova13:48
*** k_mouza has joined #openstack-nova13:50
*** moshele has quit IRC13:52
efriedn-sch meeting in 5 minutes in #openstack-meeting-alt13:55
*** adrianc_ has joined #openstack-nova13:56
*** adrianc has joined #openstack-nova13:56
*** liuyulong has joined #openstack-nova13:56
*** liuyulong is now known as liuyulong|away13:57
*** lpetrut has joined #openstack-nova14:03
mriedemalex_xu: you should probably take a look at this https://review.openstack.org/#/c/609591/14:03
*** tbachman has joined #openstack-nova14:03
stephenfinmriedem: Could you take a look at this bug fix series this week? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/178435314:03
alex_xumriedem: got it, will take a look14:04
mriedemstephenfin: yeah14:04
stephenfinta14:04
*** Luzi has quit IRC14:05
sapd1Does anyone get this problem when upgrade from queens to rocky?14:05
*** dpawlik has quit IRC14:07
*** maciejjozefczyk has quit IRC14:08
*** dpawlik has joined #openstack-nova14:08
*** maciejjozefczyk has joined #openstack-nova14:08
s10sapd1: looks like this issue is related to the upgrade to the qemu 2.11, not nova.14:10
*** SteelyDan is now known as dansmith14:11
sapd1s10: I have some compute nodes which are running qemu 2.11 and it gets same issue.14:11
*** ivve has joined #openstack-nova14:12
sapd1Does nova-compute require higher qemu/libvirt version/14:12
sapd1sorry s/2.11/2.10/14:13
mriedemjohnthetubaguy: you might care about this https://review.openstack.org/#/c/609043/14:13
*** erlon has quit IRC14:21
mriedemstephenfin: these are pretty easy https://review.openstack.org/#/q/topic:bug/1763051+(status:open+OR+status:merged) - first is a refactor, second is a try/catch/log but don't blow up sort of thing14:26
mriedemoh sorry, latter is try/catch/rollback14:27
mriedemwas mixing that up with a similar live migration error handling patch14:27
stephenfinmriedem: Sounds good14:27
stephenfinmriedem: Also, I took a look at those debug logs you reworked for the issue awaugama and I are seeing. +2'd the first but do you think the second one should be merged now? Comments are confusing/misleading14:28
mriedemi believe your employer cares about those live migration ones14:28
stephenfinThis one (efried got to it too) https://review.openstack.org/#/c/597560/14:29
mriedemwhat is confusing/misleading?14:29
mriedemi also mostly restored/cleaned that up for awaugama to recreate the problem14:29
mriedemif it's not helpful then i don't care if we don't use it14:29
stephenfin"I don't really intend these overly verbose logs to get merged." doesn't apply anymore, right?14:30
* stephenfin hasn't pulled it in locally so I haven't seen just how noisy it is yet14:30
mriedemcompare PS7 to PS114:30
mriedemit's a much slimmer version14:30
stephenfinack14:31
efriedmriedem: See comments, I still think it's going to be excessive.14:31
efriedbut yeah, I didn't vote because I wasn't sure if we were now intending to actually merge it or if it was still just for people to debug with.14:32
mriedemi just replied14:33
mriedemif you want to wait for awaugama to recreate and say, yes this helped, then let's wait14:33
awaugamaowalsh: can you help with pulling in a patch from upstream into a build for extra logging?14:34
*** k_mouza has quit IRC14:37
*** hamdyk has quit IRC14:38
openstackgerritIvaylo Mitev proposed openstack/nova master: VMware: OVA and StrOpt images as VM templates  https://review.openstack.org/60973614:39
*** mlavalle has joined #openstack-nova14:39
*** tetsuro has quit IRC14:44
*** k_mouza has joined #openstack-nova14:45
*** k_mouza has quit IRC14:47
*** spatel has joined #openstack-nova14:47
*** k_mouza has joined #openstack-nova14:47
*** hshiina has joined #openstack-nova14:56
*** mrch has quit IRC15:01
*** tbachman has quit IRC15:02
*** takashin has left #openstack-nova15:03
mriedemstephenfin: lyarwood: https://review.openstack.org/#/c/587013/15/nova/tests/fixtures.py@172815:04
mriedemam i missing something?15:04
* stephenfin clicks15:04
*** s10 has quit IRC15:07
*** david-lyle has joined #openstack-nova15:07
*** ratailor has quit IRC15:07
*** pcaruana has quit IRC15:12
*** tbachman has joined #openstack-nova15:15
*** tbachman has quit IRC15:20
melwitt15:23
*** cfriesen has quit IRC15:26
*** jmlowe has quit IRC15:27
*** tbachman has joined #openstack-nova15:36
*** manjeets has joined #openstack-nova15:39
*** itlinux has joined #openstack-nova15:46
*** jangutter has quit IRC15:50
*** spatel has quit IRC15:51
*** bnemec has joined #openstack-nova15:51
*** ircuser-1 has joined #openstack-nova15:53
*** adrianc_ has quit IRC15:55
*** adrianc has quit IRC15:55
*** cfriesen has joined #openstack-nova15:56
*** helenafm has quit IRC15:56
*** hshiina has quit IRC15:57
*** aperevalov has joined #openstack-nova15:59
*** bnemec has quit IRC15:59
*** gyee has joined #openstack-nova16:00
stephenfinmriedem: Yeah, but I've confused you in addressing your nit here https://review.openstack.org/#/c/587013/12/nova/tests/fixtures.py@172116:02
stephenfinmriedem: _attachment_id on that line is actually the two item tuple16:02
stephenfinmriedem: See https://review.openstack.org/#/c/587013/15/nova/tests/fixtures.py@171116:03
openstackgerritsean mooney proposed openstack/nova master: harden placement init under wsgi  https://review.openstack.org/61003416:04
stephenfinand two lines above that, where we explode it out into the two item tuple. I guess that should go back to be called 'attachment'16:04
mriedemok commented16:07
*** david-lyle has quit IRC16:07
mriedemi'd like that re-worked because the variable naming is really confusing16:07
stephenfinAgreed. It was called 'attachment' before but I guess that isn't much better16:08
*** sahid has quit IRC16:15
openstackgerritMerged openstack/nova master: Remove more code related to extensions and testing  https://review.openstack.org/60708816:20
openstackgerritMerged openstack/nova master: Log the operation when updating generation in ProviderTree  https://review.openstack.org/59755316:21
openstackgerritStephen Finucane proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow  https://review.openstack.org/58701316:21
openstackgerritStephen Finucane proposed openstack/nova master: Add regression test for bug#1784353  https://review.openstack.org/58701416:21
openstackgerritStephen Finucane proposed openstack/nova master: conductor: Recreate volume attachments during a reschedule  https://review.openstack.org/58707116:21
*** binhnd108 has joined #openstack-nova16:24
*** ttsiouts has quit IRC16:30
*** erlon has joined #openstack-nova16:30
*** ttsiouts has joined #openstack-nova16:36
openstackgerritsean mooney proposed openstack/nova master: harden placement init under wsgi  https://review.openstack.org/61003416:39
*** ttsiouts has quit IRC16:39
openstackgerritMatthew Booth proposed openstack/nova master: Add regression test for bug 1550919  https://review.openstack.org/59173316:41
openstackbug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)16:41
*** k_mouza has quit IRC16:42
mnaserfyi -- if anyone from the nova team hasn't seen this (yet): http://lists.openstack.org/pipermail/openstack-dev/2018-October/135941.html16:43
openstackgerritMatt Riedemann proposed openstack/nova master: Fix jsonutils.to_primitive UserWarning  https://review.openstack.org/61244516:43
*** spatel has joined #openstack-nova16:44
spatelsean-k-mooney: you there?16:44
openstackgerritMatt Riedemann proposed openstack/nova master: Fix jsonutils.to_primitive UserWarning  https://review.openstack.org/61244516:45
sean-k-mooneyspatel:  i was away for a minute but im back16:47
spatelOn 64GB compute node i have configure huge page "hugepagesz=2M hugepages=27000 transparent_hugepage=never"16:47
spatelbased on calculation around 55GB huge pages16:48
spatelis that ok or i should squeeze more pages and keep 4GB for host only16:48
*** efried has quit IRC16:49
*** efried has joined #openstack-nova16:49
sean-k-mooneyi think that is ok you could squees it down to 6 or 4GB for the host but without data to support that it does not need more i would start with leaveing 8GB free16:49
spatelokay! let me go with 8GB free for host16:50
sean-k-mooneyspatel: "wasting" 2-4GB by not allocating extra hugepages is still proably better then having the OOM killer trigger because you were too agressive16:51
spateltotally valid point ++16:51
sean-k-mooneyas you start to operate the cluster over time and you mesure/log the ram usage you will have data that can let you make a more informed desision16:52
spatelalso if i run some agent for data collection or collected / snmp etc.. they will suck some memory too16:52
sean-k-mooneyyep16:53
sean-k-mooney4Gs might be fine without loging but adding collectd or someing else might bump it to 5Gs16:53
spatelsee its alway helpful to have second set of eye :)16:53
*** jmlowe has joined #openstack-nova16:54
spatelThanks!16:54
spateli am going with 8G everywhere now!16:54
*** jpena is now known as jpena|off16:55
sean-k-mooneyleaving 8G free on a host with 32GB is a little painful but 8G on a host with 256GB of ram is perfectly resonable. finding a blance will depend on your workload so you will figure out what makes sense once you deploy and mesure your workload16:55
spatelyup! i am slowly learning all equation and tuning compute nodes based on hardware specs16:56
spatelDo you disable "specter" in kernel ?16:57
spatelI heard it hit you CPU performance16:57
sean-k-mooneydisable the mitigations or disable the speculation16:57
*** mvkr has quit IRC16:57
sean-k-mooneyspatel: that really depends on your threat model.16:57
spatelThis is what i am doing on my compute nodes "spectre_v2=off nopti pti=off nospec_store_bypass_disable spec_store_bypass_disable=off l1tf=off"16:58
sean-k-mooneyspatel: that will give you more performance but also mean your system is vulnerbale to specter attacks16:59
sean-k-mooneyspatel: that said you have a private cloud correct16:59
spatelyes totally private and we are not holding any PI data of any personal16:59
sean-k-mooneyspatel: so you controll all application that are running on the cloud e.g. there are no untrsed applications16:59
spatelwe are just streaming voip data16:59
mriedemmnaser: replied17:00
*** derekh has quit IRC17:00
sean-k-mooneyspatel: well if the lack of mitigations is fine based on your treat model and you know the implications of truning them off then that is your choice to make.17:00
*** tbachman has quit IRC17:01
sean-k-mooneyspatel: it will certenly impove the performnce of some workloads17:01
spatelyup! i have noticed performance in benchmark based on that i have disabled17:01
*** dklyle has joined #openstack-nova17:02
*** erlon has quit IRC17:03
*** moshele has joined #openstack-nova17:03
artommelwitt, did you get a chance to look over https://review.openstack.org/#/c/610739/1? The patch on top (which is the "real" bugfix is already +W)17:04
*** eharney has quit IRC17:04
melwittartom: oh, yes. I started reviewing it and then didn't finish. sorry about that. I'll review it in a few17:05
artommelwitt, no worries. And thanks!17:05
*** dtantsur is now known as dtantsur|afk17:06
mriedempretty sure this is invalid https://bugs.launchpad.net/nova/+bug/179918617:09
openstackLaunchpad bug 1799186 in OpenStack Compute (nova) "Queens compute node is not compatible with Pike Controller node" [Undecided,New]17:09
* artom is wondering what those keys are: " 2. Upgrade controller node to Queens by adding new keys"17:11
dansmithmriedem: definitely17:12
dansmithmriedem: newer compute than control17:13
mriedemthe whole thing is confused,17:13
mriedem"The RPC version of Pike compute is not supported with RPC version of Queens controller."17:13
mriedemthey just aren't doing the rpc version pin setting17:13
mriedemi linked in docs17:13
*** eharney has joined #openstack-nova17:15
openstackgerritDan Smith proposed openstack/nova master: Make CellDatabases fixture reentrant  https://review.openstack.org/61166517:17
openstackgerritDan Smith proposed openstack/nova master: Modify get_by_cell_and_project() to get_not_deleted_by_cell_and_projects()  https://review.openstack.org/60766317:17
openstackgerritDan Smith proposed openstack/nova master: Minimal construct plumbing for nova list when a cell is down  https://review.openstack.org/56778517:17
openstackgerritDan Smith proposed openstack/nova master: Refactor scatter-gather utility to return exception objects  https://review.openstack.org/60793417:17
openstackgerritDan Smith proposed openstack/nova master: Return a minimal construct for nova show when a cell is down  https://review.openstack.org/59165817:17
openstackgerritDan Smith proposed openstack/nova master: Return a minimal construct for nova service-list when a cell is down  https://review.openstack.org/58482917:17
*** sambetts is now known as sambetts|afk17:18
*** bnemec has joined #openstack-nova17:19
*** pcaruana has joined #openstack-nova17:27
*** tbachman has joined #openstack-nova17:28
*** jmlowe has quit IRC17:31
*** jmlowe has joined #openstack-nova17:31
*** irclogbot_4 has joined #openstack-nova17:34
*** ccamacho has quit IRC17:41
*** ralonsoh has quit IRC17:42
*** jmlowe has quit IRC17:43
*** moshele has quit IRC17:52
*** mvkr has joined #openstack-nova17:54
*** AJaeger has joined #openstack-nova17:55
dansmithmriedem: on that metadata thread from the ML, we shouldn't be doing the join thing that multiplies instances * metadata * sysmeta row counts17:56
dansmithbecause we fetch the metadata bits separate and explicitly17:56
AJaegernova cores, I have two changes for your zuul config up - please review https://review.openstack.org/610271 and https://review.openstack.org/612230 . The first converts a job to new Zuul v3 tempest job, the second renames an ill-named job. If you have questions, I'm around to answer...17:56
*** irclogbot_4 has quit IRC17:56
*** pcaruana has quit IRC17:57
dansmithmriedem: because of this: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L224417:57
AJaegernova cores, one more job change: https://review.openstack.org/61014117:58
*** ttsiouts has joined #openstack-nova17:58
*** moshele has joined #openstack-nova18:03
*** jmlowe has joined #openstack-nova18:03
*** irclogbot_4 has joined #openstack-nova18:06
*** spatel has quit IRC18:07
mriedemdansmith: that's when pulling an instance list though, yeah? the metadata api isn't listing instances.18:12
dansmithhmm, is it not on get too?18:12
* dansmith looks18:12
dansmithokay I guess not18:13
dansmithwe use it from a bunch of places but not there18:13
dansmithI guess because it was always that instances * meta * sysmeta made it big(ger)18:14
dansmithso, makes sense I guess18:14
mriedemas far as i can see from a quick search i just don't think we need to pre-load system_metadata in the meta api18:15
dansmithwe did at the time, because of flavors18:15
mriedemhmm, ok, and that's in case of 2007-08-29 being requested i guess18:16
mriedemto return the flavor name18:16
mriedemwell, anything after 2007-08-2918:16
*** faridda has joined #openstack-nova18:16
AJaegermriedem: thanks for reviews!18:17
dansmithmriedem: so I guess I'd just say we could/should just convert single-instance to do the same fill approach, as the real exposure is the user putting a ton of metadata items in (which they control) and making it get big18:18
openstackgerritsean mooney proposed openstack/os-vif master: add support for generic tap device plug  https://review.openstack.org/60238418:24
dansmithmriedem: commented on this: https://review.openstack.org/#/c/602174/1318:32
dansmithmriedem: ready to +2 either way, but seems like a quick removal of the related bug and a host assertion can be done easily if you're up for it18:32
mriedemi can18:34
mriedemnote the coincidental fun with forcing the evacuate to bypass the ComputeFilter https://review.openstack.org/#/c/602174/13/gate/test_evacuate.sh@5618:35
dansmithwhat about it?18:36
mriedemrelated to what was being discussed in gibi's spec about deprecating the force flag,18:37
mriedemand when people might actually use it (to bypass the ComputeFilter)18:37
dansmithoh18:37
mriedembut in this case, it's just negatively trying to force us to the down host18:37
*** markguz_ has joined #openstack-nova18:40
dansmithokay I'm not sure if I'm missing something,18:42
markguz_Hi nova peeps.  Question about nova-compute on pike , on ppc64le on ubuntu 16.04LTS.18:42
dansmithyou're downing libvirt so the current host is marked as down,18:42
dansmithand then making sure that we fail even when forcing it right?18:42
markguz_after running updates recently i noticed that i couldn't spin up vms any longer18:42
markguz_getting this error qemu-system-ppc64: failed to initialize KVM: Invalid argument18:42
markguz_any other ppc64le users out there having issues on ubuntu?18:43
AJaegerthanks, melwitt !18:44
dansmithmarkguz_: see topic18:44
melwittAJaeger: thanks for the patches18:45
markguz_dansmith: i see. sorry!18:45
*** moshele has quit IRC18:46
sean-k-mooneymarkguz_: #openstack may be more helpful but it sounds like you may have some packaging/depency issues. not many people use powerkvm as far as i know18:47
markguz_sean-k-mooney: could be, but i'm using the official ubuntu stuff, not making my own packages.18:49
sean-k-mooneymarkguz_: well i was more thinking you kvm module paramters file my have somthing that was invalid. anyway this seams more like a distro issue then nova. sorry i can be of more help.18:50
*** AJaeger has left #openstack-nova18:54
*** lpetrut has quit IRC18:56
mmedvedemarkguz_: Hi, I can help. Not sure what is the best place to discuss, probably #openstack is as good place as any18:57
*** moshele has joined #openstack-nova18:58
markguz_mmedvede: i'm logged into #openstack18:58
mriedemdansmith: "and then making sure that we fail even when forcing it right?" yes18:58
mriedemit's a negative scenario,18:58
dansmithright18:58
mriedemthen bring the target back up and evacuate to it18:58
*** ttsiouts has quit IRC19:01
mriedemis lee still out for awhile?19:03
dansmithI think another week at least, IIRC19:03
mriedemok19:05
*** moshele has quit IRC19:05
dansmithback wednesday of next week it looks like19:07
*** irclogbot_4 has quit IRC19:14
*** ttsiouts has joined #openstack-nova19:18
openstackgerritMerged openstack/nova master: Move live_migration.pre.start to the start of the method  https://review.openstack.org/61073919:33
openstackgerritMerged openstack/nova master: Ensure attachment cleanup on failure in driver.pre_live_migration  https://review.openstack.org/58743919:33
openstackgerritMatt Riedemann proposed openstack/nova master: conductor: Recreate volume attachments during a reschedule  https://review.openstack.org/58707119:35
mriedemstephenfin: addressed my nits in ^ and i'll +219:35
*** awaugama has quit IRC19:42
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: fixtures: Track volume attachments within CinderFixtureNewAttachFlow  https://review.openstack.org/61248519:43
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Add regression test for bug#1784353  https://review.openstack.org/61248619:43
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: conductor: Recreate volume attachments during a reschedule  https://review.openstack.org/61248719:43
*** jmlowe has quit IRC19:46
*** jmlowe has joined #openstack-nova19:49
*** jmlowe has quit IRC19:51
*** ttsiouts has quit IRC20:04
*** ttsiouts has joined #openstack-nova20:05
*** jamesdenton has quit IRC20:09
*** ttsiouts has quit IRC20:09
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: fixtures: Track volume attachments within CinderFixtureNewAttachFlow  https://review.openstack.org/61249420:10
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Add regression test for bug#1784353  https://review.openstack.org/61249520:10
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: conductor: Recreate volume attachments during a reschedule  https://review.openstack.org/61249620:10
*** jmlowe has joined #openstack-nova20:12
openstackgerritEric Fried proposed openstack/nova-specs master: Spec: Provider config YAML file  https://review.openstack.org/61249720:16
*** faridda has quit IRC20:20
*** faridda has joined #openstack-nova20:21
*** cdent has quit IRC20:22
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/61250020:27
cfriesenif I get a -1 from Zuul in the "verified" section of a review, is there a way to see what it didn't like?20:28
cfriesenmaybe that's an infra question20:28
melwittwhat do you mean other than clicking the link for the failed job?20:29
*** ttsiouts has joined #openstack-nova20:39
*** ttsiouts has quit IRC20:43
*** ivve has quit IRC20:48
*** ttsiouts has joined #openstack-nova21:01
mriedemartom: are you going to start backports? https://review.openstack.org/#/q/topic:bug/1763051+(status:open+OR+status:merged)21:07
mriedemmelwitt: dansmith: one of you probably want this https://review.openstack.org/#/c/611093/21:08
mriedemfor your sweet sweet customers21:08
melwittyass21:08
*** faridda has quit IRC21:10
melwittgah, gate, you are killing me21:15
*** slaweq has quit IRC21:24
artommriedem, yep21:25
openstackgerritEric Fried proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/61250021:30
*** gouthamr has quit IRC21:32
openstackgerritMatt Riedemann proposed openstack/nova master: Add post-test hook for testing evacuate  https://review.openstack.org/60217421:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add volume-backed evacuate test  https://review.openstack.org/60439721:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional regression test for bug 1794996  https://review.openstack.org/60610621:34
openstackbug 1794996 in OpenStack Compute (nova) "_destroy_evacuated_instances fails and kills n-cpu startup if lazy-loading flavor on a deleted instance" [High,In progress] https://launchpad.net/bugs/1794996 - Assigned to Matt Riedemann (mriedem)21:34
openstackgerritMatt Riedemann proposed openstack/nova master: Fix InstanceNotFound during _destroy_evacuated_instances  https://review.openstack.org/60612221:34
openstackgerritMatt Riedemann proposed openstack/nova master: Refactor TestEvacuateDeleteServerRestartOriginalCompute  https://review.openstack.org/60870521:34
openstackgerritMatt Riedemann proposed openstack/nova master: Run evacuate tests with local/lvm and shared/rbd storage  https://review.openstack.org/60440021:34
mriedemdansmith: added that assertion to the bottom change ^21:34
*** spsurya has quit IRC21:38
dansmithmriedem: cool, I meant to work my way up that series but got distracted.. will try to hit the rest tomorrow21:46
mriedemnp21:49
mriedemthanks21:49
*** priteau has quit IRC21:49
mriedemoh btw, i dawned on me on friday that with this detach/attach root volume spec https://review.openstack.org/#/c/600628/ that the underlying root image could change, which is maybe part of the point21:51
mriedemthat's also kind of a rebuild territory kind of thing21:51
mriedem*it dawned21:52
mriedemanyway, i'm sure nothing could go wrong there.21:53
*** spatel has joined #openstack-nova22:01
*** spatel has quit IRC22:05
*** mriedem is now known as mriedem_away22:07
*** spatel has joined #openstack-nova22:08
*** spatel has quit IRC22:13
*** bnemec has quit IRC22:18
openstackgerritAdam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances  https://review.openstack.org/60977922:23
*** markguz_ has quit IRC22:45
*** rcernin has joined #openstack-nova22:46
openstackgerritMerged openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow  https://review.openstack.org/58701322:53
openstackgerritMerged openstack/nova master: Add regression test for bug#1784353  https://review.openstack.org/58701422:53
*** cfriesen has quit IRC22:56
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Per-instance serial number (spec)  https://review.openstack.org/61253122:56
*** gyee has quit IRC23:00
*** eharney has quit IRC23:08
openstackgerritsean mooney proposed openstack/os-vif master: add isolate_vif config option  https://review.openstack.org/61253423:22
*** mriedem_away has quit IRC23:27
*** rcernin_ has joined #openstack-nova23:28
*** rcernin has quit IRC23:30
*** liuyulong|away has quit IRC23:31
*** dklyle has quit IRC23:32
*** david-lyle has joined #openstack-nova23:32
*** david-lyle has quit IRC23:35
*** dklyle has joined #openstack-nova23:35

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