| *** gbarros has quit IRC | 00:02 | |
| *** gbarros_ has quit IRC | 00:20 | |
| *** markvoelker has joined #openstack-nova | 00:24 | |
| *** sapd1_x has joined #openstack-nova | 00:25 | |
| *** tellesnobrega has quit IRC | 00:32 | |
| *** gyee has quit IRC | 00:52 | |
| *** mriedem has quit IRC | 01:00 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-next multinode and drop tempest-slow-py3 https://review.opendev.org/683988 | 01:00 |
|---|---|---|
| *** gbarros has joined #openstack-nova | 01:06 | |
| *** markvoelker has quit IRC | 01:08 | |
| *** sapd1_x has quit IRC | 01:09 | |
| *** yedongcan has joined #openstack-nova | 01:21 | |
| *** tkajinam has quit IRC | 01:24 | |
| *** tkajinam has joined #openstack-nova | 01:24 | |
| *** sapd1_x has joined #openstack-nova | 01:25 | |
| *** gbarros has quit IRC | 01:35 | |
| *** sapd1_x has quit IRC | 01:37 | |
| *** takashin has joined #openstack-nova | 01:47 | |
| *** hongbin has joined #openstack-nova | 01:49 | |
| *** mkrai has joined #openstack-nova | 02:06 | |
| *** factor has joined #openstack-nova | 02:07 | |
| *** markvoelker has joined #openstack-nova | 02:35 | |
| *** hongbin has quit IRC | 02:36 | |
| *** igordc has quit IRC | 02:39 | |
| *** markvoelker has quit IRC | 02:40 | |
| *** BjoernT_ has quit IRC | 02:42 | |
| *** BjoernT has joined #openstack-nova | 02:44 | |
| *** ricolin has joined #openstack-nova | 02:47 | |
| *** BjoernT has quit IRC | 02:47 | |
| *** dave-mccowan has quit IRC | 02:53 | |
| *** yedongcan has left #openstack-nova | 03:16 | |
| *** psachin has joined #openstack-nova | 03:31 | |
| openstackgerrit | Merged openstack/nova master: Handle legacy request spec dict in ComputeTaskManager._cold_migrate https://review.opendev.org/680762 | 03:55 |
| openstackgerrit | Merged openstack/nova master: Isolate request spec handling from _cold_migrate https://review.opendev.org/680763 | 03:55 |
| *** udesale has joined #openstack-nova | 04:07 | |
| *** boxiang has quit IRC | 04:16 | |
| *** zhubx has joined #openstack-nova | 04:17 | |
| *** ociuhandu has joined #openstack-nova | 04:30 | |
| *** ociuhandu has quit IRC | 04:35 | |
| *** artom has quit IRC | 04:41 | |
| *** belmoreira has joined #openstack-nova | 04:51 | |
| *** jaosorior has joined #openstack-nova | 04:56 | |
| *** TxGirlGeek has quit IRC | 04:57 | |
| *** TxGirlGeek has joined #openstack-nova | 04:57 | |
| *** cfriesen has quit IRC | 04:58 | |
| *** ratailor has joined #openstack-nova | 05:01 | |
| *** TxGirlGeek has quit IRC | 05:02 | |
| *** mkrai has quit IRC | 05:02 | |
| *** mkrai has joined #openstack-nova | 05:03 | |
| *** TxGirlGeek has joined #openstack-nova | 05:03 | |
| *** belmoreira has quit IRC | 05:05 | |
| *** dpawlik has joined #openstack-nova | 05:13 | |
| *** TxGirlGeek has quit IRC | 05:15 | |
| *** mkrai has quit IRC | 05:29 | |
| *** mkrai_ has joined #openstack-nova | 05:30 | |
| *** zbr|ruck has quit IRC | 05:43 | |
| *** zbr has joined #openstack-nova | 05:43 | |
| *** brault has joined #openstack-nova | 05:46 | |
| *** yedongcan has joined #openstack-nova | 05:51 | |
| *** takashin has left #openstack-nova | 06:00 | |
| *** luksky has joined #openstack-nova | 06:21 | |
| *** belmoreira has joined #openstack-nova | 06:35 | |
| *** mkrai_ has quit IRC | 06:35 | |
| *** mkrai__ has joined #openstack-nova | 06:35 | |
| *** pcaruana has joined #openstack-nova | 06:42 | |
| *** eharney has joined #openstack-nova | 06:46 | |
| *** rpittau|afk is now known as rpittau | 06:47 | |
| *** markvoelker has joined #openstack-nova | 06:47 | |
| *** markvoelker has quit IRC | 06:52 | |
| *** slaweq has joined #openstack-nova | 06:55 | |
| *** bhagyashris has joined #openstack-nova | 07:00 | |
| *** tesseract has joined #openstack-nova | 07:01 | |
| *** tellesnobrega has joined #openstack-nova | 07:03 | |
| *** xek has joined #openstack-nova | 07:14 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:19 | |
| *** brault has quit IRC | 07:21 | |
| *** damien_r has joined #openstack-nova | 07:23 | |
| *** luksky has quit IRC | 07:27 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/nova master: Ensure volume filesystems are mounted on resume https://review.opendev.org/684290 | 07:28 |
| openstackgerrit | Luyao Zhong proposed openstack/nova master: doc: attaching virtual persistent memory to guests https://review.opendev.org/680300 | 07:29 |
| *** zhubx has quit IRC | 07:38 | |
| *** zhubx has joined #openstack-nova | 07:38 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Functional reproduction for bug 1845291 https://review.opendev.org/684545 | 07:42 |
| openstack | bug 1845291 in OpenStack Compute (nova) "migration is not recheduled if the server originally booted with --availability-zone <az>:<host>" [Undecided,New] https://launchpad.net/bugs/1845291 - Assigned to Balazs Gibizer (balazs-gibizer) | 07:42 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reset forced_destination before migration at a proper time https://review.opendev.org/684546 | 07:42 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242 | 07:42 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243 | 07:42 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | 07:42 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245 | 07:42 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | 07:42 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999 | 07:42 |
| *** yaawang has quit IRC | 07:47 | |
| *** ccamacho has joined #openstack-nova | 07:47 | |
| *** yaawang has joined #openstack-nova | 07:49 | |
| *** ralonsoh has joined #openstack-nova | 07:51 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move HostNameWeigher to a common fixture https://review.opendev.org/683874 | 07:59 |
| *** ivve has joined #openstack-nova | 08:01 | |
| *** rpittau is now known as rpittau|bbl | 08:01 | |
| *** stephenfin has left #openstack-nova | 08:20 | |
| *** jangutter has joined #openstack-nova | 08:20 | |
| *** cdent has joined #openstack-nova | 08:20 | |
| *** luksky has joined #openstack-nova | 08:23 | |
| *** mkrai__ has quit IRC | 08:29 | |
| *** jaosorior has quit IRC | 08:32 | |
| *** priteau has joined #openstack-nova | 08:34 | |
| *** brault has joined #openstack-nova | 08:39 | |
| *** derekh has joined #openstack-nova | 08:39 | |
| *** zbr is now known as zbr|ruck | 08:40 | |
| *** brault has quit IRC | 08:43 | |
| *** markvoelker has joined #openstack-nova | 08:48 | |
| *** tkajinam has quit IRC | 08:52 | |
| *** markvoelker has quit IRC | 08:53 | |
| *** markvoelker has joined #openstack-nova | 08:56 | |
| *** markvoelker has quit IRC | 09:00 | |
| *** bhagyashris has quit IRC | 09:01 | |
| *** ccamacho has quit IRC | 09:03 | |
| *** tellesnobrega has quit IRC | 09:03 | |
| *** tellesnobrega has joined #openstack-nova | 09:04 | |
| *** jaosorior has joined #openstack-nova | 09:10 | |
| *** stephenfin has joined #openstack-nova | 09:10 | |
| *** ratailor has quit IRC | 09:10 | |
| *** ratailor has joined #openstack-nova | 09:11 | |
| *** dtantsur has joined #openstack-nova | 09:12 | |
| *** ttsiouts has joined #openstack-nova | 09:12 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Delete instance_id_mappings record in instance_destroy https://review.opendev.org/684658 | 09:22 |
| *** brault has joined #openstack-nova | 09:24 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Correct link to placement upgrade notes https://review.opendev.org/684661 | 09:28 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Mask the token used to allow access to consoles https://review.opendev.org/220622 | 09:30 |
| *** dpawlik has quit IRC | 09:33 | |
| *** ociuhandu has joined #openstack-nova | 09:35 | |
| *** dpawlik has joined #openstack-nova | 09:36 | |
| *** ociuhandu has quit IRC | 09:36 | |
| *** mdbooth has quit IRC | 09:37 | |
| *** ociuhandu has joined #openstack-nova | 09:38 | |
| *** ociuhandu has quit IRC | 09:39 | |
| *** mdbooth has joined #openstack-nova | 09:40 | |
| *** brault has quit IRC | 09:43 | |
| *** bbowen has quit IRC | 09:46 | |
| *** bbowen has joined #openstack-nova | 09:46 | |
| *** avolkov has joined #openstack-nova | 09:48 | |
| *** brault has joined #openstack-nova | 09:51 | |
| *** bhagyashris has joined #openstack-nova | 09:52 | |
| *** luksky has quit IRC | 09:54 | |
| *** rpittau|bbl is now known as rpittau | 09:57 | |
| *** ociuhandu has joined #openstack-nova | 09:59 | |
| *** ociuhandu has quit IRC | 10:04 | |
| *** brault has quit IRC | 10:17 | |
| *** BjoernT has joined #openstack-nova | 10:23 | |
| *** pcaruana has quit IRC | 10:23 | |
| *** rcernin has quit IRC | 10:23 | |
| *** ratailor has quit IRC | 10:23 | |
| *** BjoernT_ has joined #openstack-nova | 10:28 | |
| openstackgerrit | Merged openstack/nova master: Remove upgrade specific info from user facing exception text https://review.opendev.org/684253 | 10:29 |
| *** BjoernT has quit IRC | 10:31 | |
| *** bhagyashris has quit IRC | 10:33 | |
| *** luksky has joined #openstack-nova | 10:33 | |
| *** pcaruana has joined #openstack-nova | 10:36 | |
| openstackgerrit | Merged openstack/python-novaclient master: Add a check for --config-drive option on nova boot https://review.opendev.org/653683 | 10:37 |
| *** tellesnobrega has quit IRC | 10:37 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: single pass instance info fetch in host manager https://review.opendev.org/623558 | 10:37 |
| *** cdent has quit IRC | 10:41 | |
| *** ratailor has joined #openstack-nova | 10:46 | |
| *** tbachman has quit IRC | 10:47 | |
| *** ratailor has quit IRC | 10:47 | |
| *** ratailor has joined #openstack-nova | 10:51 | |
| *** artom has joined #openstack-nova | 10:53 | |
| *** dpawlik has quit IRC | 10:56 | |
| *** markvoelker has joined #openstack-nova | 10:57 | |
| *** ociuhandu has joined #openstack-nova | 10:58 | |
| *** BjoernT_ has quit IRC | 10:59 | |
| *** ttsiouts has quit IRC | 11:01 | |
| *** markvoelker has quit IRC | 11:01 | |
| *** brault has joined #openstack-nova | 11:09 | |
| *** brault has quit IRC | 11:13 | |
| *** udesale has quit IRC | 11:15 | |
| *** ttsiouts has joined #openstack-nova | 11:22 | |
| *** ccamacho has joined #openstack-nova | 11:25 | |
| *** ccamacho has quit IRC | 11:25 | |
| *** ccamacho has joined #openstack-nova | 11:25 | |
| *** ratailor has quit IRC | 11:27 | |
| *** tellesnobrega has joined #openstack-nova | 11:27 | |
| *** dpawlik has joined #openstack-nova | 11:29 | |
| openstackgerrit | zhurong proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999 | 11:30 |
| gibi | stephenfin, alex_xu: do you have ideas how we will do the nova project update in Shanghai? | 11:32 |
| *** cdent has joined #openstack-nova | 11:37 | |
| *** xek_ has joined #openstack-nova | 11:38 | |
| *** xek has quit IRC | 11:39 | |
| *** markvoelker has joined #openstack-nova | 11:42 | |
| cdent | stephenfin: this is a nova docs change driven by the structural changes in the placement docs: https://review.opendev.org/#/c/684661/ | 11:43 |
| *** brault has joined #openstack-nova | 11:43 | |
| *** brault has quit IRC | 11:50 | |
| *** belmoreira has quit IRC | 11:54 | |
| *** belmoreira has joined #openstack-nova | 11:58 | |
| *** yaawang has quit IRC | 12:03 | |
| stephenfin | gibi: Haven't a clue. I was hoping efried would just give us slides and we'd use those \o/ | 12:06 |
| * stephenfin likes creating busywork for efried | 12:06 | |
| *** xek_ has quit IRC | 12:06 | |
| gibi | stephenfin: :) | 12:07 |
| *** tbachman has joined #openstack-nova | 12:12 | |
| *** tbachman_ has joined #openstack-nova | 12:21 | |
| *** tbachman has quit IRC | 12:21 | |
| *** tbachman_ is now known as tbachman | 12:21 | |
| *** dave-mccowan has joined #openstack-nova | 12:22 | |
| *** dave-mccowan has quit IRC | 12:26 | |
| donnyd | did we ever get that grenade thing sorted out? | 12:26 |
| *** mdbooth has quit IRC | 12:28 | |
| *** dave-mccowan has joined #openstack-nova | 12:28 | |
| *** tellesnobrega has quit IRC | 12:31 | |
| *** tellesnobrega has joined #openstack-nova | 12:32 | |
| *** cdent has quit IRC | 12:36 | |
| *** dave-mccowan has quit IRC | 12:42 | |
| *** dave-mccowan has joined #openstack-nova | 12:43 | |
| *** mriedem has joined #openstack-nova | 12:47 | |
| *** nweinber_ has joined #openstack-nova | 12:49 | |
| *** nicholas has joined #openstack-nova | 12:55 | |
| mriedem | stephenfin: you might want to propose wording for the pcpu stuff you want to see in https://review.opendev.org/#/c/683327/ | 12:55 |
| *** BjoernT_ has joined #openstack-nova | 12:56 | |
| mriedem | i guess just something like, "It's now possible to place pinned and unpinned instances on the same host when using the libvirt driver."? | 12:56 |
| stephenfin | basically, yeah. tbh, I was hoping someone less in the trenches would be able to provide exact wording to make sure I was assuming too much | 12:56 |
| stephenfin | if that makes sense | 12:57 |
| *** tellesnobrega_ has joined #openstack-nova | 12:57 | |
| mriedem | alex_xu: can you review the vpmem docs today? https://review.opendev.org/#/c/680300/ - rc1 is tomorrow | 12:57 |
| mriedem | pinned and unpinned + same host + libvirt seems generic enough to me | 12:58 |
| mriedem | i'd rebase the prelude on the actual admin docs patch and just link to that for details | 12:58 |
| mriedem | i can make that change | 12:58 |
| *** tellesnobrega has quit IRC | 13:00 | |
| *** BjoernT_ has quit IRC | 13:00 | |
| *** tellesnobrega_ is now known as tellesnobrega | 13:00 | |
| *** BjoernT has joined #openstack-nova | 13:01 | |
| *** dpawlik has quit IRC | 13:01 | |
| *** psachin has quit IRC | 13:04 | |
| *** ociuhandu has quit IRC | 13:04 | |
| *** damien_r has quit IRC | 13:04 | |
| mriedem | stephenfin: who else reviewed the pcpu series that can review those docs? | 13:06 |
| mriedem | bauzas: ? | 13:06 |
| stephenfin | bauzas, alex_xu or efried should be able to handle it, I suspect | 13:06 |
| *** ociuhandu has joined #openstack-nova | 13:07 | |
| *** ociuhandu has quit IRC | 13:11 | |
| *** derekh has quit IRC | 13:12 | |
| *** belmoreira has quit IRC | 13:12 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a prelude for the Train release https://review.opendev.org/683327 | 13:12 |
| mriedem | is it a french holiday today? | 13:13 |
| mriedem | the feast of maximum strikage | 13:13 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reset forced_destination before migration at a proper time https://review.opendev.org/684546 | 13:13 |
| bauzas | mriedem: I can do | 13:14 |
| gibi | mriedem: fixed your comments in ^^ | 13:14 |
| mriedem | sean-k-mooney: stephenfin: how much of sean's comments are going to be addressed in https://review.opendev.org/#/c/683437/ which i thought was going to be more of a refactor to ease where the new pcpu docs go but sounds like it's adding new content as well | 13:15 |
| mriedem | new content is fine, but not really when it's in a critical path to merging changes we need for rc1 | 13:15 |
| stephenfin | I can fix them the first two now | 13:16 |
| stephenfin | *the last two | 13:16 |
| stephenfin | the first one (documenting RT) is much more work and not needed here | 13:17 |
| mriedem | ok i guess leave a .. todo:: for whatever needs a follow up | 13:17 |
| *** dpawlik has joined #openstack-nova | 13:18 | |
| stephenfin | there's already one but in another doc (the flavor extra spec one) | 13:18 |
| stephenfin | so I think we're covered | 13:18 |
| stephenfin | I'll comment as much | 13:18 |
| stephenfin | I purposefully wanted to avoid getting into the weeks on the emulator threads stuff. Should have known sean-k-mooney would call me on it, heh | 13:18 |
| * stephenfin should have fought Sahid even more on that overloading of 'share' policy. It's as dumb an idea now as it was then | 13:22 | |
| *** udesale has joined #openstack-nova | 13:22 | |
| *** belmoreira has joined #openstack-nova | 13:23 | |
| *** brault has joined #openstack-nova | 13:24 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/nova master: Ensure volume filesystems are mounted on resume https://review.opendev.org/684290 | 13:25 |
| openstackgerrit | Ivaylo Mitev proposed openstack/nova master: VMware VMDK detach: get adapter type from instance VM https://review.opendev.org/653738 | 13:25 |
| *** yaawang has joined #openstack-nova | 13:26 | |
| *** dave-mccowan has quit IRC | 13:26 | |
| *** brault has quit IRC | 13:28 | |
| mriedem | gibi: on https://review.opendev.org/#/c/684545/ i think we should do that outside of the ServerMovingTests class, | 13:28 |
| mriedem | and make it a standalone test class under regressions - easier to backport that way and we won't run duplicates of it in extended classes | 13:29 |
| gibi | mriedem: ack. I will move the test under regressions | 13:29 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Clarify everything CPU pinning https://review.opendev.org/683437 | 13:30 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Update CPU topologies guide to reflect the new PCPU world https://review.opendev.org/683485 | 13:30 |
| stephenfin | mriedem: incoming | 13:30 |
| stephenfin | bauzas, alex_xu, sean-k-mooney: https://review.opendev.org/683437 https://review.opendev.org/683485 | 13:31 |
| *** ociuhandu has joined #openstack-nova | 13:31 | |
| bauzas | stephenfin: I just provided comments for PS3 :) | 13:31 |
| bauzas | but honestly, it's just a nirt | 13:31 |
| *** liuyulong has joined #openstack-nova | 13:31 | |
| stephenfin | bauzas: I replied | 13:31 |
| bauzas | stephenfin: anyway, it's docs | 13:32 |
| stephenfin | (I already have a glossary at the top of the document explaining the terms) | 13:32 |
| bauzas | meaning that I'm fine with +2ing the change and just saying that we could provide better explanations in some follow-up | 13:32 |
| bauzas | stephenfin: eh ok | 13:32 |
| bauzas | FWIW, my personal opinion is that we should just accept any documentation providing more explanations and just asking for follow-ups if some nits are there | 13:33 |
| bauzas | unless a doc is wrong, honestly no need for -1ing it | 13:33 |
| bauzas | because it's just better to have a documentation than none | 13:34 |
| *** xek has joined #openstack-nova | 13:34 | |
| *** ociuhandu has quit IRC | 13:34 | |
| *** ociuhandu has joined #openstack-nova | 13:35 | |
| mriedem | kashyap: fyi if you or any other libguestfs minded people are about https://bugs.launchpad.net/nova/+bug/1845284 | 13:35 |
| openstack | Launchpad bug 1845284 in OpenStack Compute (nova) "Config CONF.libvirt.inject_partition=-1, build instance with metadata will remain some images in /var/tmp/.guestfs-0/appliance.d" [Low,New] | 13:35 |
| kashyap | mriedem: Hiya | 13:36 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document how to revert, confirm a cold migration https://review.opendev.org/664396 | 13:36 |
| gibi | bauzas: hi! is there a way to fake some VGPUs in devstack without having real hardware? | 13:36 |
| * kashyap clicks | 13:36 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Update resize doc https://review.opendev.org/665412 | 13:36 |
| bauzas | gibi: hah, good question | 13:36 |
| stephenfin | bauzas: In that case, I have two lovely docs here for you ;) https://review.opendev.org/#/c/670125/ https://review.opendev.org/#/c/667165/ | 13:37 |
| stephenfin | assuming you know anything about quotas | 13:37 |
| *** yaawang has quit IRC | 13:37 | |
| *** ociuhandu has quit IRC | 13:37 | |
| stephenfin | bauzas: and if not, https://review.opendev.org/#/c/667133/ is definitely your wheelhouse | 13:37 |
| *** gbarros has joined #openstack-nova | 13:37 | |
| *** ociuhandu has joined #openstack-nova | 13:37 | |
| bauzas | stephenfin: I still have one from you I remember to review https://review.opendev.org/#/c/667133/ | 13:38 |
| bauzas | hah, this | 13:38 |
| kashyap | mriedem: Right off the bat, the "No operating system ..." error from libguestfs means what it says on the tin - libguestfs couldn't find an OS in the disk - a bogus disk, corrupted, etc | 13:38 |
| stephenfin | That's the one :D | 13:38 |
| mriedem | lyarwood: melwitt: mnaser: stephenfin: weee ironic serial consoles are broken https://bugs.launchpad.net/nova/+bug/1845243 | 13:38 |
| openstack | Launchpad bug 1845243 in OpenStack Compute (nova) "Nested 'path' query param in console URL breaks serialproxy" [Undecided,New] | 13:38 |
| kashyap | mriedem: I'll add words to the bug | 13:38 |
| mriedem | kashyap: thanks | 13:38 |
| bauzas | gibi: I was thinking of using the fake libvirt driver (https://libvirt.org/drvtest.html) for telling that we have mdevs, but looks like we have problems with it | 13:39 |
| *** ociuhandu has quit IRC | 13:39 | |
| mriedem | stephenfin: i reviewed https://review.opendev.org/#/c/670125/ once, just forgot about it, will look again | 13:39 |
| stephenfin | mriedem: That's unfortunate. Should be an easy fix though. Lemme try | 13:39 |
| gibi | bauzas: thanks | 13:39 |
| bauzas | gibi: for example, I could pass my own XML to the test driver | 13:39 |
| kashyap | mriedem: Also, what they're asking is completely off: if I parse the reporter, they want the "images in the /var/tmp/ files to be "left forever" | 13:39 |
| *** ociuhandu has joined #openstack-nova | 13:40 | |
| kashyap | ... except that's by design; the hint is in the file path, they _are_ temporary images that _will_ go away. | 13:40 |
| mriedem | kashyap: i think they are saying the images are left and should be removed | 13:41 |
| mriedem | expected: "In the end of booting instance, can remove those images in /var/tmp/.guestfs-0/appliance.d" | 13:41 |
| mriedem | but yeah, if it's /tmp they should be gone on restart | 13:41 |
| kashyap | mriedem: I parsed it upside down; by design of my brain; ignore me | 13:41 |
| kashyap | But guestfs / libvirt / QEMU version is needed to reproduce. And you asked it there | 13:42 |
| *** nweinber_ has quit IRC | 13:44 | |
| *** damien_r has joined #openstack-nova | 13:48 | |
| kashyap | mriedem: So, checking w/ a libguestfs maintainer (Rich Jones) -- he says files in that /var/tmp are a cache and libguestfs _won't_ remove them | 13:49 |
| kashyap | It relies on external component to do it | 13:49 |
| *** ociuhandu has quit IRC | 13:49 | |
| kashyap | And Nova _shouldn't_ delete them either - that feels wrong. | 13:49 |
| * kashyap goes to add words to bug | 13:49 | |
| *** damien_r has quit IRC | 13:49 | |
| mriedem | kashyap: ok, so basically it's glance but at a lower level :) | 13:49 |
| mriedem | i seem to remember issues in ibm's internal CI where booting the first guest would take longer b/c libguestfs had to build the cache for the image | 13:50 |
| mriedem | something like that | 13:50 |
| kashyap | Yeap | 13:50 |
| kashyap | mriedem: Also there's a systemd tool (there are a gazillion of them) which removes temp files, while making a sandwich for you | 13:50 |
| * kashyap goes to look for the tool name | 13:50 | |
| kashyap | Ha! Found the sucker: | 13:51 |
| kashyap | "`systemd-tmpfiles` creates, deletes, and cleans up volatile and temporary files and directories, based on the configuration file format and location specified in tmpfiles.d(5)" | 13:51 |
| kashyap | So they should something like that | 13:51 |
| *** damien_r has joined #openstack-nova | 13:51 | |
| mriedem | kashyap: ok i guess just drop those details in the bug report and then mark it invalid | 13:52 |
| mriedem | thanks | 13:52 |
| kashyap | Also, unrelated funninness (is it a word?) I have to admit: "the feast of maximum strikage" :D | 13:52 |
| *** damien_r has quit IRC | 13:53 | |
| *** damien_r has joined #openstack-nova | 13:54 | |
| *** slaweq has quit IRC | 13:54 | |
| kashyap | mriedem: Done; words added, turned the state to invalid. | 13:54 |
| mriedem | TheJulia: do you know if it's possible to test ironic serial consoles in tempest? https://bugs.launchpad.net/nova/+bug/1845243 | 13:55 |
| openstack | Launchpad bug 1845243 in OpenStack Compute (nova) stein "Nested 'path' query param in console URL breaks serialproxy" [High,Confirmed] | 13:55 |
| mriedem | tempest has a test for novnc consoles but i'm not sure how possible that is with ironic serial consoles | 13:55 |
| *** slaweq has joined #openstack-nova | 13:57 | |
| *** panda is now known as panda|bbl | 13:59 | |
| *** yaawang has joined #openstack-nova | 13:59 | |
| mriedem | gibi: i feel like i should get some retroactive compensation from ericsson for getting the stable branch extended maintenance policy change enacted :) | 14:01 |
| mriedem | making all of your pike dreams come true | 14:02 |
| mriedem | i bet there is still someone at the ops meeting that wants to talk about LTS though... | 14:02 |
| gibi | mriedem: Thank you. (I can give you the email of my boss. but I guess that will not help much. ) | 14:02 |
| *** dpawlik has quit IRC | 14:02 | |
| mriedem | ha | 14:03 |
| gibi | and it would be so nice to have more people from ericsson contribution to the ext maintenance effort | 14:04 |
| gibi | and I continously ask for more people without success | 14:05 |
| *** xek has quit IRC | 14:05 | |
| *** brault has joined #openstack-nova | 14:06 | |
| *** xek has joined #openstack-nova | 14:07 | |
| *** yedongcan has left #openstack-nova | 14:12 | |
| *** cdent has joined #openstack-nova | 14:13 | |
| efried | stephenfin: You had a -1 on a previous PS of https://review.opendev.org/#/c/683783/ -- I think your feedback was addressed (repath was reverted) but would you please have another quick look and make sure there's nothing blocking remaining? | 14:13 |
| *** dpawlik has joined #openstack-nova | 14:14 | |
| bauzas | gibi: mriedem: honestly, not a lot of people work for 100% upstream you know | 14:14 |
| bauzas | so we have a lot of other stuff | 14:15 |
| efried | sean-k-mooney: I'll proxy your approval of https://review.opendev.org/#/c/683437/ if you could have another look please ("everything CPU pinning" doc) | 14:15 |
| *** slaweq has quit IRC | 14:15 | |
| bauzas | efried: like I said previously, unless a doc is having some wrong explanation, I should we should just accept it, and then providing some FUP | 14:15 |
| bauzas | (in case of comments) | 14:16 |
| gibi | bauzas: I would be happy to get an extra two half-people for stable/pike from my employee but... | 14:16 |
| *** slaweq has joined #openstack-nova | 14:16 | |
| *** ociuhandu has joined #openstack-nova | 14:16 | |
| efried | bauzas: I agree; I'm trying to delegate to someone else to confirm that there's nothing actually wrong in there. | 14:16 |
| bauzas | stephenfin: FWIW, finally looking at https://review.opendev.org/#/c/667133 :) | 14:18 |
| *** dpawlik has quit IRC | 14:18 | |
| *** ociuhandu has quit IRC | 14:20 | |
| *** mrch_ has quit IRC | 14:21 | |
| mriedem | bauzas: i thought most people were 100% upstream... | 14:21 |
| bauzas | yeah, me too honestly | 14:22 |
| mriedem | sorry, i'm being facetious | 14:22 |
| bauzas | and then I discovered the other way once I went a core | 14:22 |
| bauzas | because 'heh, can you do this please' | 14:22 |
| bauzas | or 'look at this, is this right?' | 14:22 |
| *** damien_r has quit IRC | 14:23 | |
| *** ociuhandu has joined #openstack-nova | 14:25 | |
| *** mlavalle has joined #openstack-nova | 14:25 | |
| *** liuyulong has quit IRC | 14:26 | |
| *** ociuhandu has quit IRC | 14:31 | |
| *** ociuhandu has joined #openstack-nova | 14:31 | |
| *** gbarros has quit IRC | 14:33 | |
| *** rpittau is now known as rpittau|afk | 14:34 | |
| mriedem | gibi: looks like the show version tempest test failures in https://review.opendev.org/#/c/684658/ are real (on pike) | 14:36 |
| mriedem | tempest doesn't test pike since it's in extended maintenance mode so it's likely something changed in tempest | 14:36 |
| *** xek has quit IRC | 14:36 | |
| mriedem | probably dropped support for cinder v1 or v2 or something | 14:36 |
| mriedem | this https://review.opendev.org/#/c/680790/ | 14:39 |
| mriedem | gmann: stable/pike is also broken since ^ | 14:39 |
| mriedem | so i guess we're going to have to run with a tagged tempest on stable/pike as well | 14:40 |
| openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Stop silently ignoring invalid 'nova boot --hint' options https://review.opendev.org/684762 | 14:41 |
| stephenfin | that novaclient unit tests are dumb and I hate them | 14:41 |
| gibi | mriedem: thanks | 14:42 |
| *** TxGirlGeek has joined #openstack-nova | 14:42 | |
| gibi | mriedem: I will have to get back to that | 14:42 |
| *** mrch_ has joined #openstack-nova | 14:43 | |
| mriedem | my guess is tempest is failing b/c the cinder v1 api was removed in queens https://review.opendev.org/#/c/499342/ | 14:43 |
| *** nweinber_ has joined #openstack-nova | 14:43 | |
| *** luksky has quit IRC | 14:46 | |
| gibi | I try to get elod look into this | 14:46 |
| *** tellesnobrega has quit IRC | 14:47 | |
| mriedem | gibi: just posted to the ML | 14:51 |
| *** TxGirlGeek has quit IRC | 14:53 | |
| gibi | mriedem: ack, thanks | 14:54 |
| *** TxGirlGeek has joined #openstack-nova | 14:56 | |
| *** brault has quit IRC | 14:57 | |
| mriedem | https://review.opendev.org/#/c/684769/ | 14:58 |
| *** brault has joined #openstack-nova | 14:59 | |
| *** brault has quit IRC | 15:00 | |
| *** dtantsur is now known as dtantsur|afk | 15:00 | |
| openstackgerrit | Martin Midolesov proposed openstack/nova master: Implementing graceful shutdown. https://review.opendev.org/666245 | 15:03 |
| *** belmoreira has quit IRC | 15:05 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Ignore warning from sqlalchemy-migrate https://review.opendev.org/684772 | 15:06 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Nuke OS_STDOUT_CAPTURE, OS_STDERR_CAPTURE https://review.opendev.org/684773 | 15:06 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Use common 'command' definition for unit tests https://review.opendev.org/684774 | 15:06 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Stop overriding the 'install_command' https://review.opendev.org/684775 | 15:06 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Delete instance_id_mappings record in instance_destroy https://review.opendev.org/684658 | 15:06 |
| stephenfin | mriedem: You'd asked for the first of those last week ^ | 15:06 |
| *** dpawlik has joined #openstack-nova | 15:07 | |
| mnaser | mriedem: ouch, i think it should be easy to fix .. i hope :X | 15:07 |
| *** BjoernT has quit IRC | 15:07 | |
| *** BjoernT has joined #openstack-nova | 15:08 | |
| *** cfriesen has joined #openstack-nova | 15:09 | |
| mriedem | stephenfin: so about that, | 15:10 |
| mriedem | i was trying to do that last week myself and found the warnings go away for py27 but not py36 runs | 15:10 |
| *** brault has joined #openstack-nova | 15:10 | |
| mriedem | stephenfin: comments in there, but the py removal is wrong, and we should split fixing that out since it's on stable branches | 15:12 |
| *** zhubx has quit IRC | 15:13 | |
| stephenfin | mriedem: Oh, so it is (called py). My eyes completely glazed over that \o/ | 15:13 |
| *** ttsiouts has quit IRC | 15:13 | |
| stephenfin | fix coming right up | 15:14 |
| *** boxiang has joined #openstack-nova | 15:14 | |
| *** eharney has quit IRC | 15:14 | |
| mriedem | stephenfin: also a question in the nuke patch after that | 15:14 |
| *** artom has quit IRC | 15:15 | |
| mriedem | track that warning ignore patch against bug 1813147 as well | 15:15 |
| openstack | bug 1813147 in OpenStack Compute (nova) "p35 jobs are failing with subunit.parser ... FAILED" [High,In progress] https://launchpad.net/bugs/1813147 - Assigned to Balazs Gibizer (balazs-gibizer) | 15:15 |
| mriedem | please | 15:15 |
| *** artom has joined #openstack-nova | 15:15 | |
| *** ttsiouts has joined #openstack-nova | 15:15 | |
| *** ttsiouts has quit IRC | 15:15 | |
| mriedem | dansmith: care to get https://review.opendev.org/#/c/684140/ so the cross_az_attach func test patch can be unblocked? | 15:18 |
| dansmith | oh sorry did I miss that yesterday? | 15:19 |
| *** ttsiouts has joined #openstack-nova | 15:20 | |
| *** ccamacho has quit IRC | 15:20 | |
| *** openstackgerrit has quit IRC | 15:21 | |
| gregwork | is there an equivalent of nova get-password in the openstack client | 15:21 |
| *** openstackgerrit has joined #openstack-nova | 15:22 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Ignore warning from sqlalchemy-migrate https://review.opendev.org/684772 | 15:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Use common 'command' definition for unit tests https://review.opendev.org/684774 | 15:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Stop overriding the 'install_command' https://review.opendev.org/684775 | 15:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Nuke OS_STDOUT_CAPTURE, OS_STDERR_CAPTURE https://review.opendev.org/684773 | 15:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Ignore sqla-migrate inspect.getargspec deprecation warnings on py36 https://review.opendev.org/684781 | 15:22 |
| *** sridharg has joined #openstack-nova | 15:23 | |
| *** ttsiouts has quit IRC | 15:24 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Handle legacy request spec dict in ComputeTaskManager._cold_migrate https://review.opendev.org/684407 | 15:25 |
| mriedem | gregwork: nope | 15:27 |
| mriedem | could throw it at the bottom of https://etherpad.openstack.org/p/compute-api-microversion-gap-in-osc | 15:27 |
| mriedem | (i just did) | 15:28 |
| gregwork | i saw saw that :) | 15:28 |
| gregwork | *just | 15:28 |
| gregwork | getting our windows team members to become familiar with the openstack client and one of the things they need for new instances is to nova get-password server rsa-key .. trying to streamline the tools they start off with | 15:29 |
| mriedem | yeah definitely want new people using osc if they can rather than the mish-mash of per-project CLIs | 15:30 |
| mriedem | but someone would have to add that to osc | 15:30 |
| *** pcaruana has quit IRC | 15:30 | |
| mriedem | it would be a pretty easy addition to osc | 15:30 |
| *** gyee has joined #openstack-nova | 15:35 | |
| mriedem | gregwork: the question is if you have developers? | 15:40 |
| *** TxGirlGeek has quit IRC | 15:40 | |
| *** macz has joined #openstack-nova | 15:40 | |
| gregwork | so its early days for my org, it would probably be me writing something like that. We are in the middle of a prod build of openstack where the broader organization must now get involved. | 15:41 |
| gregwork | it would also need to be something id get our vendor to backport | 15:42 |
| mriedem | ok, shouldn't really be too hard, it's mostly just argparse stuff in osc and it calls through to the novaclient python api bindings | 15:42 |
| gregwork | we are definitely not brave enough to run upstream in production | 15:42 |
| mriedem | backport how far? queens? | 15:42 |
| * gregwork nods | 15:42 | |
| mriedem | ah red hat | 15:42 |
| gregwork | wha .. who where ? | 15:42 |
| gregwork | :D | 15:42 |
| mriedem | if only we (nova) had some red hat developers around | 15:43 |
| gregwork | indeed | 15:44 |
| *** ivve has quit IRC | 15:45 | |
| *** markvoelker has quit IRC | 15:45 | |
| *** tellesnobrega has joined #openstack-nova | 15:46 | |
| *** macz has quit IRC | 15:47 | |
| mriedem | mlavalle: routed networks is not supported in nova yet, no - see https://review.opendev.org/656885 | 15:49 |
| mriedem | i don't plan on actively pushing that forward | 15:49 |
| mlavalle | mriedem: if my employer needs, this, could we push it forward in U? | 15:50 |
| *** mdbooth has joined #openstack-nova | 15:52 | |
| mriedem | you'll have to talk to the boss, efried | 15:52 |
| mlavalle | mriedem: thanks for following up :-) | 15:53 |
| mriedem | yw | 15:53 |
| efried | mlavalle: Depends who your employer is | 15:53 |
| efried | I kid, I kid. | 15:53 |
| mlavalle | would you be open to use pushing this forward? | 15:53 |
| efried | what are we talking about? | 15:53 |
| mlavalle | to us^^^ | 15:54 |
| *** lbragstad_ has joined #openstack-nova | 15:54 | |
| kashyap | efried: https://review.opendev.org/#/c/656885/ | 15:54 |
| kashyap | Ah, you were jesting | 15:54 |
| *** lbragstad has quit IRC | 15:55 | |
| efried | Has there been a bp and/or spec at any point? | 15:55 |
| efried | is this related? https://review.opendev.org/#/c/263898/ | 15:55 |
| mlavalle | this is the approved spec: http://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/neutron-routed-networks.html? | 15:56 |
| mlavalle | yeah same thing.... | 15:56 |
| *** itlinux has joined #openstack-nova | 15:57 | |
| *** lbragstad has joined #openstack-nova | 15:57 | |
| efried | mlavalle: I'm toying with ideas for ways to dramatically constrain the scope of what we commit to in ussuri, on the theory that stakeholders (such as your employer, whoever that may be) would rather get a hard "no" up front than have a bp/spec approved and have NO IDEA whether it's going to land, based on our track record of landing a random 60-ish percent of the things we approve, with no relation to the supposed prioritization | 15:57 |
| mriedem | mlavalle works for verizon media with jroll and penick and the gang | 15:58 |
| *** lbragstad_ has quit IRC | 15:58 | |
| mlavalle | LOL, I didn't know we are the gang | 15:58 |
| mriedem | the routed networks stuff in that spec back in newton was a lot more grandiose with refactoring all of the port creation stuff in conductor - a lot of stuff johnthetubaguy worked on while at OSIC | 15:58 |
| mriedem | what functionally landed on the nova side was actually a really small change to allow deferred fix ip allocation | 15:59 |
| mriedem | for which https://review.opendev.org/#/c/641670/ is a follow up | 15:59 |
| mriedem | the actual routed networks thing in nova is a lot more complicated, | 15:59 |
| efried | Beyond that, I have no objection to this being a thing in general. But I haven't yet figured out a reasonable way to propose and effect such a constrained scope. | 15:59 |
| mriedem | with the routed networks stuff, i'd say showing it working in a multinode ci job would be table stakes since there is a lot of orchestration and setup involved between nova, neutron and placement | 16:00 |
| mriedem | but lots of people have wanted that feature for a long time (godaddy also) | 16:00 |
| mriedem | so that you can scale out neutron, | 16:01 |
| mriedem | efried: think of it like cells v2 for neutron | 16:01 |
| mriedem | constrain a network to a host aggregate and when creating a server on that network, make nova smart enough to only ask the scheduler for hosts from that network | 16:01 |
| mriedem | rather than randomly pick some host in another aggregate that can't talk to that network | 16:02 |
| mriedem | i wouldn't be surprised if people do that with customer filters and scheduler hints today | 16:02 |
| mlavalle | Neutron already creates the data in placement. what we need now is for Nova to use that data | 16:02 |
| mlavalle | during the scheduling process | 16:02 |
| mriedem | yup, that's what my patch started | 16:03 |
| mriedem | link the requested network(s) to a resource provider aggregate and pre-filter the placement results using that | 16:03 |
| efried | Above is explaining "what" and "why", which is great. Right now though I'm only focused on "how much". The "why" will become relevant when arguing whether this thing should bump some other thing out of scope for ussuri. | 16:03 |
| *** spatel has joined #openstack-nova | 16:03 | |
| mriedem | if verizon can somehow say they'll use vpmems would that sweeten the deal? | 16:05 |
| mriedem | i kid | 16:05 |
| mriedem | cyborg is probably the biggest ticket thing that should be pushed forward in ussuri | 16:06 |
| mriedem | given the years of talk | 16:06 |
| mriedem | granted, routed networks was from newton but who's keeping track | 16:06 |
| *** spatel has quit IRC | 16:07 | |
| mlavalle | All I can add to this is that we really, really need this | 16:09 |
| *** gbarros has joined #openstack-nova | 16:09 | |
| *** yonglihe has quit IRC | 16:10 | |
| mlavalle | how can we get this topic in the "how much" consideration? | 16:10 |
| cdent | people | 16:10 |
| mlavalle | well, I already said we would push it forward | 16:11 |
| mlavalle | that's how the conversation started | 16:11 |
| mriedem | mlavalle: how hard would it be to get a 2-node CI job setup which sets up a host aggregate for one node and a host aggregate for another and a separate network for each aggregate so the neutron+placement stuff happens? | 16:12 |
| cdent | mlavalle: sorry I was joining in late and didn't mean to sound snarky (if that's the way it sounded) | 16:12 |
| mriedem | i think work could be done toward the eventual goal that needs to happen anyway | 16:12 |
| mlavalle | cdent: oh, no I didn't take it as "snarky".... I was just catching up ;-) | 16:13 |
| mriedem | maybe it's just a matter of a tempest test against a 2 node job? tempest would create the per-node network and aggregate right? | 16:13 |
| mriedem | the neutron would wire up the placement resource provider aggregate? | 16:13 |
| mriedem | *then | 16:13 |
| mlavalle | correct | 16:13 |
| mriedem | after that it's just a matter of writing a tempest test that makes sure the server gets created on the correct host | 16:14 |
| efried | mlavalle: honestly, our poor track record has more to do with reviewer resource than developer resource | 16:14 |
| efried | bbiab | 16:14 |
| *** efried is now known as efried_rollin | 16:14 | |
| mriedem | having functioning CI goes a long way in reviewer confidence | 16:14 |
| mriedem | numa live migration wouldn't have made it this cycle (again) without that | 16:14 |
| mlavalle | mriedem: this is a great point | 16:15 |
| mlavalle | we can start there | 16:15 |
| mriedem | a deterministic tempest test for routed networks would be, i think, trying to create a server with a network in aggregate 1 but requesting a host in aggregate 2 and seeing it blow up during scheduling | 16:15 |
| mriedem | the test would assert the server fails due to NoValidHost | 16:15 |
| mlavalle | yeap | 16:15 |
| mriedem | with 2.74 in train that's pretty easy https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id66 | 16:16 |
| mriedem | so you (or whoever) could get started with the tempest test which would actually assert that the server gets created on the wrong host for the network (port binding might actually fail i'd guess) until nova supports routed networks during scheduling | 16:17 |
| mlavalle | yeah, thanks for pointing that out | 16:17 |
| mriedem | david bingham on https://review.opendev.org/#/c/656885/ is from godaddy and he and another guy were at the last ptg asking about this as well - it was the only thing they talked about while at the ptg | 16:18 |
| mriedem | coincidentally | 16:18 |
| mlavalle | efried_rollin: developer resource is a start, isn't it? | 16:18 |
| mriedem | so, maybe verizon and godaddy SIG UP on this | 16:18 |
| mriedem | oh wait, | 16:18 |
| mriedem | not SIG | 16:18 |
| mriedem | POP UP TEAM UP! | 16:18 |
| mriedem | cdent: as TC emeritus you can correct me if i'm wrong | 16:19 |
| cdent | for routed networks pop up team would be the team-type of the the day | 16:19 |
| mriedem | du jour?! | 16:19 |
| cdent | so sorry, yes | 16:19 |
| cdent | i used up all my french recently | 16:20 |
| mriedem | TC santioned collaborama du jour | 16:20 |
| mriedem | is what i'm going to go with | 16:20 |
| cdent | i believe pop up came about in response to encrypted volume efforts failing to get traction | 16:20 |
| cdent | I fail to get traction understanding why a different name helps | 16:20 |
| mriedem | s/volume/image/ but yeah | 16:20 |
| *** brault has quit IRC | 16:20 | |
| mriedem | because SIGs were too formal dude | 16:20 |
| * cdent blinks | 16:21 | |
| * cdent blames capitalism | 16:21 | |
| cdent | I have really mixed feelings on all this stuff. | 16:22 |
| cdent | On the one hand it is wrong that nova gatekeeps so much | 16:22 |
| *** ociuhandu has quit IRC | 16:22 | |
| cdent | but on the other hand, if nova doesn't gate keep things go sideways | 16:22 |
| *** priteau has quit IRC | 16:23 | |
| *** ociuhandu has joined #openstack-nova | 16:23 | |
| *** jmlowe has quit IRC | 16:24 | |
| mriedem | mlavalle: i added you to https://review.opendev.org/#/c/656885/3 and left a comment there with link to this irc conversation if that helps | 16:24 |
| * cdent dinners | 16:24 | |
| mlavalle | mriedem: it definitely does. Thanks you very much! | 16:24 |
| *** ociuhandu has quit IRC | 16:28 | |
| gregwork | is it terribly difficult to to fiddle with libvirt <features> and <cpu> in nova? | 16:29 |
| gregwork | i need to figure out how to set these to fool windows into not being terrible at everything because it detects its in kvm | 16:29 |
| gregwork | https://pastebin.com/jw8Duq9S | 16:29 |
| gregwork | it actively checks to see if its in kvm and turns off stuff | 16:29 |
| gregwork | thats how i work around it using regular libvirt | 16:30 |
| mriedem | gregwork: gpu? | 16:30 |
| mriedem | gregwork: see https://review.opendev.org/#/c/579897/ | 16:31 |
| gregwork | so not an nvidia badness, i want to enable HyperV server role in my guest so I can run the cloud-base.it image generation git project | 16:31 |
| gregwork | and it wont let you do that becuase HyperV detects KVM | 16:31 |
| gregwork | and says "newp!" | 16:31 |
| gregwork | this thing: https://cloudbase.it/windows-server-2016-openstack-images/ | 16:32 |
| mriedem | well i think you're looking for the img_hide_hypervisor_id image property or the hide_hypervisor_id flavor extra spe | 16:32 |
| mriedem | *spec | 16:32 |
| mriedem | the image property isn't in queens but the flavor extra spec might be | 16:33 |
| mriedem | nope https://blueprints.launchpad.net/nova/+spec/hide-hypervisor-id-flavor-extra-spec was rocky | 16:33 |
| gregwork | and there is no way to pass libvirt domain customizations | 16:33 |
| gregwork | even kludgy ones :) | 16:33 |
| *** TxGirlGeek has joined #openstack-nova | 16:33 | |
| gregwork | or even qemu execution lines | 16:33 |
| mriedem | not through the compute api no | 16:34 |
| mriedem | that's not really...cloud | 16:34 |
| artom | Don't we have image props or something to make Windows happy? | 16:35 |
| mriedem | artom: read scrollback | 16:35 |
| * mriedem slaps artom | 16:36 | |
| gregwork | ever since they started planning to go to core based licensing instead of socket they have implemented some draconian things to prevent the potential for nested virtualization | 16:36 |
| openstackgerrit | Merged openstack/nova master: docs: Scrub available quotas https://review.opendev.org/670125 | 16:36 |
| gregwork | especially since you can do performant nested virt with kvm on intel xeon e3-v4 processors using vmcs shadowing and device passthrough of network/storage | 16:36 |
| *** markvoelker has joined #openstack-nova | 16:38 | |
| gregwork | in our lab using those chips and passing through a virtual function off our nic to the L1 guest (hypervisor) we were only seeing a 5-10% difference in perf | 16:38 |
| gregwork | it was really interesting | 16:38 |
| gregwork | this is how hitachi does LPAR's on their x86_64 platform | 16:38 |
| gregwork | hardware logical partitions which can run performant guests using intel xeon chips | 16:39 |
| gregwork | anyhow it sucks i cant fiddle with this on queens | 16:39 |
| gregwork | :/ | 16:39 |
| mriedem | i can't believe sean-k-mooney isn't around to chat about this | 16:40 |
| mriedem | gregwork: well you could if you $$$ your vendor to backport a feature | 16:41 |
| *** dpawlik has quit IRC | 16:42 | |
| artom | mriedem, next time dinner first | 16:43 |
| *** igordc has joined #openstack-nova | 16:46 | |
| mriedem | dinner? you mean lunch? | 16:47 |
| mriedem | what are you 80? | 16:47 |
| *** jangutter has quit IRC | 16:47 | |
| artom | In my mind :( | 16:47 |
| *** eandersson has joined #openstack-nova | 16:47 | |
| *** markvoelker has quit IRC | 16:48 | |
| *** xek has joined #openstack-nova | 16:49 | |
| *** jmlowe has joined #openstack-nova | 16:50 | |
| *** jamesdenton has quit IRC | 16:52 | |
| *** markvoelker has joined #openstack-nova | 16:53 | |
| mriedem | stephenfin: i'm not sure we need this https://review.opendev.org/#/c/684781/ - it appears to already be fixed in master, though i guess we might want it just for backports | 16:53 |
| mriedem | i left some notes inline | 16:53 |
| *** ivve has joined #openstack-nova | 16:53 | |
| *** henriqueof has joined #openstack-nova | 16:54 | |
| *** henriqueof1 has quit IRC | 16:55 | |
| *** brault has joined #openstack-nova | 16:56 | |
| *** xek_ has joined #openstack-nova | 16:56 | |
| *** xek has quit IRC | 16:59 | |
| *** maciejjozefczyk has quit IRC | 17:02 | |
| gmann | mriedem: ohk. did you push the fix to tag tempest on pike gate ? | 17:02 |
| mriedem | gmann: yup | 17:03 |
| gmann | thanks. got it | 17:03 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Ignore sqla-migrate inspect.getargspec deprecation warnings on py36 https://review.opendev.org/684781 | 17:09 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Ignore warning from sqlalchemy-migrate https://review.opendev.org/684772 | 17:09 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: tox: Use common 'command' definition for unit tests https://review.opendev.org/684774 | 17:09 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: tox: Stop overriding the 'install_command' https://review.opendev.org/684775 | 17:09 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: tests: Nuke OS_STDOUT_CAPTURE, OS_STDERR_CAPTURE https://review.opendev.org/684773 | 17:09 |
| *** ociuhandu has joined #openstack-nova | 17:09 | |
| *** tellesnobrega has quit IRC | 17:14 | |
| *** sridharg has quit IRC | 17:14 | |
| *** ociuhandu has quit IRC | 17:16 | |
| *** panda|bbl is now known as panda | 17:19 | |
| *** udesale has quit IRC | 17:19 | |
| *** cdent has quit IRC | 17:24 | |
| gregwork | mriedem: what about sean-k-mooney ? | 17:26 |
| sean-k-mooney | gregwork: mriedem sriov testing? | 17:33 |
| gregwork | sean-k-mooney: hi, no actually trying to fool a windows instance on openstack into not thinking it is a guest. either modifying the domain in libvirt or possibly passing args to qemu-kvm | 17:34 |
| gregwork | i have a working libvirt domain that does what i need | 17:34 |
| gregwork | trying to figure out how to do this with nova/kvm | 17:34 |
| sean-k-mooney | i tested doing nested sriov like 3 weeks ago and it does work if you enable a vIOMMU in the l1 guest and use q35 chipset but you need to play with the pci layout to fix the iommu groups | 17:34 |
| sean-k-mooney | oh | 17:34 |
| sean-k-mooney | ok | 17:34 |
| sean-k-mooney | i think we fixed that | 17:34 |
| sean-k-mooney | so you can enable hypervior hideing | 17:34 |
| gregwork | yeah the nested virt works great if you have a chip that can do vmcs shadowing, and do passthrough devices for the L1 guest | 17:34 |
| gregwork | if you dont pass through network/storage it will suck real bad for the L2 guests | 17:35 |
| gregwork | at least then you remove 1 layer of nested virt (storage/network) | 17:35 |
| sean-k-mooney | and we not hardcode a fake vendor id in the hyperv secotion so it thinks its running on a phyical host | 17:35 |
| *** jmlowe has quit IRC | 17:35 | |
| gregwork | part of it is disabling the hypervisor cpu flag in the guest | 17:35 |
| gregwork | the other is the vendor string thing | 17:35 |
| sean-k-mooney | yes if you set teh image property i think it does both | 17:36 |
| sean-k-mooney | gregwork: have you tried setting img_hide_hypervisor_id | 17:36 |
| sean-k-mooney | in the image to yes | 17:36 |
| gregwork | do you know if a certain hat wearing vendor backported that to their queens release | 17:37 |
| sean-k-mooney | i could check | 17:37 |
| sean-k-mooney | but not off the top of my head | 17:37 |
| sean-k-mooney | the image property woudl not be backportable due to object changes | 17:37 |
| sean-k-mooney | but i belive you can also contol the behavior via the flavor | 17:38 |
| sean-k-mooney | the flavor extra spec would be backportable | 17:38 |
| sean-k-mooney | downstream | 17:38 |
| sean-k-mooney | it looks like the kvm supprot is ther for linux guests | 17:40 |
| sean-k-mooney | which means the image property should be there | 17:40 |
| *** xek has joined #openstack-nova | 17:40 | |
| sean-k-mooney | but the support for windows guest has not been backported | 17:41 |
| sean-k-mooney | http://paste.openstack.org/show/779253/ | 17:41 |
| sean-k-mooney | gregwork: ^ that is the downstream code for generating the elements | 17:41 |
| *** xek_ has quit IRC | 17:42 | |
| *** xek_ has joined #openstack-nova | 17:44 | |
| *** xek has quit IRC | 17:46 | |
| *** jmlowe has joined #openstack-nova | 17:48 | |
| sean-k-mooney | gregwork: assuming you or one of your customers are a customer of said company have you filed a Bugzilla ro case to request the feature backport. | 17:49 |
| gregwork | im initially trying to figure out if this was a thing accomplished another way | 17:50 |
| mriedem | sean-k-mooney: if you're around i think we're waiting on an ack from you for https://review.opendev.org/#/c/683437/ | 17:50 |
| sean-k-mooney | gregwork: im not committing to it being granted but in principal i belive this would be a minimal risk | 17:50 |
| *** damien_r has joined #openstack-nova | 17:50 | |
| sean-k-mooney | mriedem: strictly speaking its my birthday and im off but yes im waiting for windows updates to install so if have a few minutes | 17:51 |
| gregwork | sean-k-mooney: happy bday sean :) | 17:51 |
| sean-k-mooney | thanks you :) | 17:51 |
| gregwork | i think i spoke to you here a few years about getting dpdk working with mellanox cards | 17:52 |
| gregwork | your nic is familiar | 17:52 |
| *** igordc has quit IRC | 17:52 | |
| gregwork | *nick | 17:52 |
| sean-k-mooney | yes we have spoken before | 17:52 |
| sean-k-mooney | an ya i used to work with dpdk alot | 17:52 |
| sean-k-mooney | im hopeing to get a dpdk gate job running soon | 17:53 |
| *** henriqueof1 has joined #openstack-nova | 17:53 | |
| *** henriqueof has quit IRC | 17:53 | |
| gregwork | are you still with intel | 17:53 |
| sean-k-mooney | no i moved to redhat about 14 months ago | 17:53 |
| sean-k-mooney | hence why i was able to check the OSP downstream code | 17:54 |
| gregwork | oh i just figured you were more in the know | 17:54 |
| gregwork | that link you posted, was that for osp 15 ? | 17:54 |
| *** damien_r has quit IRC | 17:54 | |
| sean-k-mooney | no it was osp 13 which is queens | 17:55 |
| sean-k-mooney | so queens has support for hideing the hypervior signature for linux guest | 17:55 |
| gregwork | not not the required bits for a windows guest | 17:55 |
| sean-k-mooney | all its missing is the hyperv vendor signiture bit | 17:55 |
| gregwork | *but | 17:55 |
| sean-k-mooney | so a backport of the fix would just be the xml generation change the rest of the code is alreay in place | 17:56 |
| sean-k-mooney | gregwork mriedem can correct me if im wrong but i think we fixed windwos guest in stien so 15 should have the support. | 17:57 |
| gregwork | we are doing a prod rollout so the business decision was to stick with LTS releases | 17:57 |
| gregwork | so 14 and 15 dont exist as options for us sadly | 17:58 |
| mriedem | sean-k-mooney: train https://review.opendev.org/#/c/579897/ | 17:58 |
| sean-k-mooney | unfortunetly you are not the only customer with that view | 17:58 |
| sean-k-mooney | oh so it was. it felt recent but i did not think it was train | 17:59 |
| sean-k-mooney | mriedem: we ended up treating this as a bug. is it a candiate for upstream backport? generally i would assume not but it does not hurt to ask | 17:59 |
| gregwork | im debating trying to patch this myself and splice it in /var/lib/config-data/puppet-generated/nova_libvirt and see if i can get the hyperv processor bits to die | 17:59 |
| gregwork | then maybe circle back with an rfe | 18:00 |
| mriedem | sean-k-mooney: exhibit A https://review.opendev.org/#/c/663616/ | 18:04 |
| mriedem | either way the hide_hypervisor_id stuff upstream was in rocky so gregwork doesn't have it anyway | 18:05 |
| mriedem | unless you guys in rhosp are going to backport a feature | 18:05 |
| gregwork | for great justice! | 18:05 |
| sean-k-mooney | we cant backport object changes downstream so that must mean that only the flaovr bassed option was suported in queens downstream | 18:06 |
| sean-k-mooney | i did not check if the full feature was there | 18:06 |
| openstackgerrit | Boris Bobrov proposed openstack/nova master: Set iommu driver for virtio controllers too https://review.opendev.org/684825 | 18:08 |
| *** tesseract has quit IRC | 18:08 | |
| *** luksky has joined #openstack-nova | 18:20 | |
| *** tbachman has quit IRC | 18:22 | |
| sean-k-mooney | mriedem: just responeded on https://review.opendev.org/#/c/683437 | 18:22 |
| sean-k-mooney | its close i have one nit but its not overtly worng but there is a subtle point i noted on ptachset 3 | 18:23 |
| *** tbachman has joined #openstack-nova | 18:29 | |
| *** jkulik has quit IRC | 18:30 | |
| *** jkulik has joined #openstack-nova | 18:31 | |
| *** xek_ has quit IRC | 18:32 | |
| mriedem | efried_rollin: i'm going to try to write a functional recreate test for https://bugs.launchpad.net/nova/+bug/1841481 since the fix(es) involved are pretty hairy | 18:39 |
| openstack | Launchpad bug 1841481 in OpenStack Compute (nova) "Race during ironic re-balance corrupts local RT ProviderTree and compute_nodes cache" [Medium,In progress] - Assigned to Eric Fried (efried) | 18:39 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691 | 18:40 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213 | 18:40 |
| *** tbachman has quit IRC | 18:45 | |
| *** tbachman has joined #openstack-nova | 18:47 | |
| mriedem | dustinc: the blueprint on those changes is going to have to be updated for ussuri | 18:48 |
| mriedem | in case you haven't re-proposed the spec and added a new blueprint | 18:48 |
| mriedem | call it openstacksdk-in-nova-ussuri | 18:49 |
| openstackgerrit | Merged openstack/nova master: Log error when volume validation fails during boot from volume https://review.opendev.org/684140 | 18:55 |
| *** ricolin has quit IRC | 18:55 | |
| openstackgerrit | Merged openstack/nova master: Add functional tests for [cinder]/cross_az_attach=False https://review.opendev.org/684141 | 18:55 |
| *** belmoreira has joined #openstack-nova | 18:55 | |
| *** gary_perkins has quit IRC | 18:57 | |
| *** gary_perkins has joined #openstack-nova | 18:57 | |
| *** belmoreira has quit IRC | 18:58 | |
| *** factor has quit IRC | 18:59 | |
| *** dpawlik has joined #openstack-nova | 19:05 | |
| *** ociuhandu has joined #openstack-nova | 19:17 | |
| *** efried_rollin is now known as efried | 19:21 | |
| efried | mriedem: Are we considering that (ironic rebalance snafu) an RC candidate? | 19:21 |
| mriedem | no, it's a regression since at least queens | 19:22 |
| *** ociuhandu has quit IRC | 19:25 | |
| efried | mriedem: what's top priority to get merged today? | 19:27 |
| efried | or approved today, anyway. Merging might take a couple weeks | 19:27 |
| ozzzo | I'm building a new OS cluster and the boxes all have the same CPU including microcode, but they update to the latest microcode when they reboot. If I set cpu_mode to host-passthrough what will happen when microcode updates? Will VMs built on older microcode be able to migrate to HV with newer? | 19:28 |
| ozzzo | or do I need to disable updating of microcode, if I want to run host-passthrough? | 19:28 |
| *** dpawlik has quit IRC | 19:29 | |
| *** TxGirlGeek has quit IRC | 19:29 | |
| efried | ozzzo: I think that kind of question is better for the #openstack channel. This one is for development. | 19:29 |
| ozzzo | nobody answers questions in #openstack | 19:30 |
| ozzzo | not even to refer you to another channel | 19:30 |
| ozzzo | questions are answered with silence | 19:30 |
| efried | oh | 19:30 |
| efried | that's a pity | 19:30 |
| ozzzo | so I am trying other #openstack-* channels | 19:30 |
| efried | sean-k-mooney, kashyap: that question ---^ seems like it might be in your wheelhouse? | 19:30 |
| mriedem | kashyap has a summit talk about this, trying to find it | 19:31 |
| mriedem | https://docs.openstack.org/nova/latest/admin/configuration/hypervisor-kvm.html#specify-the-cpu-model-of-kvm-guests | 19:31 |
| efried | well, at least I was right about something :P | 19:31 |
| mriedem | https://www.openstack.org/videos/summits/berlin-2018/effective-virtual-cpu-configuration-in-nova | 19:31 |
| mriedem | efried: i think the immediate priority is just the prelude and dependent docs | 19:32 |
| mriedem | https://etherpad.openstack.org/p/nova-train-release-todo | 19:32 |
| ozzzo | ok I'll watch that, ty! | 19:32 |
| efried | ack; I just approved the pcpu doc (sean-k-mooney hope that's okay, anything else can be fupped?). Not sure what to do about the vpmem doc | 19:32 |
| sean-k-mooney | ozzzo: no libivrt checks that the microcode matach when you use host-passthough but i belive it also does that for host-model | 19:32 |
| efried | I'd be willing to +2 despite having my fingers in it, but definitely want another (non-Intel) core. It looks like stephenfin didn't get back to it today... | 19:33 |
| sean-k-mooney | ozzzo: kashyap might be able to clarify that. | 19:33 |
| mriedem | sean-k-mooney: what happened to your birthday? | 19:34 |
| efried | ozzzo: fyi, kashyap is euro timezone (belgium I think) in case you want to stalk him earlier in the day | 19:34 |
| efried | all birthdays are suspended pending RC1 | 19:34 |
| sean-k-mooney | efried: ya as i said i have one nit that i noted on ps3 were its not quite correct but its better then it was and it can be fixed up later | 19:34 |
| ozzzo | I watched kashyap's video last week when I was figuring out which cpu_mode I wanted. Unless I missed something, he doesn't specifically address what will happen when microcode updates | 19:34 |
| ozzzo | if microcode updates won't ruin live migration, then I definitely want to use host-passthrough | 19:35 |
| sean-k-mooney | the check is done by libvirt not nova | 19:35 |
| ozzzo | even if they will, I still want to use it, if I can disable upgrade | 19:35 |
| sean-k-mooney | for host-passthough i belive libvirt will require they are the same | 19:35 |
| sean-k-mooney | for host model i am not sure | 19:35 |
| ozzzo | I guess I should do an experiment | 19:36 |
| sean-k-mooney | mriedem: i have a beer chilling in the fridge and im about to go play a game on my other monitors so im not really working at the momemt other then chating on irc | 19:37 |
| sean-k-mooney | but i also wont really be paying attention to irc shortly either | 19:37 |
| ozzzo | oic kashyap is here, nice | 19:38 |
| ozzzo | I'll ask him tomorrow morning | 19:38 |
| *** cdent has joined #openstack-nova | 19:41 | |
| *** tellesnobrega has joined #openstack-nova | 19:42 | |
| *** TxGirlGeek has joined #openstack-nova | 19:50 | |
| *** tbachman has quit IRC | 19:53 | |
| dustinc | mriedem: Thanks for heads up regarding bp, the spec was reintroduced but with same/old bp. (https://review.opendev.org/#/c/680471/). How do I find the old bp to use it for reference? I search here and it does not show up: https://blueprints.launchpad.net/openstack?searchtext=openstacksdk-in-nova | 19:54 |
| dustinc | oops ignore that, I was mixing links | 19:55 |
| dustinc | been bouncing back and forth between two things, found the bp | 19:55 |
| *** TxGirlGeek has quit IRC | 19:56 | |
| *** tbachman has joined #openstack-nova | 19:57 | |
| mriedem | rps | 19:58 |
| mriedem | oops | 19:58 |
| mriedem | https://blueprints.launchpad.net/nova/+spec/openstacksdk-in-nova | 19:58 |
| mriedem | yeah so the spec should be re-proposed for ussuri with the new openstack-sdk-in-nova-ussuri blueprint | 19:58 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/readme.html#previously-approved-specifications | 19:59 |
| dustinc | ok i will check that out, basically I just want to update the bp with what has already been done and clarify what is still left? | 20:05 |
| dustinc | and then reintroduce spec per the readme link you posted? | 20:06 |
| mriedem | efried: ok i think i have a recreate on that ironic rebalance race bug, it's pretty hairy but it does the job i think | 20:06 |
| efried | mriedem: we can do "previously approved" with continuation bps? | 20:13 |
| mriedem | i don't really see why not | 20:14 |
| dustinc | mriedem: I am reading this but still a bit confused regarding what I need to do with the bp itself, do I need to register a new bp as well? https://specs.openstack.org/openstack/nova-specs/readme.html#previously-approved-specifications | 20:14 |
| dustinc | https://wiki.openstack.org/wiki/Blueprints#Spec_.2B_Blueprints_lifecycle | 20:16 |
| mriedem | dustinc: yes | 20:23 |
| dustinc | thanks | 20:23 |
| mriedem | dustinc: register a new blueprint called openstacksdk-in-nova-ussuri, the description should just link to https://blueprints.launchpad.net/nova/+spec/openstacksdk-in-nova and say something to the effect of it's continuing the work started in the train release | 20:23 |
| mriedem | as for re-proposing the spec, you'll copy specs/train/approved/openstacksdk-in-nova.rst to specs/ussuri/approved/openstacksdk-in-nova-ussuri.rst | 20:24 |
| mriedem | that's a bit different from https://specs.openstack.org/openstack/nova-specs/readme.html#previously-approved-specifications because the name changes - i think this is the first time we've had a multi-release continuation blueprint thing like this for specs | 20:25 |
| mriedem | so it's different and weird | 20:25 |
| mriedem | when efried did the ksa-ification there was a spec so i don't remember what he did between releases | 20:25 |
| mriedem | we might have just never completed the blueprint until that work was done, idk | 20:25 |
| efried | Personally I really don't care how we track it. We already marked the current bp complete for Train, but... is it possible to un-mark it and instead push it to ussuri? | 20:26 |
| efried | I marked it complete because we completed part of the work, and I figured we would want a new proposal for ussuri that describes what we got done, what we're going to do next, and what, if anything, has changed about the roadmap. | 20:27 |
| efried | but that's more a statement of the spec than the bp | 20:27 |
| efried | dustinc, mriedem: yeah, that seemed to work fine https://blueprints.launchpad.net/nova/+spec/openstacksdk-in-nova | 20:29 |
| efried | so just reuse the same bp dustinc ^ | 20:29 |
| *** ralonsoh has quit IRC | 20:30 | |
| efried | but I think the new spec should differ as noted, so probably not a fast-approval thing. | 20:30 |
| efried | but still should be fairly trivial. | 20:30 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1841481 https://review.opendev.org/684840 | 20:31 |
| openstack | bug 1841481 in OpenStack Compute (nova) "Race during ironic re-balance corrupts local RT ProviderTree and compute_nodes cache" [Medium,In progress] https://launchpad.net/bugs/1841481 - Assigned to Eric Fried (efried) | 20:31 |
| mriedem | efried: ^ hopefully with all of the comments that is clear enough | 20:31 |
| efried | ack | 20:31 |
| mriedem | efried: for multi-release things we usually mark the bp as partially complete | 20:32 |
| mriedem | and then create a new bp with the -<new release> | 20:32 |
| mriedem | like with mox removal | 20:32 |
| efried | yeah | 20:32 |
| efried | Where/how/why does it matter which way we do it? | 20:32 |
| efried | aka "who cares?" | 20:32 |
| mriedem | idk that it does all that much, you'll just be carrying that spec forward for a long time potentially | 20:33 |
| mriedem | but that's what you did with ksa-ification i think | 20:33 |
| efried | which still isn't complete, really :( | 20:33 |
| mriedem | cinder was the only one not done wasn't it? or also barbican/castellan? | 20:33 |
| efried | cinder yes. I think we decided barbican was a no-op. | 20:34 |
| *** nweinber_ has quit IRC | 20:35 | |
| mriedem | ah crap need to add the test | 20:35 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1841481 https://review.opendev.org/684840 | 20:36 |
| openstack | bug 1841481 in OpenStack Compute (nova) "Race during ironic re-balance corrupts local RT ProviderTree and compute_nodes cache" [Medium,In progress] https://launchpad.net/bugs/1841481 - Assigned to Eric Fried (efried) | 20:36 |
| efried | I will pay attention to this after RC1 | 20:49 |
| efried | noting that I'm going to look for an opportunity to take some pto soon. | 20:49 |
| *** slaweq has quit IRC | 20:55 | |
| *** tbachman has quit IRC | 21:02 | |
| *** igordc has joined #openstack-nova | 21:03 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: doc: attaching virtual persistent memory to guests https://review.opendev.org/680300 | 21:03 |
| efried | alex_xu, stephenfin: Please merge this ^ asap assuming you're happy. | 21:07 |
| *** cdent has quit IRC | 21:12 | |
| *** TxGirlGeek has joined #openstack-nova | 21:15 | |
| *** BjoernT has quit IRC | 21:17 | |
| efried | mriedem, stephenfin: Can one/both of you +A the prelude patch https://review.opendev.org/#/c/683327/ at this point so it gets in line after the deps? | 21:21 |
| mriedem | done | 21:22 |
| efried | thx | 21:22 |
| mriedem | zzzeek: if you're about, i have a test which keeps failing trying to insert a resource to the db with this: | 21:23 |
| mriedem | sqlalchemy.exc.InvalidRequestError: This session is in 'inactive' state, due to the SQL transaction being rolled back; no further SQL can be emitted within this transaction. | 21:23 |
| mriedem | it's using a transaction factory context manager thing from oslo.db, | 21:24 |
| zzzeek | mriedem: that woudl suggest someting went wrong prior to the thing you are trying t do | 21:24 |
| zzzeek | like, youre in an exception handler or something like that | 21:24 |
| mriedem | i would think if there was a problem with the insert, like a unique constraint failure or something, it'd be logged but i'm not seeing that | 21:24 |
| mriedem | ah... | 21:24 |
| mriedem | yes there is one of those in play | 21:24 |
| zzzeek | mriedem: so, say you're in an exception throw, then you try to do somethign with an object, and something on it isn't loaded | 21:24 |
| zzzeek | mriedem: that would cause this | 21:25 |
| zzzeek | err inside a catch, that is | 21:25 |
| mriedem | yeah it's blowing up from the transaction context manager added here: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L691 | 21:26 |
| mriedem | and likely from something in https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L703 | 21:26 |
| mriedem | the weird thing is i'm hitting this https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L716 | 21:26 |
| mriedem | so it shouldn't be re-raising | 21:26 |
| mriedem | oh maybe this is failing https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L745 | 21:29 |
| mriedem | adding some try/except/log | 21:29 |
| mriedem | nope not hitting that | 21:30 |
| mriedem | i'm definitely getting here https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L716 | 21:30 |
| *** TxGirlGeek has quit IRC | 21:30 | |
| *** TxGirlGeek has joined #openstack-nova | 21:31 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Conf option cross-reference in config docs https://review.opendev.org/583025 | 21:39 |
| zzzeek | mriedem: these are hard to debug | 21:41 |
| mriedem | so we have another test that does this same thing but only within 1 nova-compute service, so i'm wondering if it's something with the thread local context stuff in oslo.db | 21:41 |
| mriedem | b/c this test has 2 services using the same context | 21:41 |
| mriedem | the other one was https://review.opendev.org/#/c/675496/ and things worked like a charm there | 21:42 |
| mriedem | we hit DBDuplicateEntry, then do a get-and-update and squash the re-raise | 21:42 |
| mriedem | but that's not working this time | 21:42 |
| efried | dustinc: It looks like the ironic CI is failing again on service discovery, but I'm not sure why... | 21:47 |
| *** avolkov has quit IRC | 21:48 | |
| efried | maybe not discovery | 21:51 |
| *** BjoernT has joined #openstack-nova | 21:54 | |
| *** TxGirlGeek has quit IRC | 21:56 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Clear RT.compute_nodes entry if re-balance update fails https://review.opendev.org/684849 | 22:09 |
| *** gbarros has quit IRC | 22:10 | |
| *** rcernin has joined #openstack-nova | 22:15 | |
| *** TxGirlGeek has joined #openstack-nova | 22:37 | |
| *** mriedem has quit IRC | 22:53 | |
| *** tkajinam has joined #openstack-nova | 22:59 | |
| *** luksky has quit IRC | 23:00 | |
| *** igordc has quit IRC | 23:10 | |
| openstackgerrit | Merged openstack/nova master: Ignore sqla-migrate inspect.getargspec deprecation warnings on py36 https://review.opendev.org/684781 | 23:10 |
| openstackgerrit | Merged openstack/nova master: Ignore warning from sqlalchemy-migrate https://review.opendev.org/684772 | 23:11 |
| openstackgerrit | Merged openstack/nova master: docs: Clarify everything CPU pinning https://review.opendev.org/683437 | 23:11 |
| openstackgerrit | Merged openstack/nova master: docs: Update CPU topologies guide to reflect the new PCPU world https://review.opendev.org/683485 | 23:11 |
| openstackgerrit | Merged openstack/nova master: Move HostNameWeigher to a common fixture https://review.opendev.org/683874 | 23:11 |
| openstackgerrit | Merged openstack/nova master: Correct link to placement upgrade notes https://review.opendev.org/684661 | 23:11 |
| *** BjoernT has quit IRC | 23:21 | |
| *** dave-mccowan has joined #openstack-nova | 23:28 | |
| *** ivve has quit IRC | 23:28 | |
| *** dave-mccowan has quit IRC | 23:33 | |
| *** tellesnobrega has quit IRC | 23:42 | |
| *** gyee has quit IRC | 23:44 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!