| opendevreview | Merged openstack/python-openstackclient stable/2024.2: [CI][stable-only] Fix tox-docs job and remove osc-upload-image https://review.opendev.org/c/openstack/python-openstackclient/+/986122 | 09:36 |
|---|---|---|
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Respect MV based responses https://review.opendev.org/c/openstack/codegenerator/+/985819 | 09:45 |
| opendevreview | Doug Goldstein proposed openstack/osc-lib master: fix: unbound variable when both values are false https://review.opendev.org/c/openstack/osc-lib/+/986242 | 10:19 |
| opendevreview | Joan Gilabert proposed openstack/openstacksdk master: Read connect_retry_delay in get_session_client https://review.opendev.org/c/openstack/openstacksdk/+/986350 | 11:00 |
| elodilles | hi, just a heads-up, that i've pushed a patch that adds python-openstackclient's Feature Freeze to 2026.2 Hibiscus schedule: https://review.opendev.org/c/openstack/releases/+/986358 | 12:29 |
| cardoe | stephenfin: what's osc-lib used for vs openstacksdk and python-openstackclient? | 14:05 |
| opendevreview | Doug Goldstein proposed openstack/osc-lib master: typing: union-attrs on find_project() https://review.opendev.org/c/openstack/osc-lib/+/986248 | 14:34 |
| opendevreview | Doug Goldstein proposed openstack/osc-lib master: fix: unbound variable when both values are false https://review.opendev.org/c/openstack/osc-lib/+/986242 | 14:34 |
| cardoe | Just trying to fix these bugs I noticed and just getting caught up fixing what's failing even before I fix the bug I'm trying to fix. | 14:35 |
| stephenfin | cardoe: it's used for any OSC plugins (python-manilaclient, python-barbicanclient, etc.) | 15:11 |
| opendevreview | Stephen Finucane proposed openstack/osc-lib master: Update packaging configuration https://review.opendev.org/c/openstack/osc-lib/+/979534 | 15:12 |
| stephenfin | probably ironicclient too? | 15:13 |
| stephenfin | I'd like to get rid of it and fold the bulk of the clients into OSC proper, but that's not entirely possible (like ironicclient) | 15:14 |
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Respect MV based responses https://review.opendev.org/c/openstack/codegenerator/+/985819 | 15:15 |
| cardoe | I'd be down for moving the OSC parts of ironicclient in somehow. | 15:21 |
| cardoe | There's just a group of Ironic folks that don't do anything else OpenStack and they wanna run it without the "openstack" prefix. | 15:22 |
| cardoe | If you've got an idea how we can make that happen for them then I'm game. | 15:22 |
| cardoe | I just wanna make progress. | 15:22 |
| cardoe | It's death by 1000 -1's over theoretical issues or imperfect ideals when we're attempting to solve actual issues. | 15:23 |
| stephenfin | It's doable but I just don't know how yet :) It's a packaging issue but we don't currently have a way to distribute multiple packages from a single repo. One I'll continue thinking about | 15:25 |
| stephenfin | We shouldn't be seeing issues like those osc-lib issues regularly though. We're seeing them now because the typing situation in SDK has been changing so rapidly | 15:25 |
| stephenfin | In this case, we added typing to the proxies. That allows e.g. tab-completion (good) but necessitates narrowing for services with multiple API major versions and therefore multiple proxies (not good) | 15:26 |
| cardoe | We should have the ability to ship multiple packages from one repo. | 15:48 |
| cardoe | stephenfin: what would ya like me to change in https://review.opendev.org/c/openstack/python-openstackclient/+/986239 ? | 16:21 |
| cardoe | Right now the "is_storage_endpoint_enabled()" doesn't check endpoints... it checks services... and it checks a list of services. | 16:22 |
| cardoe | So I just made "is_storage_endpoint_available()" which does check endpoints using the same list of services. | 16:22 |
| cardoe | The avoids an ugly Exception thrown at the CLI when messing with quotas and limits. | 16:22 |
| cardoe | The other stuff is just bonus. | 16:23 |
| stephenfin | cardoe: Sorry, that was a half-baked comment. I haven't had a chance to do the research and suggest an actual solution yet | 16:30 |
| stephenfin | What I was getting at is that if we're using a command that depends on cinder v3, then finding an endpoint with service type of `volumev2` doesn't help us | 16:31 |
| stephenfin | But I can't recall of the top of my head how OSC decides whether to load e.g. cinder-related commands from `openstackclient/volume/v2` or from `.../v3` | 16:31 |
| stephenfin | *off the top | 16:31 |
| stephenfin | So maybe it's a irrelevant comment because we do a version check later, idk | 16:32 |
| cardoe | So every where the actual client is checked later on but this is more like early checks to say "hey should we even have storage or network or compute options" | 16:32 |
| stephenfin | Also, we have quite a few "is this service/version supported" utilities scattered across osc-lib and openstacksdk and I'd be surprised if there wasn't one already existing that fit your purpose (though it's of course possible). I just haven't got to research that today and find out | 16:33 |
| stephenfin | cardoe: yeah, this is primarily for e.g. the quota commands, I'm guessing | 16:33 |
| stephenfin | If you wanna take a look at those questions (or throw e.g. claude at it), please be my guest. Otherwise, if this isn't super urgent (as in *this week*), I'll get a look at it later this week and try give you a more robust bit of feedback :D | 16:36 |
| * stephenfin needs to run for rugby training | 16:37 | |
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Respect MV based responses https://review.opendev.org/c/openstack/codegenerator/+/985819 | 16:58 |
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: WIP: Respect MV based responses https://review.opendev.org/c/openstack/codegenerator/+/985819 | 17:42 |
| M0weng[m] | gtema: Should we cancel this week's JsonSchema/API meeting for Labour Day/Workers' Day? | 19:41 |
| gtema | Hey Oweng, yes please. I figured out that fact myself only today | 19:52 |
| opendevreview | Dylan McCulloch proposed openstack/python-openstackclient master: Fix user password set command https://review.opendev.org/c/openstack/python-openstackclient/+/980430 | 21:30 |
| M0weng[m] | Oh right I think @carloss is in charge of the meeting calendar event | 22:07 |
| carloss | M0weng[m]: on it :) | 22:58 |
| M0weng[m] | Thank you! :) | 22:59 |
| opendevreview | Dylan McCulloch proposed openstack/python-openstackclient master: Fix user password set command https://review.opendev.org/c/openstack/python-openstackclient/+/980430 | 23:47 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!