| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-08-29 | |||
| 13:01:15 | opendevreview | Pavlo Shchelokovskyy proposed openstack/cliff master: Fix flake8 violation E721 https://review.opendev.org/c/openstack/cliff/+/893047 | |
| 13:27:56 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove dead method https://review.opendev.org/c/openstack/openstacksdk/+/884500 | |
| 13:27:57 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Convert 'get_volume_limits' to use proxy layer https://review.opendev.org/c/openstack/openstacksdk/+/889870 | |
| 13:27:57 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove '_orchestration_client' https://review.opendev.org/c/openstack/openstacksdk/+/884498 | |
| 13:27:58 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove '_get_raw_client' https://review.opendev.org/c/openstack/openstacksdk/+/889942 | |
| 13:32:49 | opendevreview | Felix Huettner proposed openstack/openstacksdk master: Support passing a subnetpool for create_subnet https://review.opendev.org/c/openstack/openstacksdk/+/868104 | |
| 13:55:20 | opendevreview | Christian Rohmann proposed openstack/openstacksdk master: Fix swift info endpoint url extraction and add unit tests https://review.opendev.org/c/openstack/openstacksdk/+/893062 | |
| 14:52:00 | opendevreview | Christian Rohmann proposed openstack/openstacksdk master: Fix bulk_delete support determination https://review.opendev.org/c/openstack/openstacksdk/+/893071 | |
| 15:00:08 | gtema | stephenfin: can you explain me how Nova wants type: bool to be ["bool", "string"]. What the heck is that? It is about server.live_migrate.block_migration which is boolean OR "auto" | |
| 15:02:09 | stephenfin | Yeah, that's one case | |
| 15:02:12 | stephenfin | there might be more | |
| 15:06:27 | gtema | this is really insane | |
| 15:06:44 | gtema | openapi does not actually allow this, and rightfully | |
| 15:07:09 | gtema | how should you implement this with a strongly typed language | |
| 15:08:47 | gtema | I nearly covered the whole live migration with OpenAPI so that whole mad logic from server (wrt microversion) goes away, but this crap is a beast | |
| 15:09:58 | gtema | I guess this is one more reason to FORCE everybody to start using openapi or anything else similar, so that people stop producing insane things | |
| 15:59:46 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | |
| 16:06:53 | opendevreview | Antonia Gaete proposed openstack/openstacksdk master: WIP metadef property operations command, running into auth issues https://review.opendev.org/c/openstack/openstacksdk/+/893088 | |
| 16:15:23 | opendevreview | Antonia Gaete proposed openstack/openstacksdk master: WIP metadef property operations command, running into auth issues https://review.opendev.org/c/openstack/openstacksdk/+/893088 | |
| 16:19:09 | frickler | can someone remind me where is the cinder/osc command map? I can't find the OSC equivalent of "cinder default-type-set/-unset" | |
| 16:28:40 | frickler | ah, found it, but those commands are missing in the list https://opendev.org/openstack/python-openstackclient/src/branch/master/doc/source/cli/data/cinder.csv | |
| 16:29:06 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove '_orchestration_client' https://review.opendev.org/c/openstack/openstacksdk/+/884498 | |
| 16:29:07 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove '_get_raw_client' https://review.opendev.org/c/openstack/openstacksdk/+/889942 | |
| 16:29:07 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Convert 'get_volume_limits' to use proxy layer https://review.opendev.org/c/openstack/openstacksdk/+/889870 | |
| 16:29:44 | frickler | gtema: do you know if there is an intentional feature gap for admin-only cinder commands, or was this just missed so far? | |
| 16:33:03 | stephenfin | gtema: OpenAPI 3.1 does support it, iirc? | |
| 16:33:29 | stephenfin | since 3.1 is a superset of jsonschema rather than a subset like 3.0 and before | |
| 16:36:07 | stephenfin | https://cswr.github.io/JsonSchema/spec/multiple_types/ | |
| 16:37:01 | stephenfin | gtema: https://paste.opendev.org/show/bifSo2ljyVb5X98iWEIQ/ should do the trick in 3.1, I think? | |
| 16:38:29 | stephenfin | frickler: isn't on line 134? 'volume type list --default'? | |
| 16:38:34 | stephenfin | *isn't it | |
| 16:40:05 | frickler | stephenfin: no, that's a different command, "cinder type-default" | |
| 16:42:37 | frickler | which is a global default, the other commands set the default per project | |
| 17:07:14 | gtema | Stephenfin: something like this failed validation with openapi_core package and 3.1.0 spec used. But I will try few more things. I have workarounded this now differently, but if that would work properly I'd be glad | |
| 17:25:15 | gtema | damn, you are right. It failed on my side but in pydantic while generating spec. Thanks | |
| 17:45:41 | stephenfin | frickler: Ah, in that case I guess it was just never added to the decoder. I would guess that it might have been added to cinderclient after that doc was CSV file was first authored | |
| 17:46:02 | stephenfin | though I do recall updating it some time ago and am surprised I missed it if so | |
| 18:11:13 | frickler | stephenfin: seems it was added in 2020 https://review.opendev.org/c/openstack/python-cinderclient/+/739223 , I'll ping rajat when they are back | |
| 18:19:25 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add 'details' parameter to various 'find' baremetal proxy methods https://review.opendev.org/c/openstack/openstacksdk/+/869370 | |
| 18:19:54 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: baremetal: Decode 'config_drive' argument to 'set_provision_state' https://review.opendev.org/c/openstack/openstacksdk/+/889690 | |
| 18:22:12 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | |
| 18:24:34 | stephenfin | gtema: first user of the fake generator in SDK here btw https://review.opendev.org/c/openstack/python-openstackclient/+/892975 | |
| 18:25:09 | gtema | Great | |
| 18:25:12 | stephenfin | in case you were interested in seeing it in use | |
| 18:26:09 | gtema | Hehe, I know how it is in use, my generator uses this also ;-) | |
| 18:27:30 | gtema | I adapted the complex type is open API for migrations - looks great now | |
| 18:27:57 | gtema | I feel the future behind that initiative | |
| 20:33:27 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Fix: Update/Delete type encryption operation https://review.opendev.org/c/openstack/openstacksdk/+/893010 | |
| #openstack-sdks - 2023-08-30 | |||
| 05:57:53 | opendevreview | Felix Huettner proposed openstack/openstacksdk master: Support passing a subnetpool for create_subnet https://review.opendev.org/c/openstack/openstacksdk/+/868104 | |
| 07:16:32 | opendevreview | Artem Goncharov proposed openstack/os-client-config master: replace "-" with "_" in setup.cfg https://review.opendev.org/c/openstack/os-client-config/+/893132 | |
| 10:49:33 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add 'details' parameter to various 'find' baremetal proxy methods https://review.opendev.org/c/openstack/openstacksdk/+/869370 | |
| 11:33:30 | chris218 | Is there a way to do hard/soft shutdown with OpenStack or it's just not an option because of nova? | |
| 11:39:20 | chris218 | Also when server is in vm_state stopped calling server stop raises Exception is that intended? In theory that should be allowed according to this https://docs.openstack.org/nova/latest/reference/vm-states.html or am I reading that table incorrectly? | |
| 14:11:31 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Fix usage of subnet_id key for router https://review.opendev.org/c/openstack/ansible-collections-openstack/+/893193 | |
| 14:42:30 | opendevreview | Christian Rohmann proposed openstack/openstacksdk master: Fix Swift endpoint conversion to determine info/caps url https://review.opendev.org/c/openstack/openstacksdk/+/893062 | |
| 15:13:02 | opendevreview | Christian Rohmann proposed openstack/openstacksdk master: Fix bulk_delete support determination https://review.opendev.org/c/openstack/openstacksdk/+/893071 | |
| 15:55:15 | stephenfin | gtema: https://docs.google.com/document/d/1dYiNSV9cAsWq1_vB6X5z8vWp6XFU2eUkGfC867iw-4M/edit | |
| 15:55:53 | stephenfin | chris218: where is the exception raised? In SDK (client-side) or nova-compute? | |
| 16:02:25 | gtema | stephenfin - joining the call? | |
| 17:28:20 | opendevreview | Merged openstack/openstacksdk master: cloud: Remove dead method https://review.opendev.org/c/openstack/openstacksdk/+/884500 | |
| 17:28:22 | opendevreview | Merged openstack/openstacksdk master: cloud: Remove '_orchestration_client' https://review.opendev.org/c/openstack/openstacksdk/+/884498 | |
| 17:28:24 | opendevreview | Merged openstack/openstacksdk master: cloud: Remove '_get_raw_client' https://review.opendev.org/c/openstack/openstacksdk/+/889942 | |
| 17:28:26 | opendevreview | Merged openstack/openstacksdk master: cloud: Convert 'get_volume_limits' to use proxy layer https://review.opendev.org/c/openstack/openstacksdk/+/889870 | |
| 17:31:35 | chris218 | stephenfin: on client openstack.exceptions.ConflictException 409 Client error cannot stop instance in vm_state stoppend | |
| 17:32:22 | chris218 | Similar things happen on start when powering on and reboot when device is in state SHUTOFF so I assume it's intended but wanted to make sure | |
| 18:20:47 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: WIP: Implement auto-code generation https://review.opendev.org/c/openstack/openstacksdk/+/882326 | |
| 18:21:17 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: utils: Don't warn for partial formatter classes https://review.opendev.org/c/openstack/osc-lib/+/893239 | |
| 18:33:46 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: Fix pre-commit issues https://review.opendev.org/c/openstack/osc-lib/+/893261 | |
| 18:33:46 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: Add pre-commit https://review.opendev.org/c/openstack/osc-lib/+/893262 | |
| 18:33:47 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: Blacken code https://review.opendev.org/c/openstack/osc-lib/+/893263 | |
| 18:33:47 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: pre-commit: Integrate black https://review.opendev.org/c/openstack/osc-lib/+/893264 | |
| 18:33:48 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: Ignore black changes https://review.opendev.org/c/openstack/osc-lib/+/893265 | |
| 18:33:48 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: tox: Use pre-commit for style checks https://review.opendev.org/c/openstack/osc-lib/+/893266 | |
| 18:33:49 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: Remove lower-constraints.txt https://review.opendev.org/c/openstack/osc-lib/+/893267 | |
| 19:28:15 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Migrate volume type commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/893277 | |
| 20:12:52 | opendevreview | Andrew Bogott proposed openstack/openstacksdk master: config loader: Prefer cli/env over clouds.yaml args for some args https://review.opendev.org/c/openstack/openstacksdk/+/893283 | |
| 20:52:58 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume attachment support https://review.opendev.org/c/openstack/openstacksdk/+/884167 | |
| #openstack-sdks - 2023-08-31 | |||
| 04:51:15 | opendevreview | YeJun, Jung proposed openstack/openstacksdk master: Add support for other metadef property operations https://review.opendev.org/c/openstack/openstacksdk/+/857727 | |
| 05:01:07 | opendevreview | YeJun, Jung proposed openstack/openstacksdk master: Add support for other metadef property operations https://review.opendev.org/c/openstack/openstacksdk/+/857727 | |
| 06:19:35 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds command ``image metadef object delete`` https://review.opendev.org/c/openstack/python-openstackclient/+/893310 | |
| 07:30:38 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | |
| 07:34:10 | opendevreview | YeJun, Jung proposed openstack/python-openstackclient master: Add image metadef property command https://review.opendev.org/c/openstack/python-openstackclient/+/879228 | |
| 07:47:32 | opendevreview | YeJun, Jung proposed openstack/python-openstackclient master: Add image metadef property command https://review.opendev.org/c/openstack/python-openstackclient/+/879228 | |
| 08:00:57 | opendevreview | YeJun, Jung proposed openstack/openstacksdk master: Add support for other metadef property operations https://review.opendev.org/c/openstack/openstacksdk/+/857727 | |
| 08:45:33 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Fix ReleaseNotes build https://review.opendev.org/c/openstack/python-openstackclient/+/893316 | |
| 09:27:30 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Propocol release versions properly https://review.opendev.org/c/openstack/python-openstackclient/+/893343 | |
| 09:34:22 | frickler | gtema: I'm also worried by the "Current release notes" showing everything back to 3.0.0, but it looks like that is not a new issue https://docs.openstack.org/releasenotes/python-openstackclient/unreleased.html | |
| 09:34:49 | gtema | yeah, agree it looks weird | |
| 09:39:35 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Propocol release versions properly https://review.opendev.org/c/openstack/python-openstackclient/+/893343 | |
| 09:42:29 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Propocol release versions properly https://review.opendev.org/c/openstack/python-openstackclient/+/893343 | |
| 09:43:02 | gtema | frickler - I updated that in 3rd patchset | |
| 09:44:43 | frickler | gtema: oh, that's hardcoded. that's an easy fix then, thx. I was expecting something going bonkers in the automatic tag detection | |
| 09:45:10 | gtema | well, I think on the OSC side we have like a workaround for that | |
| 09:46:01 | gtema | my expectation would be also that on the "unreleased" you show everything what is not tagged yet, but it somehow not working properly. Explicitly complex it becomes when for certain OpenStack release you have few OSC releases | |
| 09:46:28 | gtema | so what should this page be showing: not released at all, or what is going into the next OpenStack release | |
| 09:47:31 | gtema | basically for the 1st you drop "earliest-version" as such, while for the later you set it to last release of prev OpenStack release | |
| 09:47:46 | frickler | gtema: ah, yes, makes sense to hardcode this way, then. just need to remember to update after each openstack release then | |
| 09:47:56 | gtema | right | |
| 09:49:04 | frickler | gtema: btw. the unreleased stuff is in there, just with a pbr-generated fake tag, like currently the "6.2.0-57" | |
| 09:51:03 | gtema | yes, this is for the 1st option - not yet tagged is without "earliest-version" option | |