Monday, 2019-05-13

*** boxiang has quit IRC00:48
*** altlogbot_1 has quit IRC00:50
openstackgerritzhufl proposed openstack/nova master: Change some URLs to point to better targets  https://review.opendev.org/65831200:52
*** altlogbot_3 has joined #openstack-nova00:55
*** ileixe has quit IRC01:03
*** bbowen has quit IRC01:06
*** bbowen has joined #openstack-nova01:08
*** slaweq has joined #openstack-nova01:11
*** slaweq has quit IRC01:15
*** awalende has joined #openstack-nova01:23
*** awalende has quit IRC01:28
*** ircuser-1 has quit IRC01:32
*** bbowen has quit IRC01:38
*** bbowen has joined #openstack-nova01:38
*** brinzhang has joined #openstack-nova01:49
*** ricolin has joined #openstack-nova02:22
*** ricolin_ has joined #openstack-nova02:31
openstackgerritYongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.opendev.org/65871602:31
*** ricolin has quit IRC02:33
*** BjoernT has joined #openstack-nova02:49
*** BjoernT has quit IRC02:53
*** BjoernT has joined #openstack-nova02:54
*** whoami-rajat has joined #openstack-nova03:05
*** slaweq has joined #openstack-nova03:11
*** slaweq has quit IRC03:16
*** psachin has joined #openstack-nova03:19
*** ricolin_ has quit IRC03:44
*** udesale has joined #openstack-nova03:56
*** david-lyle is now known as dklyle03:58
*** ricolin_ has joined #openstack-nova04:03
*** dklyle has quit IRC04:03
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8)  https://review.opendev.org/57531104:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9)  https://review.opendev.org/57558104:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10)  https://review.opendev.org/57601704:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11)  https://review.opendev.org/57601804:06
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12)  https://review.opendev.org/57601904:06
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13)  https://review.opendev.org/57602004:06
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14)  https://review.opendev.org/57602704:07
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15)  https://review.opendev.org/57603104:07
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16)  https://review.opendev.org/57629904:08
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17)  https://review.opendev.org/57634404:08
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18)  https://review.opendev.org/57667304:08
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19)  https://review.opendev.org/57667604:09
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20)  https://review.opendev.org/57668904:09
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21)  https://review.opendev.org/57670904:09
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22)  https://review.opendev.org/57671204:09
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py  https://review.opendev.org/56615304:10
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens  https://review.opendev.org/63771604:10
*** itlinux has joined #openstack-nova04:10
openstackgerritTakashi NATSUME proposed openstack/nova master: Add a live migration regression test  https://review.opendev.org/64120004:10
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Fix a description for config_drive parameter  https://review.opendev.org/65368304:11
*** slaweq has joined #openstack-nova04:11
*** slaweq has quit IRC04:16
*** BjoernT has quit IRC04:25
*** Luzi has joined #openstack-nova04:42
*** jaosorior has joined #openstack-nova04:43
*** dpawlik has joined #openstack-nova04:44
*** jaosorior has quit IRC04:52
*** tjgresha has quit IRC04:57
*** tjgresha has joined #openstack-nova04:57
*** jaosorior has joined #openstack-nova05:00
*** hongbin has quit IRC05:02
*** ratailor has joined #openstack-nova05:03
*** ivve has quit IRC05:07
*** slaweq has joined #openstack-nova05:11
*** slaweq has quit IRC05:16
*** boxiang has joined #openstack-nova05:18
*** ricolin_ has quit IRC05:31
*** ricolin has joined #openstack-nova05:36
*** guozijn has joined #openstack-nova05:38
*** lpetrut has joined #openstack-nova05:39
*** guozijn has left #openstack-nova05:40
*** guozijn has joined #openstack-nova05:41
*** udesale has quit IRC05:45
*** udesale has joined #openstack-nova05:45
*** udesale has quit IRC05:49
*** udesale has joined #openstack-nova05:49
*** guozijn_ has joined #openstack-nova05:55
*** lpetrut has quit IRC05:55
*** guozijn has quit IRC05:56
*** guozijn_ is now known as guozijn05:56
*** alex_xu has joined #openstack-nova05:59
*** lpetrut has joined #openstack-nova06:06
*** sapd1_x has joined #openstack-nova06:11
*** slaweq has joined #openstack-nova06:11
*** slaweq has quit IRC06:16
*** pcaruana has joined #openstack-nova06:16
*** boxiang has quit IRC06:17
*** ivve has joined #openstack-nova06:17
openstackgerritYongli He proposed openstack/nova master: Add server sub-resource topology API  https://review.opendev.org/62147606:17
*** boxiang has joined #openstack-nova06:17
*** lpetrut has quit IRC06:22
openstackgerritMerged openstack/nova master: Change some URLs to point to better targets  https://review.opendev.org/65831206:38
*** udesale has quit IRC06:47
*** udesale has joined #openstack-nova06:47
*** maciejjozefczyk has joined #openstack-nova06:56
*** awalende has joined #openstack-nova06:58
*** awalende has quit IRC07:01
*** rcernin has quit IRC07:06
*** slaweq has joined #openstack-nova07:12
*** tesseract has joined #openstack-nova07:17
*** rpittau|afk is now known as rpittau07:18
*** ccamacho has joined #openstack-nova07:20
*** yan0s has joined #openstack-nova07:21
*** slaweq has quit IRC07:28
*** igordc has quit IRC07:29
*** tssurya has joined #openstack-nova07:31
*** itlinux has quit IRC07:35
*** helenafm has joined #openstack-nova07:39
*** awalende has joined #openstack-nova07:40
*** igordc has joined #openstack-nova07:44
*** janki has joined #openstack-nova07:46
*** ralonsoh has joined #openstack-nova07:53
*** igordc has quit IRC07:55
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif stable/queens: Prevent "qbr" Linux Bridge from replying to ARP messages  https://review.opendev.org/65569408:05
*** jangutter has joined #openstack-nova08:05
*** ricolin has quit IRC08:15
*** ricolin has joined #openstack-nova08:15
*** ricolin has quit IRC08:19
*** ricolin has joined #openstack-nova08:20
*** ricolin has quit IRC08:21
*** tkajinam has quit IRC08:31
openstackgerritBoxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server  https://review.opendev.org/64552008:38
*** derekh has joined #openstack-nova08:48
*** sapd1_x has quit IRC08:57
*** takashin has left #openstack-nova09:00
*** xek has joined #openstack-nova09:18
ralonsohstephenfin, hi. I'm having problems with https://review.opendev.org/#/c/655694/ (Queens) because of sphinx09:24
*** sapd1_x has joined #openstack-nova09:24
ralonsohstephenfin, is ok if I cherry-pick https://review.opendev.org/#/c/479862/ to Queens?09:24
*** mgoddard has joined #openstack-nova09:25
*** wxy-xiyuan has joined #openstack-nova09:39
*** dpawlik has quit IRC09:41
*** dpawlik has joined #openstack-nova09:41
jangutterralonsoh: hmmm, the other option is to pin sphinx to an older version for the queens release.09:57
ralonsohI'll take a look at sphinx versions to know which one supports both, 2 and 309:57
*** janki has quit IRC09:58
ralonsohjangutter, probably <2.009:58
*** janki has joined #openstack-nova09:58
jangutterralonsoh: yep, out of interest, what does Nova do for queens?09:58
ralonsohjangutter, I'll review this now09:58
stephenfinralonsoh: Sphinx isn't the problem. We need this 96b5a11d10:02
stephenfinSorry, https://opendev.org/openstack/os-vif/commit/96b5a11d10:03
ralonsohstephenfin, yes, I saw this. os-vif was not using upper-constraints10:03
stephenfinyup10:04
stephenfinbad os-vif10:04
stephenfinbad10:04
ralonsohhehehe10:04
ralonsohshould I cherry-pick this patch to previous versions? Queens?10:04
stephenfinYeah, but change the URL to https://releases.openstack.org/constraints/upper/queens10:05
ralonsohstephenfin, perfect. Thanks!!10:05
stephenfin(And include a 'NOTE(ralonsoh):' in the commit message saying that, so people know wtf happened in the future :D)10:05
ralonsohstephenfin, perfect10:06
*** lpetrut has joined #openstack-nova10:12
openstackgerritBalazs Gibizer proposed openstack/nova master: Defaults missing group_policy to 'none'  https://review.opendev.org/65779610:19
*** guozijn has quit IRC10:23
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif stable/queens: add upper_constraints support  https://review.opendev.org/65878010:24
ralonsohstephenfin, please feel free to modify the commit message to add a good NOTE https://review.opendev.org/#/c/658780/10:24
ralonsohstephenfin, yours will be better than mine10:24
*** slaweq has joined #openstack-nova10:25
stephenfinralonsoh: done10:29
ralonsohstephenfin, thanks!10:29
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif stable/queens: add upper_constraints support  https://review.opendev.org/65878010:33
openstackgerritMerged openstack/python-novaclient master: Tiny fix of documentation  https://review.opendev.org/65752210:35
openstackgerritHamdy Khader proposed openstack/nova master: Process VIF type ovs-dpdk for dpdk representor ports  https://review.opendev.org/65878510:44
openstackgerritHamdy Khader proposed openstack/os-vif master: Plug ovs-dpdk vif port type for dpdk representor integration  https://review.opendev.org/65878610:47
*** tbachman has quit IRC10:48
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif stable/queens: Prevent "qbr" Linux Bridge from replying to ARP messages  https://review.opendev.org/65569410:54
*** nicolasbock has joined #openstack-nova11:04
*** slaweq has quit IRC11:09
*** udesale has quit IRC11:17
*** baderbuddy has joined #openstack-nova11:20
*** nicolasbock has quit IRC11:20
*** nicolasbock has joined #openstack-nova11:20
*** baderbuddy has quit IRC11:26
*** janki has quit IRC11:29
*** jchhatbar has joined #openstack-nova11:29
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif stable/queens: add upper_constraints support  https://review.opendev.org/65878011:41
openstackgerritRodrigo Barbieri proposed openstack/nova stable/queens: [DEBUG] Add functional confirm_migration_error test  https://review.opendev.org/65813611:44
*** boxiang has quit IRC11:51
*** boxiang has joined #openstack-nova11:51
*** boxiang has quit IRC11:54
*** boxiang has joined #openstack-nova11:57
*** tbachman has joined #openstack-nova12:11
*** slaweq has joined #openstack-nova12:26
*** mgariepy has quit IRC12:27
*** mgariepy has joined #openstack-nova12:31
*** ratailor has quit IRC12:34
openstackgerritSurya Seetharaman proposed openstack/python-novaclient master: Microversion 2.73: Support adding the reason behind a server lock  https://review.opendev.org/64865912:38
*** psachin has quit IRC12:46
openstackgerritMerged openstack/nova master: api-ref: fix mention of all_tenants filter for non-admins  https://review.opendev.org/65762012:47
*** slaweq has quit IRC12:52
*** pcaruana|afk| has joined #openstack-nova12:55
*** pcaruana has quit IRC12:57
*** rchurch has joined #openstack-nova12:58
openstackgerritMerged openstack/nova master: Update Python 3 test runtimes for Train  https://review.opendev.org/65794112:58
*** wxy-xiyuan has quit IRC12:59
*** jbernard_ has joined #openstack-nova12:59
*** rpittau has quit IRC12:59
*** rchurch_ has quit IRC12:59
*** psyton has quit IRC12:59
*** vdrok has quit IRC12:59
*** jdillaman has joined #openstack-nova13:00
*** jbernard has quit IRC13:00
*** vdrok has joined #openstack-nova13:00
*** rpittau has joined #openstack-nova13:02
*** BjoernT has joined #openstack-nova13:03
*** brinzhang has quit IRC13:09
*** slaweq has joined #openstack-nova13:09
*** Luzi has quit IRC13:10
*** udesale has joined #openstack-nova13:10
*** wxy-xiyuan has joined #openstack-nova13:14
*** udesale has quit IRC13:18
*** udesale has joined #openstack-nova13:18
*** efried has quit IRC13:19
*** efried has joined #openstack-nova13:20
*** mchlumsky has joined #openstack-nova13:23
*** slaweq has quit IRC13:26
*** dpawlik has quit IRC13:35
kashyapefried: When you're about, thanks for the feedback on the 'traits' thingie.  While I don't want to belabor this, please see the "Another Idea" section here: https://review.opendev.org/#/c/655193/413:39
*** awalende has quit IRC13:40
kashyapefried: If you (and any others) think that's sensible (I think it is, FWIW), I'll go and file a bug to track it.13:40
kashyap(And of course, then fix it :D)13:40
efriedkashyap: ack, looking.13:40
*** awalende has joined #openstack-nova13:41
kashyap(Sorry for the broken formatting; I wrote it locally in plain text and posted it there.  As typing in the tiny window of Grrrrit was infuriating.)13:41
*** awalende has quit IRC13:42
efriedkashyap: You know you can resize the tiny window, right?13:43
efriedhandle on the bottom right13:43
kashyapefried: No, I didn't realize that13:43
efriedre intel vs amd, it sounds like you're agreeing with me? or not?13:43
*** awalende has joined #openstack-nova13:43
kashyapHa!13:43
*** awalende has quit IRC13:43
kashyap(Just saw that)13:43
*** awalende has joined #openstack-nova13:44
efriedbrb, gotta med a cat13:44
kashyapefried: You mean, on having the generic *_CURE traits?13:44
kashyapSure13:44
kashyap(I need to go into a 20-min meeting in 16 min, then be here for 30 min, then head out to catch a train back home)13:45
openstackgerritchenker proposed openstack/python-novaclient master: Optimize limit option docs string description for novaclient  https://review.opendev.org/64303513:45
dtantsurhi folks! how crazy would it be to backport https://review.opendev.org/#/c/563722/ ? it's a bug fix, but it adds new driver calls..13:45
*** janki has joined #openstack-nova13:47
*** janki has quit IRC13:47
kashyapefried: When you're back: on picking Intel/AMD-specific traits, I think we should try not to "abstract" them out, and have separate traits per vendor, for clarity.  (Open to change my mind if I missed to consider something more important.)13:48
*** jchhatbar has quit IRC13:48
*** awalende has quit IRC13:48
kashyapdtantsur: Looks fairly invasive to me.  Depends on how far you want to backport13:49
kashyapdtantsur: Also, how 'important' it is - as in, if the bug is severe, then probably you might get away with it13:49
dtantsurI'm mostly interested in queens13:49
efriedkashyap: Intel/AMD I'm just talking about making the namespaces more distinctive.13:49
efriedkashyap: tbc, I agree that we should keep the granular traits.13:49
efriedAnd making rollup traits, if we do it at all, could still be done later.13:50
kashyapAh, okay.  efried -- Sorry, more distinctive, how?13:50
kashyapefried: Yeah, good point (that the door is open for roll-up for future).13:50
efriedlike hw/cpu/x86/intel and hw/cpu/x86/amd13:50
*** amodi has joined #openstack-nova13:51
efriedkashyap: but my point about the granular traits with no rollups is that they don't help operators unless they're okay predetermining whether they want amd or intel.13:51
efriedbecause we don't have required=in: yet13:51
dtantsurpeople suspect it may be the cause or one of the causes of https://bugzilla.redhat.com/show_bug.cgi?id=1560690#c6013:52
openstackbugzilla.redhat.com bug 1560690 in openstack-heat "Re-deployment of overcloud fails with: ResourceInError: resources.OpenShiftWorker: Went to status ERROR due to "Message: Build of instance c7c5191b-59ed-44a0-8b2a-0f68e48e9a52 aborted: Failure prepping block device., Code: 500"" [High,Assigned] - Assigned to jkreger13:53
* dtantsur hopes the link is public13:53
kashyapefried: Yeah, okay, I agree with the making more distinctive part.13:53
dansmithdtantsur: doesn't seem too crazy13:53
kashyapefried: Right - I see that point.  Hmm, damn "real world"13:53
dansmithIMHO13:53
*** priteau has joined #openstack-nova13:53
kashyapefried: What do you think about the 'sysfs' thingie?13:53
*** BjoernT has quit IRC13:53
kashyapefried: I.e. Nova traversing /sys/devices/system/cpu/vulnerabilities/meltdown (and /sys/devices/system/cpu/vulnerabilities/spectre_*) -- if either of them report vulnerable, then warn that future Nova will refuse to launch VMs on that host13:53
kashyap(And next release, make it good on the warning promise.13:54
kashyap)13:54
* kashyap bbiab13:54
efriedkashyap: Reporting generically "vulnerable" as opposed to "mitigation" - sure, that's fine by me, whatever the implementation winds up being.13:54
*** takashin has joined #openstack-nova13:54
kashyapefried: Right.  I was just noting the notion of what the 'sysfs' reports.  If you're _secure_, it is represented as: (You have) "Mitigation": $MITIGATION13:55
efriedkashyap: I don't think we should enforce "you can't land on a vulnerable host" across the board. There are surely some environments where they don't care (if all vms in the az are trusted, or whatever)13:55
kashyapefried: Yeah, I hear ya.13:55
efriedkashyap:  being able to expose that as a trait, where it's not intel/amd specific, is the part I care about.13:55
efriedwhether it's positive or negative, or how it's determined, I trust you :)13:56
efriedso13:56
efriedfor this patch13:56
efriedfix the typo13:56
efriedmake intel and amd namespaces and move the granular traits appropriately13:56
efrieddone13:57
efriedyah?13:57
kashyapefried: Yeah, makes sense.13:57
efriedand then make a new patch with the 'vulnerable' thing in the generic x86 namespace in whatever form you decide will work.13:57
kashyapNod13:58
kashyapefried: Thanks for your time and discussion!13:58
*** BjoernT has joined #openstack-nova13:59
efriedthanks for your patience13:59
*** BjoernT has quit IRC14:02
*** BjoernT has joined #openstack-nova14:03
*** slaweq has joined #openstack-nova14:05
*** gmann has joined #openstack-nova14:07
*** gmann is now known as gmann_pto14:07
*** ratailor has joined #openstack-nova14:10
*** jbernard_ is now known as jbernard14:13
*** artom has joined #openstack-nova14:24
*** ratailor has quit IRC14:24
openstackgerritBalazs Gibizer proposed openstack/nova master: Defaults missing group_policy to 'none'  https://review.opendev.org/65779614:25
*** mriedem has joined #openstack-nova14:25
*** slaweq has quit IRC14:34
*** takashin has quit IRC14:35
openstackgerritRodrigo Barbieri proposed openstack/nova stable/rocky: [DEBUG] Add functional confirm_migration_error test  https://review.opendev.org/65883414:35
*** sean-k-mooney has joined #openstack-nova14:39
*** sean-k-mooney has left #openstack-nova14:39
*** sean-k-mooney has quit IRC14:39
*** sean-k-mooney has joined #openstack-nova14:40
*** hongbin has joined #openstack-nova14:53
johnthetubaguystephenfin: added a -2 as this looks like an API change that breaks our microversion contract, would be great if you can double check my reasoning on there: https://review.opendev.org/#/c/35029914:54
*** pcaruana|afk| has quit IRC14:55
stephenfinjohnthetubaguy: Ack. I'll take a look14:56
johnthetubaguythanks14:57
*** macza has joined #openstack-nova14:57
*** __hemna has quit IRC14:57
*** belmoreira has joined #openstack-nova14:58
*** beekneemech is now known as bnemec15:00
*** tbachman has quit IRC15:01
sean-k-mooneyjohnthetubaguy: ya i think that proably should have a micro version bump if we make that change15:02
johnthetubaguyits a deprecated API now, so I think we just block it? seems harsh though15:03
johnthetubaguyone alternative is to error with conflict when ports that have a floating ip are changed via that API, as we know something odd happens15:04
sean-k-mooneyjohnthetubaguy: i was thinking something similar15:04
sean-k-mooneybut i was not sure we were allowed to15:05
sean-k-mooneypersonally i would prefer that you were requried to disassociate the fip first15:05
johnthetubaguyits borderline...15:06
sean-k-mooneybut again that would need a micro version too right15:06
sean-k-mooneywhat we could do is just remove the fixed ip but not the port id from the floating ip15:07
johnthetubaguyhmm, I though that is what happened today?15:08
johnthetubaguyoh wait, I see what you mean now15:08
johnthetubaguyhmm, that is probably more surprising at this point15:08
sean-k-mooneyif the port has multiple fixed ips and you dont specify one when setting the fip assocation it just uses the first15:09
johnthetubaguyhmm, re-reading15:09
sean-k-mooneyso if the port has multiple fixed ips i dont know if you would expect it to be retarged to one of the others or not15:09
artomefried, https://review.opendev.org/#/c/603955/12/specs/train/approved/nova-cyborg-interaction.rst@24915:12
artomSee PS11, L25015:12
efriedthanks artom15:13
sean-k-mooneyanyway i can see the logic of removing the fip association and also of not. since we dont know what we should do i feel raising an error is best but honestly i would be fine with doing nothing and not merging the patch also.15:14
dansmithsean-k-mooney: johnthetubaguy that was last touched by the owner in 2017 right?15:14
dansmithI think we can probably just ignore it15:14
*** wwriverrat has joined #openstack-nova15:14
*** ircuser-1 has joined #openstack-nova15:15
johnthetubaguydansmith: oops, I see even the +2 was from 2017 now...15:16
dansmithjohnthetubaguy: yeah :)15:16
openstackgerritMatthew Booth proposed openstack/nova master: Fix retry of instance_update_and_get_original  https://review.opendev.org/65884515:17
sean-k-mooneydansmith: mriedem  rebased it in april i wonder if he had a customer ask about it15:17
mriedemnope15:17
*** yan0s has quit IRC15:17
sean-k-mooneyin that case ya we canb proably just abandon it15:18
johnthetubaguyI was trying to find some low hanging +Ws (facepalm)15:18
mriedemi likely saw it at the top of http://status.openstack.org/reviews/#nova during the forum and just rebased it to see if it still worked15:18
dansmithjohnthetubaguy: a noble goal :)15:18
johnthetubaguyah, similar, found it in the review dashboard15:18
johnthetubaguyabandoned it, its a distraction for sure15:19
openstackgerritMerged openstack/nova master: Trivial: Adds comments and tests for scheduler  https://review.opendev.org/65851615:19
mriedemas noted in the review, the remove_fixed_ip_from_instance (and add counterpart) were added to mirror nova-net functionality and i think i pointed out (now years ago) that it was more or less super janky15:20
gansomriedem: Hi Matt, I fixed the functional test and ran it against queens and rocky (https://review.opendev.org/#/c/658136 and https://review.opendev.org/#/c/658834) and unfortunately it seems the fix that merged in rocky does not solve the issue15:20
mriedemok15:24
mriedemjohnthetubaguy: super old bug for the same api https://bugs.launchpad.net/nova/+bug/153326015:28
openstackLaunchpad bug 1533260 in OpenStack Compute (nova) "Unexpected API Error is returned, when trying to add fixed IP to an instance in ERROR state" [Medium,Confirmed]15:28
mriedemwell, same set of apis15:28
mriedemhttps://review.opendev.org/#/c/163864/ was the super old re-write of the apis15:29
mriedemabandoned in 2016 with "does anyone use this? don't think so"15:30
johnthetubaguyyeah, although that fix looks fine, but meh.15:31
mriedem"The more I think about, maybe the existing behaviour is correct. I need to catch up with mriedem about this."15:32
mriedemjuly 4 201615:32
mriedemi was knee deep in grilled meats and explosives15:33
mriedemganso: i'll pull your rocky test down and play with it to see if i can figure out what's going on15:35
gansomriedem: thanks Matt! =)15:36
mriedemsean-k-mooney: in case you didn't notice, i got a recreate on https://review.opendev.org/#/c/653506/ stacked above it and cleaned it up, should be good to go now15:36
mriedemalso figured out what i was doing wrong in the cross-cell resize series that was hitting that, so once i got that fixed i got a clean run in the multi cell job for cross-cell cold migration15:36
*** itlinux has joined #openstack-nova15:37
*** igordc has joined #openstack-nova15:37
*** hongbin has quit IRC15:39
*** igordc has quit IRC15:40
*** pcaruana|afk| has joined #openstack-nova15:44
*** mvkr has joined #openstack-nova15:46
*** igordc has joined #openstack-nova15:47
openstackgerritBalazs Gibizer proposed openstack/os-traits master: Add REPORT_PARENT_INTERFACE_NAME_FOR_SRIOV_NIC trait  https://review.opendev.org/65885215:51
*** jangutter has quit IRC15:52
*** __hemna has joined #openstack-nova15:57
*** __hemna has quit IRC15:58
*** tbachman has joined #openstack-nova15:58
*** hemna has joined #openstack-nova15:59
*** mlavalle has joined #openstack-nova16:03
*** helenafm has quit IRC16:03
*** BjoernT has quit IRC16:04
*** canori01 has joined #openstack-nova16:06
canori01hello, I am running nova 18.0.2 and setting disk-cachemodes to writeback16:07
canori01However, the instance xml says none. Is this intended behavior?16:08
sean-k-mooneymriedem: hi sorry was in a meeting16:09
sean-k-mooneymriedem: im just grabing lunch but ill read through comment you left thanks for looking at this in depth16:09
*** awalende has joined #openstack-nova16:14
*** awalende has quit IRC16:19
*** belmoreira has quit IRC16:19
*** tesseract has quit IRC16:20
gibiefried: I'm going to be away til Wednesday next week so I'm wondering if I should pul the heal port allocation from the runway slot16:20
*** dtantsur is now known as dtantsur|afk16:23
*** rpittau is now known as rpittau|afk16:25
*** roukoswarf is now known as rouk16:30
*** whoami-rajat has quit IRC16:31
*** rouk is now known as roukoswarf16:32
*** whoami-rajat has joined #openstack-nova16:36
*** priteau has quit IRC16:38
*** roukoswarf is now known as rouk16:40
openstackgerritMatthew Booth proposed openstack/nova master: Fix retry of instance_update_and_get_original  https://review.opendev.org/65884516:45
openstackgerritMerged openstack/os-vif master: Replace git.openstack.org URLs with opendev.org URLs  https://review.opendev.org/65506816:48
*** ivve has quit IRC16:48
*** mgoddard has quit IRC16:50
*** nicolasbock has quit IRC16:50
*** nicolasbock has joined #openstack-nova16:50
*** mgoddard has joined #openstack-nova16:52
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif stable/queens: add upper_constraints support  https://review.opendev.org/65878016:53
*** mgoddard has quit IRC16:57
*** imacdonn has joined #openstack-nova16:57
*** mgoddard has joined #openstack-nova16:58
*** udesale has quit IRC16:59
*** slaweq has joined #openstack-nova17:04
*** rouk has quit IRC17:07
*** mgoddard has quit IRC17:16
*** med_ has joined #openstack-nova17:17
*** mgoddard has joined #openstack-nova17:21
*** mchlumsky has quit IRC17:23
openstackgerritMerged openstack/nova master: Cleanup update_instance cell mapping handling  https://review.opendev.org/46738017:23
*** lpetrut has quit IRC17:27
*** mchlumsky has joined #openstack-nova17:29
*** ralonsoh has quit IRC17:32
*** jroll has quit IRC17:34
*** jroll has joined #openstack-nova17:35
openstackgerritMerged openstack/nova master: Remove cells v2 transition code from update_instance  https://review.opendev.org/46738217:35
openstackgerritMerged openstack/nova master: api-ref: 'os-hypervisors' doesn't reflect overcommit ratio  https://review.opendev.org/61160417:35
*** derekh has quit IRC17:37
*** slaweq has quit IRC17:43
efriedgibi: Probably a good idea.17:44
*** itlinux has quit IRC17:50
*** BjoernT has joined #openstack-nova17:50
*** itlinux has joined #openstack-nova17:53
*** sapd1_x has quit IRC17:56
*** dklyle has joined #openstack-nova17:59
*** slaweq has joined #openstack-nova18:01
*** igordc has quit IRC18:04
*** dklyle has quit IRC18:08
*** slaweq has quit IRC18:12
*** ivve has joined #openstack-nova18:19
*** dklyle has joined #openstack-nova18:23
openstackgerritArtom Lifshitz proposed openstack/nova master: [DNM] Testing links to inline comments, ignore  https://review.opendev.org/65888718:30
artomCrap, that was supposed to go to downstream gerrit :(18:30
sean-k-mooneyam it did18:32
sean-k-mooneyoh you posted it upstream then downstream18:32
*** dklyle has quit IRC18:34
artomI learn from my mistakes :D18:36
artomFor posterioriy, @a<line number> links to the "before"/left pane.18:37
*** igordc has joined #openstack-nova18:37
*** dpawlik has joined #openstack-nova19:03
*** slaweq has joined #openstack-nova19:17
*** JamesBenson has joined #openstack-nova19:27
*** alex_xu has quit IRC19:32
*** gmann_pto has quit IRC19:33
openstackgerritDaniel Speichert proposed openstack/nova-specs master: Directly download and upload images to RBD  https://review.opendev.org/65890319:37
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Add 'power-update' external event to listen to ironic  https://review.opendev.org/64561119:37
*** JamesBenson has quit IRC19:39
openstackgerritMatt Riedemann proposed openstack/nova master: Update usage in RT.drop_move_claim during confirm resize  https://review.opendev.org/64180619:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid  https://review.opendev.org/61401219:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method  https://review.opendev.org/61403619:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add Instance.hidden field  https://review.opendev.org/63112319:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add TargetDBSetupTask  https://review.opendev.org/62789219:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask  https://review.opendev.org/63158119:39
openstackgerritMatt Riedemann proposed openstack/nova master: Execute TargetDBSetupTask  https://review.opendev.org/63385319:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method  https://review.opendev.org/62131319:39
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add prep_snapshot_based_resize_at_dest compute method  https://review.opendev.org/63329319:39
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask  https://review.opendev.org/62789019:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method  https://review.opendev.org/63483219:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova.compute.utils.delete_image  https://review.opendev.org/63760519:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add PrepResizeAtSourceTask  https://review.opendev.org/62789119:39
openstackgerritMatt Riedemann proposed openstack/nova master: Refactor ComputeManager.remove_volume_connection  https://review.opendev.org/64218319:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add power_on kwarg to ComputeDriver.spawn() method  https://review.opendev.org/64259019:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add finish_snapshot_based_resize_at_dest compute method  https://review.opendev.org/63508019:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add FinishResizeAtDestTask  https://review.opendev.org/63564619:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field  https://review.opendev.org/61403519:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add CrossCellWeigher  https://review.opendev.org/61435319:40
openstackgerritMatt Riedemann proposed openstack/nova master: Support cross-cell moves in external_instance_event  https://review.opendev.org/65847819:40
openstackgerritMatt Riedemann proposed openstack/nova master: Add cross-cell resize policy rule and enable in API  https://review.opendev.org/63826919:40
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Enable cross-cell resize in the nova-multi-cell job  https://review.opendev.org/65665619:40
openstackgerritMatt Riedemann proposed openstack/nova master: Robustify attachment tracking in CinderFixtureNewAttachFlow  https://review.opendev.org/65890419:40
mriedemdansmith: i've hit a not-so-fun snag in pre-validating the target host wrt attached volumes in the cross-cell resize stuff and i'd like to brain dump19:41
dansmithokay19:41
mriedemso we want to try and connect the volume on the dest host to see if it will work before starting the whole process,19:42
mriedemto do that, i need to update the volume attachment with the dest host connector to get the connection_info dict that the virt driver uses,19:42
mriedemthe problem is, at the point i'm doing that, the source host volume attachment still exists and cinder pukes saying you essentially can't have more than one connected attachment for the same volume at the same time19:42
mriedemwhich makes sense i guess, and is not something we hit in a normal move because we delete the source host attachment before finishing on the dest19:43
mriedemgranted normal resize could fail the same thing today - if you can't connect the volume on the dest after we've transferred the disks and stuff19:44
dansmithcross-cell is actually easier in this case because you have two separate instances and attachments on the nova side19:44
dansmithnormal resize would need to stash that in case it needs to revert right? or maybe just ask cinder again.. tbh, I dunno how that attachment stuff works really :)19:44
mriedemattachments in cinder are essentially ephemeral, we can create and delete them as needed, or that's the idea anyway19:45
mriedemso if i want to make this work,19:45
*** macza has quit IRC19:45
openstackgerritMerged openstack/python-novaclient master: Microversion 2.73: Support adding the reason behind a server lock  https://review.opendev.org/64865919:46
mriedemi'd need to do something like delete the source host attachment, update the attachment with the dest host connector so we can test the connection on the dest host,19:46
*** macza has joined #openstack-nova19:46
mriedemand then if something fails and we're rolling back, i'd have to create a new attachment and complete it for the source host (using the source host connector, which we don't stash away)19:46
dansmithwhich entity is going to give you the go/no-go on the new attachment? cinder? brick? libvirt?19:46
mriedemso not impossible, but kinda gross19:46
mriedemupdating the attachment with the host connector is what creates the exports in the storage backend19:47
mriedemand gives us back the connection_info dict19:47
dansmithokay, so, cinder yeah?19:48
*** BjoernT has quit IRC19:48
mriedemi'm honestly not sure if that's enough or not, i.e. if you still need the low-level connection on the host for libvirt/brick19:48
mriedemwe maybe don't need that - we don't try to plug vifs on the dest host during pre-validation19:48
dansmithcinder might tell you no, or it might not know and say "sure" but then the actual attach fails I guess19:48
mriedemyeah i'm not sure, that's why i was trying to do it all19:49
mriedemeither way i'm not disconnecting the guest / volume from the source host until we've pre-validated the dest19:49
mriedemand start doing the power off / snapshot stuff19:50
dansmithhmm19:50
dansmithin what case is cinder going to tell you no to the attempted attachment change?19:50
dansmithI mean... why do you think you need to do this special for cross-cell?19:51
mriedemi was thinking in case we hit a scenario where the target cell is using a different storage backend from the source cell19:51
mriedemnot sure how realistic that is though19:52
dansmithbut I mean,19:52
dansmithhow is that different from single-cell resize?19:52
sean-k-mooneymriedem: in an edge deployment its likely not that uncommon that they would have different backends19:52
mriedemit's not really, like i said before we could hit this same thing during normal resize and we really don't try to pre-validate that (in prep_resize) or rollback from that kind of failure19:52
dansmithyeah, I'm not sure what the backend difference is that you're talking about19:53
dansmithin this case, the volume is in cinder, on a backend, but migrating won't change what *cinder* backend it's on, regardless of where you move it19:53
sean-k-mooneydansmith: that is ture but that may mean the backen is not reachale from the new host. although if we are using sharing resouce providres to model this in placement it shoudl be reachable19:54
sean-k-mooneyassuming you have set up the aggreate correctly19:55
mriedemwe are not using sharing providers for any of this, or anything19:55
dansmithsean-k-mooney: right, but that's what I'm saying has nothing to do with cross-cell migration19:55
sean-k-mooneydansmith: sure19:55
sean-k-mooneyits not a special case that only comes up with cross cell19:56
mriedemso if this is really not any different from normal resize then i'm probably bending over backwards for no good reason19:57
mriedembut yeah i was thinking kind of like what sean said, the target host can't connect to the storage for the volume19:57
dansmithmriedem: yep, but that's just a thing that can happen any time,19:58
dansmithso if we're going to do something different, implement this test for regular and cross-cell, and assume they're the same until then, IMHO19:58
tssuryaefried: heh was just about to reply to that thread :)19:58
mriedemtssurya: since you're around, does cern use the same cinder for all of its cells? or most anyway?19:59
efriedplease do tssurya. I'm sure you can provide better insight.19:59
mriedems/cinder/volume backend/19:59
dansmithmriedem: do you mean the same cinder, or the same backend?19:59
sean-k-mooneydansmith: mriedem for normal resize do we pass a member  of parmater to ensure we select a host that is attache to the same sharign resouce provider19:59
dansmithyeah :)19:59
dansmiththe latter, I'm sure not, but that's what cross-az-attach is for19:59
tssuryamriedem: cinder for specific cells as far as I know19:59
tssuryabut yes same cinder19:59
mriedemi only know of 2 users of cross_az_attach=False (NeCTAR and Target)20:00
tssuryaefried: yea what they are saying is true though, just making resource_provider_association_refresh to large value doesn't still solve it for ironic20:00
tssuryawe have been having the same problem and mostly might end up adding more computes20:01
tssuryathe periodic task locks really long enough (sometimes ~20mins) and doesn't allow boot request to walk past it and we get stuff stuck in scheduling20:01
dansmithmriedem: you mean only two that speak up often or what? we definitely have users of it and are prescribing it for all edge deploys with cinder of course20:02
tssuryaso its interesting to see if for them setting update_resources_interval stays fine or not (in terms of things going out of sync)20:02
mriedemright, i only saw target people mention it in a talk once20:03
*** BjoernT has joined #openstack-nova20:03
mriedemwell i've commented out the trouble code for now and will have a gate run with a volume-backed cross-cell resize so i'll see if i run into any other bumps along the way; if the only issues with volume-backed cross-cell resize is this pre-validation thing, i'll probably just drop it.20:05
*** mlavalle has quit IRC20:07
efriedtssurya: oh, okay, I thought I might be on the wrong track with refresh interval20:09
efriedplease do correct me on the thred.20:09
efriedthread20:09
tssuryayep no problem20:10
*** maciejjozefczyk has quit IRC20:14
*** panda has quit IRC20:19
*** mlavalle has joined #openstack-nova20:19
*** slaweq has quit IRC20:19
*** panda has joined #openstack-nova20:21
*** partlycloudy has quit IRC20:27
openstackgerritEric Fried proposed openstack/nova master: Remove fake_libvirt_utils from connection tests.  https://review.opendev.org/64255720:27
openstackgerritEric Fried proposed openstack/nova master: Remove fake_libvirt_utils from snapshot tests.  https://review.opendev.org/64255820:27
openstackgerritEric Fried proposed openstack/nova master: Remove fake_libvirt_utils from virt driver tests.  https://review.opendev.org/64389420:27
openstackgerritEric Fried proposed openstack/nova master: Remove fake_libvirt_utils from libvirt imagebackend tests.  https://review.opendev.org/64389520:27
openstackgerritEric Fried proposed openstack/nova master: Remove remaining vestiges of fake_libvirt_utils from unit tests.  https://review.opendev.org/64389620:27
openstackgerritEric Fried proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing.  https://review.opendev.org/64479320:27
*** itlinux has quit IRC20:32
*** dpawlik has quit IRC20:35
*** tssurya has quit IRC20:38
*** pcaruana|afk| has quit IRC20:46
*** xek has quit IRC20:47
*** ttsiouts has joined #openstack-nova20:56
*** derekh has joined #openstack-nova20:59
*** artom has quit IRC21:07
sean-k-mooneymriedem: so regarding https://review.opendev.org/#/c/653506/ https://review.opendev.org/#/c/653506/4/nova/network/neutronv2/api.py@3358 is really the only part i would proably change but i think  the updated code is more or less correct.21:08
*** BjoernT_ has joined #openstack-nova21:09
sean-k-mooneythe pci requst update code for the non migration flow i belive is correct i just got confused by the comment21:09
*** openstackgerrit has quit IRC21:09
*** BjoernT has quit IRC21:10
*** slaweq has joined #openstack-nova21:11
mriedemganso: figured it out21:13
*** dklyle has joined #openstack-nova21:13
gansoyay \o/ mriedem: what was the problem?21:13
mriedemtight coupling in _delete_allocation_after_move which was dropped in stein,21:14
mriedemdetails are in https://review.opendev.org/#/c/658834/1/nova/tests/functional/test_servers.py21:14
mriedemi think i've got a fix21:15
*** slaweq has quit IRC21:15
mriedemyour persistence has paid off21:15
mriedemganso: so if you want, i can cleanup your rocky-only test with the proper context and such in the commit message, make the test assert the bug, and then stack the fix on top21:16
mriedemand also explain why it's not a problem in stein21:17
*** ttsiouts has quit IRC21:18
gansomriedem: hmmm would the fix be a) or b) ? I am not sure I understand which case (a) does not cover21:18
gansomriedem: as far as I understand, we would always have a migration-based allocation if confirm_migration ever fails21:20
mriedemthe fix i'm going with is (b)21:22
mriedemwhich is safe for rocky and queens21:22
mriedemmigration-based allocation during resize means that conductor moves the source node allocations to the migration record, which was new in queens and only happens iff the computes in the migration are new enough to handle that logic21:22
mriedemwhich might not be the case if you're doing a rolling upgrade in queens21:22
mriedemso (b) is safer i guess for backports21:23
mriedemit's also how that code is meant to work assuming you hit _delete_allocation_after_move without driver.confirm_migration failing21:23
*** dklyle has quit IRC21:24
mriedemso i guess we just need to figure out what to do about your recreate test21:25
gansomriedem: hmm I understand now21:26
mriedemi think we could land your test in master, backport to stein, and backport to rocky but in rocky we lay the rocky-only fix on it (and then backport that to queens as well), or we change the test patch in rocky to assert it fails until we have a rocky-only fix on top21:27
gansomriedem: so, about the test in master, I am not sure it is 100% the same. If it is the same, we would need to make a lot of changes to backport it to rocky21:27
mriedemyou already made those changes right? and it's not the same as the point i pointed out21:28
mriedemit's close, but in the existing test in master the PUT /allocations call to placement is stubbed to fail with a 40921:28
mriedemso not really what we'd want for asserting this fix in rocky/queens21:28
mriedem*not the same as the test i pointed out21:28
gansomriedem: oh sorry, just saw your response! great we can merge it then =D21:28
mriedemyeah given https://review.opendev.org/#/c/652153/ hasn't merged in queens, we can hold that up until your recreate test is ready in queens21:30
gansoyes21:30
gansomriedem: ok sounds like a good plan, I can improve the rocky patch once the stein backport lands, so it makes sense21:31
mriedemack ok21:31
mriedemdansmith: can you hit this easy test-only patch? https://review.opendev.org/#/c/657870/21:31
mriedemactually hold up21:32
* dansmith holds21:32
mriedemganso: one thing in https://review.opendev.org/#/c/657870/ - you can remove that self.failed_host thing right?21:33
mriedemit's not used21:33
mriedemmust have been copy/paste21:33
gansomriedem: yea was copy paste21:33
mriedemok remove that and i'm +221:33
gansomriedem: will remove and push an update21:33
gansomriedem: great =D21:33
gansomriedem: doing that now21:33
mriedemsean-k-mooney: so on https://review.opendev.org/#/c/653506/4/nova/network/neutronv2/api.py@3358 - what do you want to do instead? just remove the try/except and let the PortBindingActivationFailed raise up? if so i'm cool with that.21:34
sean-k-mooneymriedem: i was just responding. either that or as i previous said in stead of just doing the update delete the inactive port bidning then leave it fall back to updating the souce binding21:35
*** openstackgerrit has joined #openstack-nova21:36
openstackgerritRodrigo Barbieri proposed openstack/nova master: Add functional confirm_migration_error test  https://review.opendev.org/65787021:36
*** bbowen has quit IRC21:36
mriedemsean-k-mooney: just letting that raise up is easier21:36
mriedemthis is all workaround for weird edge casing anyway21:36
sean-k-mooneyya that is ture.21:36
mriedemtrue even21:37
sean-k-mooney:)21:37
mriedemi'll just do that then21:37
mriedemdansmith: ok i'm +2 on https://review.opendev.org/#/c/657870/21:37
sean-k-mooneycool in which case the last test can just be an assertRaises21:38
mriedemdansmith: tl;dr i had fixed an allocation leak in master and stein but ganso found it it didn't really fix the bug before stein b/c of some code refactor stuff that was missed in the backport21:38
*** mchlumsky has quit IRC21:38
dansmithokay21:38
mriedemthe test is used to recreate the bug in rocky and queens https://review.opendev.org/#/q/I9d6478f492351b58aa87b8f56e907ee633d6d1c621:38
*** ttsiouts has joined #openstack-nova21:38
*** whoami-rajat has quit IRC21:41
*** ttsiouts has quit IRC21:42
openstackgerritMatt Riedemann proposed openstack/nova master: Handle inactive port binding in _update_port_binding_for_instance  https://review.opendev.org/65350621:45
*** zhongjun2_ has quit IRC21:45
*** slaweq has joined #openstack-nova21:46
*** slaweq has quit IRC21:51
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: Add functional confirm_migration_error test  https://review.opendev.org/65892921:52
mriedemganso: there is the stein backport ^21:53
*** mriedem is now known as mriedem_afk21:53
gansomriedem_afk: cool, ok! I will improve the text message of the other ones21:54
aspiersis it just me or is https://etherpad.openstack.org/p/nova-ptg-train borked? is that why there was a https://etherpad.openstack.org/p/nova-ptg-train-2 ? but that seems borked too22:00
openstackgerritRodrigo Barbieri proposed openstack/nova stable/rocky: Add functional confirm_migration_error test  https://review.opendev.org/65883422:03
*** bbowen has joined #openstack-nova22:03
sean-k-mooneyaspiers: -4 was working last week22:07
sean-k-mooneylooks like that is broken too. efried has an offline copy. i have one on my personal laptop too22:09
aspierssean-k-mooney: yikes, etherpad zombies everywhere22:09
sean-k-mooneywe proably should jsut email it to the mailing list22:09
aspiersdid it just get too big for etherpad to handle?22:09
efriedI made -5 last week22:09
aspiersdoes the infra team know about this?22:09
sean-k-mooneyefried: -5 is not loading either22:10
efriedaspiers: I've seen way bigger etherpads that have survived, so I doubt it's the size.22:10
aspiersyeah, -5 not loading for me either22:10
sean-k-mooneythere is proably some content in it that it does not like22:10
efriedurm, looks like I might have killed my open window to -522:11
aspiersI've just notified infra22:11
efriedwas there anything in particular you needed from there?22:11
sean-k-mooneyi have a copy i saved offline22:11
sean-k-mooneyi can email it to the list22:11
sean-k-mooneyif i was to guess at anything i woudl guess it was the non ascii chater that was added to the start of the etherpad22:12
*** partlycloudy has joined #openstack-nova22:13
*** yonglihe has quit IRC22:14
*** BjoernT_ has quit IRC22:17
sean-k-mooneyaspiers: i sent a mail to the list with subject " [nova][ptg] main etherpad backup"22:23
sean-k-mooneywhich is apparently held for moderator apporval becasue apprently its over 40KB...22:24
sean-k-mooneythis is 2019 right22:25
openstackgerritEric Fried proposed openstack/nova master: WIP: TC Vision Reflection  https://review.opendev.org/65893222:25
sean-k-mooneyefried: am i missing somthing or is the reflect missing form ^22:30
efriedsean-k-mooney: You mean did I forget to git-add22:30
efriedyes22:30
efriedyes I did.22:30
efriedthanks22:30
sean-k-mooneyah ok :)22:30
efrieddoesn't do me much good to "back up my work" and "save my place" if I don't upload the freakin document.22:30
openstackgerritEric Fried proposed openstack/nova master: WIP: TC Vision Reflection  https://review.opendev.org/65893222:31
efriedsean-k-mooney: ^22:31
sean-k-mooneydid this come up in the ptg by the way. i was in the fourm session but i never got around to bringin this up in the ptg22:31
efriedno, it's just something that's been on my to-do list for months and months22:31
efriedokay, since March22:32
sean-k-mooneyah ok, we discussed this a bit in the fourm session but since bauzas and i were the only novaish people in the room we did not really have a reason other then no one has done it yet for why nova had not done one when asked22:33
efriedyeah, I figured it was my responsibility, just hadn't gotten to it yet22:33
efried(dreading, dreading)22:33
sean-k-mooneyout of interest is it still technically possible to deploy nova without keystone.22:36
sean-k-mooneyi think our configs imply you coudl deploy the api with no auth but im pretty sure it totally would bork things if you tried22:37
efriedno idea. I expect I will commit a number of inaccuracies in my writeup due to my relatively brief history on the project. I expect it to be vetted thoroughly by Nova veterans such as yourself before it is considered mergeable.22:37
efriedgtg22:38
sean-k-mooneywell you can still fall back to noauth2 but i dont know how functional the deployment would be https://github.com/openstack/nova/blob/master/nova/conf/api.py#L24-L3622:38
sean-k-mooneyo/22:38
*** tkajinam has joined #openstack-nova22:56
*** rcernin has joined #openstack-nova23:04
*** rcernin has quit IRC23:07
*** rcernin has joined #openstack-nova23:07
*** macza has quit IRC23:10
*** macza has joined #openstack-nova23:11
*** macza has quit IRC23:16
*** dklyle has joined #openstack-nova23:18
*** alex_xu has joined #openstack-nova23:20
*** artom has joined #openstack-nova23:22
mriedem_afkdansmith: well with commenting out that volume attachment stuff i did get a clean run on a volume-backed cross-cell resize confirm tempest test http://logs.openstack.org/56/656656/10/check/nova-multi-cell/f1def72/testr_results.html.gz so i guess i'll just nix that code from the series23:30
dansmithsweet23:30
openstackgerritJohn Garbutt proposed openstack/nova master: WIP: Integrating with unified limits  https://review.opendev.org/61518023:30
*** derekh has quit IRC23:30
*** mriedem_afk has quit IRC23:31
*** mlavalle has quit IRC23:57
*** hemna has quit IRC23:59

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