| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-23 | |||
| 17:14:20 | KeithMnemonic | ok thanks | |
| 17:17:06 | dansmith | supposedly the SEV thing was resolvable by someone else, but has that happened? | |
| 17:18:24 | KeithMnemonic | someone at SUSE? dansmith | |
| 17:18:29 | mriedem | you mean boris? https://review.opendev.org/#/c/684825/ | |
| 17:18:32 | mriedem | he's also suse | |
| 17:18:46 | KeithMnemonic | he is still on openstack as well | |
| 17:29:05 | openstackgerrit | Merged openstack/nova master: Adds view builders for keypairs controller https://review.opendev.org/347289 | |
| 17:34:19 | dansmith | KeithMnemonic: no, I meant assuming nobody from SUSE is going to fix it, there was an assertion from the libvirty people that they could resolve it too, I just haven't heard that it happened | |
| 17:34:20 | dansmith | efried: ? | |
| 17:35:38 | KeithMnemonic | ok | |
| 18:00:45 | mriedem | dansmith: so i'd be +2 on https://review.opendev.org/#/c/687348/ except, | |
| 18:01:05 | mriedem | i think we should probably drop the dependency on the osc command for now because the horizon thing holding up novaclient 16.0.0 in upper-constraints is not going well | |
| 18:01:27 | mriedem | could use the nova cli in the docs for now and leave a todo in there to replace with osc later | |
| 18:01:52 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341 | |
| 18:01:53 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | |
| 18:01:53 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Merge provider configs to provider tree https://review.opendev.org/676522 | |
| 18:02:08 | dansmith | mriedem: there's no real rush to merge that is there? | |
| 18:02:16 | mriedem | the docs? | |
| 18:02:28 | mriedem | i'd like to merge it sooner than later so i don't lose the context on it | |
| 18:02:35 | mriedem | especially since we have a nova cli for the thing | |
| 18:04:13 | dansmith | okay | |
| 18:10:19 | efried | dansmith: I talked to aspiers a couple weeks ago and he said he was going to try to fix the thing regardless of what SUSE does. | |
| 18:10:42 | efried | but like as a swan song, not any kind of ongoing support. | |
| 18:11:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove fixed sqlalchemy-migrate deprecation warning filters https://review.opendev.org/690704 | |
| 18:11:15 | mriedem | the song....remains the same? | |
| 18:11:19 | efried | but it's also still not out of the question that amd will pony up (or partner with) someone to maintain moving forward. | |
| 18:11:35 | openstackgerrit | Dan Smith proposed openstack/nova master: Add image precaching docs for aggregates https://review.opendev.org/687348 | |
| 18:11:48 | dansmith | mriedem: ^ | |
| 18:11:55 | mriedem | you forgot to remove the depends-on | |
| 18:11:59 | dansmith | oops | |
| 18:12:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Add image precaching docs for aggregates https://review.opendev.org/687348 | |
| 18:13:50 | mriedem | +2 | |
| 18:18:45 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix policy doc for host_status and extended servers attribute https://review.opendev.org/689833 | |
| 18:29:22 | efried | mriedem, dansmith: do we backport docs? | |
| 18:29:33 | openstackgerrit | Merged openstack/nova stable/rocky: Noop CantStartEngineError in targets_cell if API DB not configured https://review.opendev.org/686276 | |
| 18:29:38 | dansmith | probably depends a lot on what it is | |
| 18:29:38 | openstackgerrit | Merged openstack/nova stable/rocky: Handle get_host_availability_zone error during reschedule https://review.opendev.org/686292 | |
| 18:29:45 | openstackgerrit | Merged openstack/nova stable/queens: Explain nested guest support https://review.opendev.org/609790 | |
| 18:29:52 | openstackgerrit | Merged openstack/nova master: Add functional regression test for bug 1849409 https://review.opendev.org/690428 | |
| 18:29:52 | openstack | bug 1849409 in OpenStack Compute (nova) "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem) | |
| 18:30:10 | efried | Leading to... | |
| 18:30:10 | efried | should we consider pushing the pre-precaching parts of what dansmith just wrote in an independent patch that can be backported, and then tacking the precaching bits on top in a separate master-only patch? | |
| 18:32:33 | mriedem | efried: we haven't ever had docs about the image cache so i don't think anyone is chomping at the bit to have those in stable | |
| 18:32:40 | dansmith | I guess I'm not opposed to that, but we have had image caching since forever, no docs since forever, | |
| 18:32:48 | mriedem | we finish each other's | |
| 18:32:51 | dansmith | and the master docs (minus precaching) will be live in google and applicable | |
| 18:32:53 | dansmith | so kinda meh | |
| 18:32:56 | dansmith | mriedem: sentences! | |
| 18:33:00 | dansmith | mriedem: <3 | |
| 18:33:01 | mriedem | teehee! | |
| 18:33:41 | efried | not sure I agree with the philosophy of "nobody ever bitched about it so it must not be worth doing" but okay. | |
| 18:34:12 | mriedem | well, it's docs | |
| 18:34:21 | mriedem | and this is already ready to go | |
| 18:34:54 | efried | is gerrit slow for anyone else rn? | |
| 18:34:54 | dansmith | just doesn't seem that important to have the docs back there.. people will google for this, and find this doc regardless | |
| 18:35:03 | dansmith | it's not like stable gets burned to a cd and shipped to people | |
| 18:35:58 | efried | okay. +W. I'm going to work up the patch to move those conf opts. | |
| 18:36:16 | efried | unless someone is already doing thot. | |
| 18:36:17 | efried | that | |
| 18:36:21 | mriedem | it's all yours | |
| 18:36:56 | mriedem | but first, https://www.youtube.com/watch?v=az8ht2-U0Q0 | |
| 18:37:25 | dustinc | efried: gerrit page loads are fine for me right now | |
| 18:37:38 | efried | thanks. I'll bounce my router then. | |
| 18:38:48 | efried | I don't get it mriedem. You're weird. | |
| 18:39:59 | dansmith | efried: messing with conf options is a fool's errand, is his point | |
| 18:40:07 | dansmith | efried: and stephenfin is the boogieman in this case | |
| 18:40:16 | dansmith | mriedem: did I get that right? | |
| 18:41:20 | mriedem | you're both wrong | |
| 18:41:23 | mriedem | i was just thinking of it | |
| 18:41:28 | dansmith | dammit | |
| 18:41:40 | mriedem | watched that episode the other night with my daughter | |
| 18:41:43 | mriedem | one of my favorites | |
| 18:49:51 | mriedem | lyarwood: probably too late for you today but a couple of small things in your fix https://review.opendev.org/#/c/682622/ | |
| 18:52:02 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/train: Add functional regression test for bug 1849409 https://review.opendev.org/690710 | |
| 18:52:02 | openstack | bug 1849409 in OpenStack Compute (nova) "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem) | |
| 18:52:02 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/train: Fix listing deleted servers with a marker https://review.opendev.org/690711 | |
| 19:03:49 | efried | dansmith, mriedem: ugh, what about [libvirt]remove_unused_resized_minimum_age_seconds? Should it be a) mentioned in the docs, b) moved to [image_cache]? | |
| 19:04:10 | dansmith | is that imagecache related? | |
| 19:04:36 | dansmith | hmm, guess so | |
| 19:04:37 | dansmith | however, | |
| 19:04:44 | dansmith | that looks to be libvirt-specific | |
| 19:04:55 | efried | right | |
| 19:04:58 | efried | ...now | |
| 19:04:59 | dansmith | so I dunno that I'd put that in imagecache | |
| 19:05:57 | dansmith | you wouldn't move libvirt-only things out to DEFAULT just because someone else could implement them in the future, would you? | |
| 19:06:19 | efried | no. But it's definitely related to image cache; and it's weird that there's remove_unused_original_minimum_age_seconds in a *different* place (whether that's [DEFAULT] or [image_cache]) | |
| 19:06:33 | efried | and | |
| 19:06:43 | efried | I wouldn't put up a patch to just do that move | |
| 19:06:54 | efried | but since I'm moving the rest, this seems like the right thing to do in concert. | |
| 19:07:30 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Add functional regression test for bug 1849409 https://review.opendev.org/690715 | |
| 19:07:30 | openstack | bug 1849409 in OpenStack Compute (nova) train "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem) | |
| 19:07:30 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Fix listing deleted servers with a marker https://review.opendev.org/690716 | |
| 19:09:31 | dansmith | efried: but DEFAULT is related to everything, so the same argument for other things would seem to apply | |
| 19:09:48 | mriedem | dansmith: he's moving the image cache stuff from [DEFAULT] to a new [image_cache] group | |
| 19:09:51 | mriedem | to group those like options | |
| 19:10:09 | dansmith | mriedem: the remove_unused is already in the libvirt group | |
| 19:10:17 | mriedem | yeah i know, that's the only one (besides your new one) | |
| 19:10:26 | mriedem | i don't have a strong opinion on moving this one to the new group | |
| 19:10:35 | efried | remove_unused_*original* is in [DEFAULT]; remove_unused_*resized* is in libvirt | |
| 19:11:07 | dansmith | efried: that one is used by others | |