| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-25 | |||
| 07:42:52 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243 | |
| 07:42:53 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | |
| 07:42:53 | 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:54 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999 | |
| 07:59:36 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move HostNameWeigher to a common fixture https://review.opendev.org/683874 | |
| 09:22:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Delete instance_id_mappings record in instance_destroy https://review.opendev.org/684658 | |
| 09:28:24 | openstackgerrit | Chris Dent proposed openstack/nova master: Correct link to placement upgrade notes https://review.opendev.org/684661 | |
| 09:30:51 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Mask the token used to allow access to consoles https://review.opendev.org/220622 | |
| 10:29:21 | openstackgerrit | Merged openstack/nova master: Remove upgrade specific info from user facing exception text https://review.opendev.org/684253 | |
| 10:37:04 | openstackgerrit | Merged openstack/python-novaclient master: Add a check for --config-drive option on nova boot https://review.opendev.org/653683 | |
| 10:37:48 | openstackgerrit | Chris Dent proposed openstack/nova master: single pass instance info fetch in host manager https://review.opendev.org/623558 | |
| 11:30:26 | openstackgerrit | zhurong proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999 | |
| 11:32:04 | gibi | stephenfin, alex_xu: do you have ideas how we will do the nova project update in Shanghai? | |
| 11:43:03 | cdent | stephenfin: this is a nova docs change driven by the structural changes in the placement docs: https://review.opendev.org/#/c/684661/ | |
| 12:06:44 | stephenfin | gibi: Haven't a clue. I was hoping efried would just give us slides and we'd use those \o/ | |
| 12:07:10 | gibi | stephenfin: :) | |
| 12:26:57 | donnyd | did we ever get that grenade thing sorted out? | |
| 12:55:58 | mriedem | stephenfin: you might want to propose wording for the pcpu stuff you want to see in https://review.opendev.org/#/c/683327/ | |
| 12:56:31 | 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:59 | 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:57:01 | stephenfin | if that makes sense | |
| 12:57:47 | mriedem | alex_xu: can you review the vpmem docs today? https://review.opendev.org/#/c/680300/ - rc1 is tomorrow | |
| 12:58:08 | mriedem | pinned and unpinned + same host + libvirt seems generic enough to me | |
| 12:58:20 | mriedem | i'd rebase the prelude on the actual admin docs patch and just link to that for details | |
| 12:58:28 | mriedem | i can make that change | |
| 13:06:18 | mriedem | stephenfin: who else reviewed the pcpu series that can review those docs? | |
| 13:06:19 | mriedem | bauzas: ? | |
| 13:06:43 | stephenfin | bauzas, alex_xu or efried should be able to handle it, I suspect | |
| 13:12:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a prelude for the Train release https://review.opendev.org/683327 | |
| 13:13:00 | mriedem | is it a french holiday today? | |
| 13:13:12 | mriedem | the feast of maximum strikage | |
| 13:13:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reset forced_destination before migration at a proper time https://review.opendev.org/684546 | |
| 13:14:00 | bauzas | mriedem: I can do | |
| 13:14:13 | gibi | mriedem: fixed your comments in ^^ | |
| 13:15:36 | 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:45 | mriedem | new content is fine, but not really when it's in a critical path to merging changes we need for rc1 | |
| 13:16:35 | stephenfin | I can fix them the first two now | |
| 13:16:40 | stephenfin | *the last two | |
| 13:17:01 | stephenfin | the first one (documenting RT) is much more work and not needed here | |
| 13:17:46 | mriedem | ok i guess leave a .. todo:: for whatever needs a follow up | |
| 13:18:07 | stephenfin | there's already one but in another doc (the flavor extra spec one) | |
| 13:18:16 | stephenfin | so I think we're covered | |
| 13:18:21 | stephenfin | I'll comment as much | |
| 13:18:54 | 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:25:29 | openstackgerrit | Jacek Tomasiak proposed openstack/nova master: Ensure volume filesystems are mounted on resume https://review.opendev.org/684290 | |
| 13:25:47 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: VMware VMDK detach: get adapter type from instance VM https://review.opendev.org/653738 | |
| 13:28:47 | mriedem | gibi: on https://review.opendev.org/#/c/684545/ i think we should do that outside of the ServerMovingTests class, | |
| 13:29:10 | 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:28 | gibi | mriedem: ack. I will move the test under regressions | |
| 13:30:36 | stephenfin | mriedem: incoming | |
| 13:30:36 | 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:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Clarify everything CPU pinning https://review.opendev.org/683437 | |
| 13:31:03 | stephenfin | bauzas, alex_xu, sean-k-mooney: https://review.opendev.org/683437 https://review.opendev.org/683485 | |
| 13:31:16 | bauzas | stephenfin: I just provided comments for PS3 :) | |
| 13:31:28 | bauzas | but honestly, it's just a nirt | |
| 13:31:49 | stephenfin | bauzas: I replied | |
| 13:32:03 | bauzas | stephenfin: anyway, it's docs | |
| 13:32:05 | stephenfin | (I already have a glossary at the top of the document explaining the terms) | |
| 13:32:30 | 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:54 | bauzas | stephenfin: eh ok | |
| 13:33:29 | 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:53 | bauzas | unless a doc is wrong, honestly no need for -1ing it | |
| 13:34:11 | bauzas | because it's just better to have a documentation than none | |
| 13:35:59 | 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:59 | mriedem | kashyap: fyi if you or any other libguestfs minded people are about https://bugs.launchpad.net/nova/+bug/1845284 | |
| 13:36:08 | kashyap | mriedem: Hiya | |
| 13:36:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document how to revert, confirm a cold migration https://review.opendev.org/664396 | |
| 13:36:21 | gibi | bauzas: hi! is there a way to fake some VGPUs in devstack without having real hardware? | |
| 13:36:28 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Update resize doc https://review.opendev.org/665412 | |
| 13:36:34 | bauzas | gibi: hah, good question | |
| 13:37:06 | 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:13 | stephenfin | assuming you know anything about quotas | |
| 13:37:26 | stephenfin | bauzas: and if not, https://review.opendev.org/#/c/667133/ is definitely your wheelhouse | |
| 13:38:01 | bauzas | stephenfin: I still have one from you I remember to review https://review.opendev.org/#/c/667133/ | |
| 13:38:08 | bauzas | hah, this | |
| 13:38:23 | 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:24 | stephenfin | That's the one :D | |
| 13:38:33 | openstack | Launchpad bug 1845243 in OpenStack Compute (nova) "Nested 'path' query param in console URL breaks serialproxy" [Undecided,New] | |
| 13:38:33 | mriedem | lyarwood: melwitt: mnaser: stephenfin: weee ironic serial consoles are broken https://bugs.launchpad.net/nova/+bug/1845243 | |
| 13:38:36 | kashyap | mriedem: I'll add words to the bug | |
| 13:38:42 | mriedem | kashyap: thanks | |
| 13:39:20 | 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:37 | mriedem | stephenfin: i reviewed https://review.opendev.org/#/c/670125/ once, just forgot about it, will look again | |
| 13:39:46 | stephenfin | mriedem: That's unfortunate. Should be an easy fix though. Lemme try | |
| 13:39:48 | gibi | bauzas: thanks | |
| 13:39:57 | bauzas | gibi: for example, I could pass my own XML to the test driver | |
| 13:39:59 | 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:40:22 | kashyap | ... except that's by design; the hint is in the file path, they _are_ temporary images that _will_ go away. | |
| 13:41:26 | mriedem | kashyap: i think they are saying the images are left and should be removed | |
| 13:41:42 | mriedem | expected: "In the end of booting instance, can remove those images in /var/tmp/.guestfs-0/appliance.d" | |
| 13:41:48 | mriedem | but yeah, if it's /tmp they should be gone on restart | |
| 13:41:58 | kashyap | mriedem: I parsed it upside down; by design of my brain; ignore me | |
| 13:42:34 | kashyap | But guestfs / libvirt / QEMU version is needed to reproduce. And you asked it there | |
| 13:49:05 | 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:13 | kashyap | It relies on external component to do it | |
| 13:49:27 | kashyap | And Nova _shouldn't_ delete them either - that feels wrong. | |
| 13:49:53 | mriedem | kashyap: ok, so basically it's glance but at a lower level :) | |
| 13:50:14 | 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:17 | mriedem | something like that | |
| 13:50:18 | kashyap | Yeap | |