| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-12-12 | |||
| 14:31:41 | opendevreview | Merged openstack/osc-lib master: Drop redundant target-version option https://review.opendev.org/c/openstack/osc-lib/+/970669 | |
| 14:49:34 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Remove functional testenv for py38/py39 https://review.opendev.org/c/openstack/python-openstackclient/+/968107 | |
| 15:03:26 | opendevreview | Merged openstack/python-openstackclient master: volume: Remove negotiation for v1 API https://review.opendev.org/c/openstack/python-openstackclient/+/970816 | |
| 15:03:28 | opendevreview | Merged openstack/python-openstackclient master: common: Remove references to pkg_resources https://review.opendev.org/c/openstack/python-openstackclient/+/970817 | |
| 16:04:28 | opendevreview | Merged openstack/python-openstackclient stable/2024.2: compute: Workaround bug #2089821 https://review.opendev.org/c/openstack/python-openstackclient/+/964166 | |
| 16:04:30 | opendevreview | Merged openstack/python-openstackclient stable/2024.2: tests: Add functional test for adding, removing SGs https://review.opendev.org/c/openstack/python-openstackclient/+/964157 | |
| 17:59:11 | opendevreview | Merged openstack/python-openstackclient master: Remove functional testenv for py38/py39 https://review.opendev.org/c/openstack/python-openstackclient/+/968107 | |
| #openstack-sdks - 2025-12-13 | |||
| 01:03:00 | opendevreview | Merged openstack/openstacksdk master: ruff: Enable W rules https://review.opendev.org/c/openstack/openstacksdk/+/967377 | |
| #openstack-sdks - 2025-12-15 | |||
| 09:00:41 | opendevreview | Jan-Philipp Litza proposed openstack/ansible-collections-openstack master: fix(trunk): Always add relevant sub_ports https://review.opendev.org/c/openstack/ansible-collections-openstack/+/971005 | |
| 09:02:41 | opendevreview | Jan-Philipp Litza proposed openstack/ansible-collections-openstack master: fix(trunk): Always add relevant sub_ports https://review.opendev.org/c/openstack/ansible-collections-openstack/+/971005 | |
| 11:16:54 | stephenfin | gtema: Could you hit https://review.opendev.org/c/openstack/releases/+/970623 and https://review.opendev.org/c/openstack/releases/+/970843 when you have a moment? | |
| 11:31:54 | gtema | Stephenfin, you are release liaison, so my review is not technically required. Anyway I left my vote on both | |
| 11:32:06 | stephenfin | oh yeah | |
| 11:32:14 | stephenfin | 😅 | |
| 11:32:19 | stephenfin | well thank you in any case :) | |
| 11:33:19 | gtema | Wlcm | |
| 11:39:51 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Fixups for typed osc-lib https://review.opendev.org/c/openstack/python-openstackclient/+/970683 | |
| 11:39:52 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: identity: Use plural dest for append opts https://review.opendev.org/c/openstack/python-openstackclient/+/971023 | |
| 11:39:52 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Run mypy from tox https://review.opendev.org/c/openstack/python-openstackclient/+/970684 | |
| 11:39:53 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: taas: Use custom command classes https://review.opendev.org/c/openstack/python-openstackclient/+/971024 | |
| 13:14:39 | opendevreview | Patrick Pfurtscheller proposed openstack/ansible-collections-openstack master: feat: add compute service module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/971040 | |
| 16:06:27 | opendevreview | Patrick Pfurtscheller proposed openstack/ansible-collections-openstack master: feat: add compute service module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/971040 | |
| 16:23:24 | opendevreview | Doug Goldstein proposed openstack/python-openstackclient master: fix(keystone): correct the args submitted on user creation https://review.opendev.org/c/openstack/python-openstackclient/+/971084 | |
| 16:25:41 | cardoe | stephenfin: we conflict on ^ for your osc-lib change just fyi. | |
| 16:47:39 | croelandt | I think the endpoint types in the SDK can be 'public', 'internal' or 'admin'. In the original clients, we had 'publicURL', 'adminURL', 'internalURL'. Am I right? I'm asking because I'm trying to replace uses of glanceclient with the SDK | |
| 17:08:04 | stephenfin | cardoe: that's fine: bug fix takes priority | |
| 17:09:31 | stephenfin | fwiw, I really dislike that aspect of keystone's API | |
| 17:13:09 | cardoe | which part? | |
| 17:19:48 | stephenfin | the non-nullability | |
| 17:20:07 | stephenfin | For POST requests, that is | |
| 17:26:11 | stephenfin | croelandt: tkajinam just did some oslo.limit work on this same thing | |
| 17:29:27 | stephenfin | croelandt: Yes, SDK expects one of public, internal or admin. This is the same as what keystone v3 expects. The URL suffixed variants were a v2.0 API thing | |
| 17:29:40 | stephenfin | https://docs.openstack.org/api-ref/identity/v3/#create-endpoint | |
| 17:29:44 | stephenfin | gtema: can keep me honest though | |
| 17:30:19 | stephenfin | this is also reflected in keystone code itself https://github.com/openstack/keystone/blob/bd178ee91f92f50398cf940b1db3eb2157ed7fd3/keystone/catalog/backends/base.py#L212-L267 | |
| 17:35:18 | gtema | well, interface is an enum and checked by the jsonschema | |
| 17:36:19 | gtema | moreover the db table column is only 8 chars wide | |
| 17:38:18 | stephenfin | ack, so yeah, croelandt: you want the non-suffixed versions | |
| 18:05:41 | tkajinam | https://github.com/openstack/keystoneauth/blob/0e3863577ac153ef02afaac364daf2c967a092f1/keystoneauth1/access/service_catalog.py#L632-L635 | |
| 18:06:38 | tkajinam | afaik keystoneauth has been taken care of the conversion (It removes the URL suffix to accept v2 values for v3) but that's just a hack and we should prefer the ones without URL suffix, I agree | |
| 18:06:52 | tkajinam | (just leaving this note hoping it is kept in logs for future reference | |
| 18:14:47 | croelandt | stephenfin: ok I think somehow the suffixed versions were still the default when configuring some apps, so I'm gonna have to be careful there | |
| 18:51:04 | opendevreview | Merged openstack/python-openstackclient master: identity: Use plural dest for append opts https://review.opendev.org/c/openstack/python-openstackclient/+/971023 | |
| 19:56:35 | opendevreview | Merged openstack/python-openstackclient master: fix(keystone): correct the args submitted on user creation https://review.opendev.org/c/openstack/python-openstackclient/+/971084 | |
| 20:33:19 | opendevreview | Doug Goldstein proposed openstack/python-openstackclient stable/2025.2: fix(keystone): correct the args submitted on user creation https://review.opendev.org/c/openstack/python-openstackclient/+/971125 | |
| #openstack-sdks - 2025-12-16 | |||
| 01:09:01 | cardoe | Test-volume failed cause of backup_id appearing? | |
| 05:51:01 | frickler | actually looks like a new field, see https://review.opendev.org/c/openstack/openstacksdk/+/961253 , but no idea why tests start failing only now | |
| 08:28:41 | opendevreview | Riccardo Pittau proposed openstack/ansible-collections-openstack master: Use new bifrost CI jobs names https://review.opendev.org/c/openstack/ansible-collections-openstack/+/969078 | |
| 11:49:58 | opendevreview | Naoki Hanakawa proposed openstack/ansible-collections-openstack master: Support updating extra_specs in project module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/971172 | |
| 13:15:14 | opendevreview | Merged openstack/keystoneauth master: Drop redundant target-version option https://review.opendev.org/c/openstack/keystoneauth/+/970668 | |
| 13:34:39 | opendevreview | Naoki Hanakawa proposed openstack/ansible-collections-openstack master: Support updating extra_specs in project module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/971172 | |
| 14:16:44 | opendevreview | Merged openstack/keystoneauth master: ruff: Enable S checks https://review.opendev.org/c/openstack/keystoneauth/+/970458 | |
| 14:16:46 | opendevreview | Merged openstack/keystoneauth master: docs: Update note on v2 API https://review.opendev.org/c/openstack/keystoneauth/+/970459 | |
| 14:16:47 | opendevreview | Merged openstack/keystoneauth master: typing: Simplify mypy configuration https://review.opendev.org/c/openstack/keystoneauth/+/970460 | |
| 14:19:04 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Enable logging related ruff checks https://review.opendev.org/c/openstack/keystoneauth/+/970587 | |
| 16:14:53 | opendevreview | Merged openstack/ansible-collections-openstack master: Use new bifrost CI jobs names https://review.opendev.org/c/openstack/ansible-collections-openstack/+/969078 | |
| 18:13:49 | opendevreview | Merged openstack/python-openstackclient master: taas: Use custom command classes https://review.opendev.org/c/openstack/python-openstackclient/+/971024 | |
| 18:13:52 | opendevreview | Merged openstack/python-openstackclient master: typing: Fixups for typed osc-lib https://review.opendev.org/c/openstack/python-openstackclient/+/970683 | |
| 21:06:13 | opendevreview | Doug Goldstein proposed openstack/python-openstackclient stable/2025.2: volume: Add missing backup_id field in tests https://review.opendev.org/c/openstack/python-openstackclient/+/971255 | |
| 21:06:36 | cardoe | frickler: I think we need to backport that otherwise the 2025.2 branch is busted. | |
| #openstack-sdks - 2025-12-17 | |||
| 00:02:59 | cardoe | Yes. That fixes the stable/2025.2 branch. So we need that. | |
| 01:14:11 | opendevreview | Naoki Hanakawa proposed openstack/ansible-collections-openstack master: Support updating extra_specs in project module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/971172 | |
| 16:22:58 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix the version decorators in Manila https://review.opendev.org/c/openstack/codegenerator/+/971352 | |
| 16:46:21 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix the version decorators in Manila https://review.opendev.org/c/openstack/codegenerator/+/971352 | |
| 18:17:11 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix the version decorators in Manila https://review.opendev.org/c/openstack/codegenerator/+/971352 | |
| 19:12:27 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix the version decorators in Manila https://review.opendev.org/c/openstack/codegenerator/+/971352 | |
| 21:39:14 | opendevreview | Merged openstack/python-openstackclient stable/2025.2: volume: Add missing backup_id field in tests https://review.opendev.org/c/openstack/python-openstackclient/+/971255 | |
| #openstack-sdks - 2025-12-18 | |||
| 06:54:28 | opendevreview | Abhishek Kekane proposed openstack/python-openstackclient master: Fix image owner change when accepting membership with --project https://review.opendev.org/c/openstack/python-openstackclient/+/971392 | |
| 08:29:22 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix the version decorators in Manila https://review.opendev.org/c/openstack/codegenerator/+/971352 | |
| 11:56:18 | opendevreview | Stephen Finucane proposed openstack/openstackclient master: Dump check_osc_commands output as yaml https://review.opendev.org/c/openstack/openstackclient/+/971407 | |
| 11:56:18 | opendevreview | Stephen Finucane proposed openstack/openstackclient master: Fix ignored modules check https://review.opendev.org/c/openstack/openstackclient/+/971406 | |
| 12:13:15 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix the version decorators in Manila https://review.opendev.org/c/openstack/codegenerator/+/971352 | |
| 12:17:32 | opendevreview | Merged openstack/openstackclient master: Fix ignored modules check https://review.opendev.org/c/openstack/openstackclient/+/971406 | |
| 12:17:33 | opendevreview | Merged openstack/openstackclient master: Dump check_osc_commands output as yaml https://review.opendev.org/c/openstack/openstackclient/+/971407 | |
| 12:22:37 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Fix image owner change when accepting membership with --project https://review.opendev.org/c/openstack/python-openstackclient/+/971392 | |
| 12:23:12 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Fix image owner change when accepting membership with --project https://review.opendev.org/c/openstack/python-openstackclient/+/971392 | |
| 14:02:15 | opendevreview | Abhishek Kekane proposed openstack/python-openstackclient master: Fix image owner change when accepting membership with --project https://review.opendev.org/c/openstack/python-openstackclient/+/971392 | |
| 16:36:47 | opendevreview | Merged openstack/python-openstackclient master: Fix image owner change when accepting membership with --project https://review.opendev.org/c/openstack/python-openstackclient/+/971392 | |
| 16:57:51 | opendevreview | Merged openstack/codegenerator master: Fix the version decorators in Manila https://review.opendev.org/c/openstack/codegenerator/+/971352 | |
| 18:12:42 | opendevreview | Abhishek Kekane proposed openstack/python-openstackclient stable/2025.2: Fix image owner change when accepting membership with --project https://review.opendev.org/c/openstack/python-openstackclient/+/971462 | |
| 19:09:12 | croelandt | Do you know of projects that use the SDK and run tests? Whenever I try to test something that involves the SDK, I end up pulling... basically everything and have to deal with failure in authentication, the SDK trying to discover all possible services etc. | |
| 19:09:26 | croelandt | There must be an easy way to say "hey, here, this is a fake SDK/Auth/whatever" | |
| 19:09:34 | croelandt | But I'm not aware of it | |
| #openstack-sdks - 2025-12-19 | |||
| 07:27:34 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix neutron qos/policies.rules definition https://review.opendev.org/c/openstack/codegenerator/+/971509 | |
| 08:19:03 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix neutron qos/policies.rules definition https://review.opendev.org/c/openstack/codegenerator/+/971509 | |
| 08:21:42 | opendevreview | Takashi Kajinami proposed openstack/codegenerator master: Drop conflicting target-version https://review.opendev.org/c/openstack/codegenerator/+/970667 | |
| 10:25:12 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add BS.scheduler_stats.pools schema and QP https://review.opendev.org/c/openstack/codegenerator/+/971522 | |
| 11:18:11 | opendevreview | Merged openstack/codegenerator master: Drop conflicting target-version https://review.opendev.org/c/openstack/codegenerator/+/970667 | |
| 11:18:13 | opendevreview | Merged openstack/codegenerator master: Fix neutron qos/policies.rules definition https://review.opendev.org/c/openstack/codegenerator/+/971509 | |
| 12:30:13 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add BS.scheduler_stats.pools schema and QP https://review.opendev.org/c/openstack/codegenerator/+/971522 | |
| 13:39:14 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add BS.scheduler_stats.pools schema and QP https://review.opendev.org/c/openstack/codegenerator/+/971522 | |
| 13:48:36 | opendevreview | OpenStack Release Bot proposed openstack/openstacksdk feature/r1: Update .gitreview for feature/r1 https://review.opendev.org/c/openstack/openstacksdk/+/971542 | |
| 13:48:38 | opendevreview | OpenStack Release Bot proposed openstack/openstacksdk feature/r1: Update TOX_CONSTRAINTS_FILE for feature/r1 https://review.opendev.org/c/openstack/openstacksdk/+/971543 | |
| 13:57:21 | opendevreview | Patrick Pfurtscheller proposed openstack/ansible-collections-openstack master: feat: add compute service module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/971040 | |
| 16:03:36 | croeland1 | Is there no exception for HTTP 413 in the SDK? I'm trying to replace glanceclient's HTTPOverLimit exception | |
| 19:38:24 | opendevreview | Merged openstack/python-openstackclient master: zuul: Make openstackclient-check-plugins voting https://review.opendev.org/c/openstack/python-openstackclient/+/970620 | |
| 19:38:27 | opendevreview | Merged openstack/python-openstackclient stable/2025.2: Fix image owner change when accepting membership with --project https://review.opendev.org/c/openstack/python-openstackclient/+/971462 | |
| 20:04:56 | opendevreview | Merged openstack/codegenerator master: Add BS.scheduler_stats.pools schema and QP https://review.opendev.org/c/openstack/codegenerator/+/971522 | |
| #openstack-sdks - 2025-12-22 | |||
| 12:36:28 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Normalize bs.scheduler-stat naming https://review.opendev.org/c/openstack/codegenerator/+/971618 | |
| 14:54:37 | cardoe | gtema: heh. distractions everywhere my friend. | |