| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-23 | |||
| 17:03:11 | mriedem | osc isn't looking for a deleted marker | |
| 17:03:44 | KeithMnemonic | it is osc, you can not do a show on a deleted sever, you get the same curl and same error | |
| 17:03:45 | mriedem | https://192.168.23.35:8774/v2.1/428982d4248a419a933668b6a4dd14a0/servers?name=c244e139-98fe-44eb-a971-c067954da026 means osc is first looking to see if the marker server is given on the command line by name | |
| 17:03:45 | melwitt | I mean, when you pass --deleted doesn't it just send deleted=True regardless? or no | |
| 17:03:50 | melwitt | oh | |
| 17:03:52 | mriedem | it's not, it gets a 404 and then tries by id, | |
| 17:04:08 | mriedem | which returns a 404 because the server is deleted and you're not including the ?deleted=true filter param | |
| 17:04:13 | KeithMnemonic | correct | |
| 17:04:23 | KeithMnemonic | but osc is called with --deleted | |
| 17:04:24 | mriedem | so it's a shell bug | |
| 17:04:30 | KeithMnemonic | it is just not passed to the api call | |
| 17:04:35 | mriedem | right, it's a bug | |
| 17:04:42 | KeithMnemonic | ok against osc | |
| 17:04:44 | mriedem | yes | |
| 17:05:17 | KeithMnemonic | i can file another today. i will reference this discussion and some debug output. | |
| 17:05:20 | KeithMnemonic | is that ok | |
| 17:05:31 | mriedem | sure | |
| 17:05:36 | mriedem | that goes against osc in storyboard though | |
| 17:05:52 | mriedem | where it will languish for 1000 years | |
| 17:06:26 | KeithMnemonic | ok let me test on something newer to see if it still exists is train for example | |
| 17:06:34 | mriedem | that code i linked above is train | |
| 17:06:38 | mriedem | so it's there | |
| 17:07:35 | mriedem | https://github.com/openstack/osc-lib/blob/1.14.1/osc_lib/utils/__init__.py#L169 | |
| 17:07:49 | mriedem | looks like the fix is relatively simple: | |
| 17:08:12 | mriedem | utils.find_resource(compute_client.servers, parsed_args.marker, deleted=parsed_args.deleted) | |
| 17:10:48 | mriedem | KeithMnemonic: feel free to have suse send me a check for the openstack support anytime :) | |
| 17:12:31 | KeithMnemonic | you know that is a sore point these days | |
| 17:13:07 | dansmith | speaking of that, | |
| 17:13:22 | dansmith | did aspiers ever show up and/or has anyone else been able to resolve the SEV issue we released with train? | |
| 17:13:45 | KeithMnemonic | AFAIK, aspiers is till with SUSE but not sure in what capacity | |
| 17:14:10 | KeithMnemonic | mriedem since that fix is simple, can i take a stab at it? | |
| 17:14:16 | mriedem | KeithMnemonic: go nuts | |
| 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: Merge provider configs to provider tree https://review.opendev.org/676522 | |
| 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: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 | openstackgerrit | Merged openstack/nova stable/rocky: Handle get_host_availability_zone error during reschedule https://review.opendev.org/686292 | |
| 18:29:38 | dansmith | probably depends a lot on what it is | |
| 18:29:45 | openstackgerrit | Merged openstack/nova stable/queens: Explain nested guest support https://review.opendev.org/609790 | |
| 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:29:52 | openstackgerrit | Merged openstack/nova master: Add functional regression test for bug 1849409 https://review.opendev.org/690428 | |
| 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:30:10 | efried | Leading to... | |
| 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 | dansmith | just doesn't seem that important to have the docs back there.. people will google for this, and find this doc regardless | |
| 18:34:54 | efried | is gerrit slow for anyone else rn? | |
| 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 | |