| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-07 | |||
| 12:11:48 | openstackgerrit | sean mooney proposed openstack/nova master: support pci numa affinity policies in flavor and image https://review.opendev.org/674072 | |
| 12:37:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Pass user context detaching volume on exceptions https://review.opendev.org/560115 | |
| 12:48:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove RBD swap disk if resizing to a flavor without swap https://review.opendev.org/534711 | |
| 13:05:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Convert floating IP tests to ddt https://review.opendev.org/471360 | |
| 13:17:43 | mdbooth | mriedem: If you've got a minute, I wonder if https://review.opendev.org/#/c/683922/1 is sufficiently baked yet. | |
| 13:18:32 | mriedem | we haven't even released train | |
| 13:18:35 | mriedem | so no i don't think so | |
| 13:18:39 | mriedem | i'm talking months | |
| 13:19:09 | mdbooth | Ack, I'll make a note in the review. | |
| 13:19:24 | efried | mdbooth: at a glance, it looks like you're going to need a train backport too if you intend to land it in stein and earlier. | |
| 13:19:45 | mdbooth | efried: It's in master | |
| 13:20:09 | efried | mdbooth: master is ussuri rn | |
| 13:20:35 | mdbooth | Ah, yes. I think we landed it just after the branch. | |
| 13:21:05 | mriedem | no it'll be in train | |
| 13:22:09 | mdbooth | Ok, thanks | |
| 13:23:14 | mdbooth | efried: Yep, double checked and it's in train | |
| 13:23:18 | efried | ah, yup | |
| 13:24:08 | efried | aspiers: ? | |
| 13:40:53 | sean-k-mooney | efried: just to clarify when i update my spec with the core liaison section do i qualify as a liaison or should i find a core to add. | |
| 13:42:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Support move operations with qos ports - Ussuri https://review.opendev.org/686779 | |
| 13:42:12 | efried | sean-k-mooney: I think you qualify. That will be a thing reviewers of your spec would be able to debate. | |
| 13:42:56 | sean-k-mooney | ok ill rebase my spec on your patch and set it to me for now | |
| 13:44:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Avoid raise InstanceNotRunning exception https://review.opendev.org/541152 | |
| 13:45:05 | stephenfin | kashyap: Think you could look into this to see if it makes sense? https://review.opendev.org/#/c/589769/ | |
| 13:59:11 | openstack | bug 1709118 in OpenStack Compute (nova) "_ContextAuthPlugin needs a refresh" [Low,Confirmed] https://launchpad.net/bugs/1709118 | |
| 13:59:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove workaround for bug #1709118 https://review.opendev.org/524316 | |
| 14:00:03 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/queens: Fixes multi-registry config in Quobyte driver https://review.opendev.org/687058 | |
| 14:01:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add boot from volume functional test with a huge request https://review.opendev.org/686734 | |
| 14:01:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cache image GETs for multi-create/multi-BDM requests https://review.opendev.org/686835 | |
| 14:04:28 | mriedem | lyarwood: i think https://review.opendev.org/#/c/679449/ is probably about the only thing worth getting into the next stable/rocky release if you want to push that through and then i'll queue up the release | |
| 14:05:36 | lyarwood | mriedem: ack I'll take a look now | |
| 14:09:11 | tellesnobrega | mriedem, hi, quick question nova reset-state command. I'm wondering if the command delete all ghost vms on the db? In case we do a backup of the DB and restore that later and the vms are gone, how can we clean up the DB | |
| 14:10:47 | mriedem | it does not | |
| 14:10:58 | mriedem | all reset-state does is change the vm_state on the instance and set task_state to None | |
| 14:11:11 | mriedem | it has no interaction with the compute service | |
| 14:11:34 | mriedem | to clean up the db you use the nova-manage db archive_deleted_rows command | |
| 14:11:48 | tellesnobrega | mriedem, awesome | |
| 14:11:50 | mriedem | with the --purge option if you really want to delete the data and not just move it to shadow tables | |
| 14:11:51 | tellesnobrega | thanks, I will take a look | |
| 14:12:01 | mriedem | info is here https://docs.openstack.org/nova/latest/cli/nova-manage.html | |
| 14:12:10 | tellesnobrega | mriedem, thanks | |
| 14:14:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Prevent using blank volume as boot device https://review.opendev.org/171984 | |
| 14:25:52 | gibi | stephenfin: I have one question in https://review.opendev.org/#/c/686795/2/nova/tests/fixtures.py@1255 | |
| 14:27:26 | gshippey | Instance wont migrate to a newly provisioned compute host, this is the error in the nova-compute log: During detach_volume, instance disappeared.: InstanceNotFound: Instance 2b5cdc09-e451-4e1d-9044-4d85ffd29b51 could not be found. Has anyone seen anything similar before? | |
| 14:28:02 | stephenfin | looking | |
| 14:28:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove fstype param from ploop init https://review.opendev.org/444970 | |
| 14:29:02 | stephenfin | gibi: Replied | |
| 14:30:04 | openstackgerrit | Eric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181 | |
| 14:30:08 | efried | alex_xu: ^ | |
| 14:30:14 | efried | thanks for the review. | |
| 14:30:28 | mriedem | stephenfin: kind of hard to care about any ploop/vz patches when their ci hasn't worked in probably over a year | |
| 14:31:25 | stephenfin | That one was fairly trivial so it shouldn't matter, hopefully, but yeah, agreed | |
| 14:31:58 | gibi | stephenfin: thanks | |
| 14:32:16 | mriedem | stephenfin: so...remove your +2 until their CI passes on it? | |
| 14:32:28 | mriedem | it's just code cleanup, not really fixing a bug | |
| 14:32:31 | mriedem | as far as i can tell | |
| 14:32:56 | stephenfin | kewl | |
| 14:32:57 | stephenfin | yeah | |
| 14:32:59 | stephenfin | it is | |
| 14:34:46 | openstackgerrit | Eric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181 | |
| 14:34:46 | openstack | bug 1845530 in OpenStack Compute (nova) "Versioned discovery endpoint should not require authentication" [Undecided,In progress] https://launchpad.net/bugs/1845530 - Assigned to Eric Fried (efried) | |
| 14:34:46 | openstackgerrit | Eric Fried proposed openstack/nova master: Repro bug 1845530: versioned discovery is authed https://review.opendev.org/685180 | |
| 14:34:53 | efried | resolved merge conflict ^ | |
| 14:40:51 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: [WIP] Move Nova Quobyte driver to LibvirtMountedFileSystemVolumeDriver https://review.opendev.org/687066 | |
| 14:47:48 | openstackgerrit | Eric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181 | |
| 14:47:48 | openstack | bug 1845530 in OpenStack Compute (nova) "Versioned discovery endpoint should not require authentication" [Undecided,In progress] https://launchpad.net/bugs/1845530 - Assigned to Eric Fried (efried) | |
| 14:47:48 | openstackgerrit | Eric Fried proposed openstack/nova master: Repro bug 1845530: versioned discovery is authed https://review.opendev.org/685180 | |
| 14:48:05 | efried | mriedem: clarified those comments in the test case -^ | |
| 14:49:57 | mriedem | stephenfin: remember when you abandon a patch linked to a bug, e.g. https://review.opendev.org/#/c/585978/ - you should probably update the bug to say it's not in progress and remove the owner | |
| 14:53:55 | mriedem | ack | |
| 14:54:29 | mriedem | stephenfin: i know you're busy with cleaning up old patches, but now that you've got my attention on this giant docs rewrite https://review.opendev.org/#/c/667133/ it'd be nice to close that out | |
| 14:55:04 | stephenfin | I can do both of those shortly, yup (y) | |
| 15:03:42 | mriedem | thanks | |
| 15:04:18 | mriedem_afk | gibi: dansmith: how are you feeling about this now? https://review.opendev.org/#/c/633293/ | |
| 15:04:34 | dansmith | will look in a biot | |
| 15:05:12 | gibi | mriedem_afk: I have to get back to that tomorrow | |
| 15:14:19 | donnyd | with the understand I know everyone is busy, is there anything I can do to move the ball forward on LXC | |
| 15:21:19 | efried | sean-k-mooney: ^ ? | |
| 15:25:04 | sean-k-mooney | donnyd: specificaly are you refering to https://review.opendev.org/#/c/667976/ or to hardeingin the lxc support in general | |
| 15:25:57 | sean-k-mooney | i tested ^ locally and it does indeed work so i think we shoudl merge it | |
| 15:25:57 | donnyd | sean-k-mooney: I am speaking more generally in terms of what can we do to | |
| 15:26:01 | donnyd | but yes that too | |
| 15:27:05 | sean-k-mooney | donnyd: mriedem_afk has a WIP lxc job i belive. feature wise the next thing i would like to see fixed is automaic resize of the disk to match the flavor for lxc | |
| 15:27:14 | sean-k-mooney | or rather the filesystem. | |
| 15:28:12 | sean-k-mooney | i think the block device is resized my nova already but i dont think you can resize the filesystme using growpart/resize2fs inside the lxc contiaer itself | |
| 15:28:41 | sean-k-mooney | once that was working my next wishlist item would be to fix the console. | |
| 15:29:22 | sean-k-mooney | neither of these i suspect would be terible hard to fix but we just have not spent the time to do it. | |
| 15:29:46 | donnyd | Isn't that a cloud-init function anyways | |
| 15:30:12 | sean-k-mooney | it is but i have tried to resize the filesystm from inside the lxc contianer manually and you cant | |
| 15:30:24 | sean-k-mooney | clould init works in teh lxc continer at startup | |
| 15:30:34 | sean-k-mooney | so i wont be able to do it either | |
| 15:30:51 | donnyd | Is that because of privileges to the underlying block device? | |
| 15:30:57 | sean-k-mooney | yep | |
| 15:31:18 | sean-k-mooney | we dont allow you to acess the block deivec in /dev directly with the permission to adjust the filesystem | |
| 15:31:52 | donnyd | So would it be possible to say start the container privileged, let cloud-init do its thing, and then restart unpriv | |
| 15:32:08 | donnyd | should all happen pretty quickly | |
| 15:32:25 | donnyd | With the understanding that right now, this is just kinda where its at | |
| 15:32:36 | sean-k-mooney | maybe but it might be cleaner to just have nova invoke resize2fs on the blockdevice before starting the container | |
| 15:32:51 | donnyd | sure, that would work too | |
| 15:33:07 | sean-k-mooney | i mean nova already grows the backing image to match the flavor root_disk | |