| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-08-28 | |||
| 17:34:20 | gtema | but wrt CPO - another param to disable/enable ENVs sounds even more crazy | |
| 17:34:20 | stephenfin | the mistakes of (project) youth | |
| 17:35:02 | gtema | lets see whether the new OSC (;-) can get this addressed | |
| 17:35:35 | andrewbogott_ | Thank you for your thoughts! I will work on a strawman/minimal patch (assuming that's even possible) and then we can see if we want to get more ambitious. | |
| 18:07:45 | opendevreview | Suyeon Cha proposed openstack/openstacksdk master: reset_state, force_delete for Share, ShareSnapshot https://review.opendev.org/c/openstack/openstacksdk/+/888311 | |
| 18:43:26 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | |
| 19:27:37 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add query string params for many APIs https://review.opendev.org/c/openstack/openstacksdk/+/892974 | |
| 19:42:25 | opendevreview | Pavlo Shchelokovskyy proposed openstack/cliff master: Autofit table output if stdout is a tty https://review.opendev.org/c/openstack/cliff/+/892991 | |
| 20:37:50 | chris218 | Hi again so when I try to stop server through conn.compute.start_server(server) it works but when trying to do it with server.start(conn) it errors with AttributeError 'Connection' object has no attribute 'default_microversion' | |
| 21:45:46 | opendevreview | Merged openstack/openstacksdk master: docs: Add missing docs for identity resources https://review.opendev.org/c/openstack/openstacksdk/+/891682 | |
| 21:48:39 | opendevreview | Merged openstack/openstacksdk master: docs: Add missing docs for compute resources https://review.opendev.org/c/openstack/openstacksdk/+/891683 | |
| 21:56:48 | opendevreview | Merged openstack/openstacksdk master: docs: Add missing docs for block storage resources https://review.opendev.org/c/openstack/openstacksdk/+/891684 | |
| 22:15:55 | chris218 | also wanted to note that being able to use OpenAPI spec for openstack sounds supper exciting kudos to you guys :) | |
| 23:15:37 | opendevreview | Merged openstack/openstacksdk master: openstack.format: Remove 'serialize' classmethod https://review.opendev.org/c/openstack/openstacksdk/+/889983 | |
| 23:15:39 | opendevreview | Merged openstack/openstacksdk master: mypy: Address issues with top-level files https://review.opendev.org/c/openstack/openstacksdk/+/889984 | |
| 23:15:41 | opendevreview | Merged openstack/openstacksdk master: Run mypy as pre-commit https://review.opendev.org/c/openstack/openstacksdk/+/889985 | |
| 23:24:36 | opendevreview | Merged openstack/openstacksdk master: block storage: Add query string params for many APIs https://review.opendev.org/c/openstack/openstacksdk/+/892974 | |
| 23:24:38 | opendevreview | Merged openstack/openstacksdk master: compute: Add missing docstrings https://review.opendev.org/c/openstack/openstacksdk/+/891685 | |
| 23:24:40 | opendevreview | Merged openstack/openstacksdk master: block storage: Add missing docstrings https://review.opendev.org/c/openstack/openstacksdk/+/891686 | |
| 23:27:01 | opendevreview | Merged openstack/openstacksdk master: baremetal: Add missing docstrings https://review.opendev.org/c/openstack/openstacksdk/+/891687 | |
| 23:45:03 | chris218 | I figured it out!! needed to pass connection.compute instead of just connection | |
| #openstack-sdks - 2023-08-29 | |||
| 00:21:31 | chris218 | So if somebody could one again walk me through, what do you guys mean by session, client and connection. And when does each get created after I do `with openstack.connection() as conn` or after `conn.compute` gets called or something else? | |
| 00:56:24 | opendevreview | SongSuheon proposed openstack/openstacksdk master: Implement heat 'stack event list' https://review.opendev.org/c/openstack/openstacksdk/+/892836 | |
| 06:14:40 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Fix: Update type encryption operation https://review.opendev.org/c/openstack/openstacksdk/+/893010 | |
| 08:05:09 | whoami-rajat | gtema, hey, replied to your comments on this https://review.opendev.org/c/openstack/python-openstackclient/+/889748 | |
| 08:05:42 | whoami-rajat | unfortunately i found out while making changes that update backup support is not there in SDK, hence we need to keep cinderclient dependency | |
| 08:36:34 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: add new version handling method https://review.opendev.org/c/openstack/openstacksdk/+/891663 | |
| 08:36:35 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | |
| 09:46:09 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: WIP: Adding CLI support for ``glance image-create-via-import`` https://review.opendev.org/c/openstack/python-openstackclient/+/893022 | |
| 09:52:15 | opendevreview | Merged openstack/openstacksdk master: docs: Document various warnings https://review.opendev.org/c/openstack/openstacksdk/+/891688 | |
| 10:31:26 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds command ``image metadef object list`` https://review.opendev.org/c/openstack/python-openstackclient/+/886821 | |
| 10:48:03 | opendevreview | Merged openstack/openstacksdk master: docs: Add exception documentation https://review.opendev.org/c/openstack/openstacksdk/+/891689 | |
| 12:49:22 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds command ``image metadef object create`` https://review.opendev.org/c/openstack/python-openstackclient/+/888765 | |
| 13:01:14 | opendevreview | Pavlo Shchelokovskyy proposed openstack/cliff master: Autofit table output if stdout is a tty https://review.opendev.org/c/openstack/cliff/+/892991 | |
| 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: Remove '_orchestration_client' https://review.opendev.org/c/openstack/openstacksdk/+/884498 | |
| 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: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: Convert 'get_volume_limits' to use proxy layer https://review.opendev.org/c/openstack/openstacksdk/+/889870 | |
| 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: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 | |