| 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:21 |
|---|---|---|
| opendevreview | SongSuheon proposed openstack/openstacksdk master: Implement heat 'stack event list' https://review.opendev.org/c/openstack/openstacksdk/+/892836 | 00:56 |
| opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Fix: Update type encryption operation https://review.opendev.org/c/openstack/openstacksdk/+/893010 | 06:14 |
| whoami-rajat | gtema, hey, replied to your comments on this https://review.opendev.org/c/openstack/python-openstackclient/+/889748 | 08:05 |
| 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:05 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: add new version handling method https://review.opendev.org/c/openstack/openstacksdk/+/891663 | 08:36 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | 08:36 |
| 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:46 |
| opendevreview | Merged openstack/openstacksdk master: docs: Document various warnings https://review.opendev.org/c/openstack/openstacksdk/+/891688 | 09:52 |
| opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds command ``image metadef object list`` https://review.opendev.org/c/openstack/python-openstackclient/+/886821 | 10:31 |
| opendevreview | Merged openstack/openstacksdk master: docs: Add exception documentation https://review.opendev.org/c/openstack/openstacksdk/+/891689 | 10:48 |
| opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds command ``image metadef object create`` https://review.opendev.org/c/openstack/python-openstackclient/+/888765 | 12:49 |
| *** ozzzo1 is now known as ozzzo | 12:57 | |
| 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 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/cliff master: Fix flake8 violation E721 https://review.opendev.org/c/openstack/cliff/+/893047 | 13:01 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove dead method https://review.opendev.org/c/openstack/openstacksdk/+/884500 | 13:27 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove '_orchestration_client' https://review.opendev.org/c/openstack/openstacksdk/+/884498 | 13:27 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove '_get_raw_client' https://review.opendev.org/c/openstack/openstacksdk/+/889942 | 13:27 |
| opendevreview | Felix Huettner proposed openstack/openstacksdk master: Support passing a subnetpool for create_subnet https://review.opendev.org/c/openstack/openstacksdk/+/868104 | 13:32 |
| 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 | 13:55 |
| opendevreview | Christian Rohmann proposed openstack/openstacksdk master: Fix bulk_delete support determination https://review.opendev.org/c/openstack/openstacksdk/+/893071 | 14:52 |
| 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:00 |
| stephenfin | Yeah, that's one case | 15:02 |
| stephenfin | there might be more | 15:02 |
| gtema | this is really insane | 15:06 |
| gtema | openapi does not actually allow this, and rightfully | 15:06 |
| gtema | how should you implement this with a strongly typed language | 15:07 |
| 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:08 |
| 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:09 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | 15:59 |
| 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:06 |
| 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 |
| 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:19 |
| 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:28 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove '_orchestration_client' https://review.opendev.org/c/openstack/openstacksdk/+/884498 | 16:29 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove '_get_raw_client' https://review.opendev.org/c/openstack/openstacksdk/+/889942 | 16:29 |
| 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 |
| 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:29 |
| stephenfin | gtema: OpenAPI 3.1 does support it, iirc? | 16:33 |
| stephenfin | since 3.1 is a superset of jsonschema rather than a subset like 3.0 and before | 16:33 |
| stephenfin | https://cswr.github.io/JsonSchema/spec/multiple_types/ | 16:36 |
| stephenfin | gtema: https://paste.opendev.org/show/bifSo2ljyVb5X98iWEIQ/ should do the trick in 3.1, I think? | 16:37 |
| stephenfin | frickler: isn't on line 134? 'volume type list --default'? | 16:38 |
| stephenfin | *isn't it | 16:38 |
| frickler | stephenfin: no, that's a different command, "cinder type-default" | 16:40 |
| frickler | which is a global default, the other commands set the default per project | 16:42 |
| 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:07 |
| gtema | damn, you are right. It failed on my side but in pydantic while generating spec. Thanks | 17:25 |
| 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:45 |
| stephenfin | though I do recall updating it some time ago and am surprised I missed it if so | 17:46 |
| 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:11 |
| 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 |
| 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:19 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | 18:22 |
| stephenfin | gtema: first user of the fake generator in SDK here btw https://review.opendev.org/c/openstack/python-openstackclient/+/892975 | 18:24 |
| gtema | Great | 18:25 |
| stephenfin | in case you were interested in seeing it in use | 18:25 |
| gtema | Hehe, I know how it is in use, my generator uses this also ;-) | 18:26 |
| gtema | I adapted the complex type is open API for migrations - looks great now | 18:27 |
| gtema | I feel the future behind that initiative | 18:27 |
| *** melwitt_ is now known as melwitt | 18:34 | |
| opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Fix: Update/Delete type encryption operation https://review.opendev.org/c/openstack/openstacksdk/+/893010 | 20:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!