| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-12-11 | |||
| 15:03:09 | opendevreview | Merged openstack/python-openstackclient master: Improve help strings for tap services https://review.opendev.org/c/openstack/python-openstackclient/+/968546 | |
| 15:50:49 | opendevreview | Merged openstack/python-openstackclient master: Implement conflict resolution https://review.opendev.org/c/openstack/python-openstackclient/+/968178 | |
| 15:50:52 | opendevreview | Merged openstack/python-openstackclient master: Use openstacksdk test generate_fake_resources factory https://review.opendev.org/c/openstack/python-openstackclient/+/966354 | |
| 18:19:21 | opendevreview | Takashi Kajinami proposed openstack/codegenerator master: Drop conflicting target-version https://review.opendev.org/c/openstack/codegenerator/+/970667 | |
| 18:22:07 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Drop redundant target-version option https://review.opendev.org/c/openstack/keystoneauth/+/970668 | |
| 18:23:36 | opendevreview | Takashi Kajinami proposed openstack/osc-lib master: Drop redundant target-version option https://review.opendev.org/c/openstack/osc-lib/+/970669 | |
| 18:33:13 | opendevreview | Stephen Finucane proposed openstack/cliff master: Run mypy from tox https://review.opendev.org/c/openstack/cliff/+/970670 | |
| 18:33:13 | opendevreview | Stephen Finucane proposed openstack/cliff master: Remove use of ABCMeta for formatters https://review.opendev.org/c/openstack/cliff/+/970671 | |
| 18:42:59 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: Run mypy from tox https://review.opendev.org/c/openstack/osc-lib/+/970672 | |
| 18:43:00 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: typing: Use Mapping, not dict https://review.opendev.org/c/openstack/osc-lib/+/970673 | |
| 19:14:22 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: clientmanager: Remove legacy cruft https://review.opendev.org/c/openstack/python-openstackclient/+/970680 | |
| 19:14:23 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add custom command classes https://review.opendev.org/c/openstack/python-openstackclient/+/970681 | |
| 19:14:23 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Add types to custom formatters https://review.opendev.org/c/openstack/python-openstackclient/+/970682 | |
| 19:14:24 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Fixups for typed osc-lib https://review.opendev.org/c/openstack/python-openstackclient/+/970683 | |
| 19:14:24 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Run mypy from tox https://review.opendev.org/c/openstack/python-openstackclient/+/970684 | |
| 19:35:55 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Add v3websso OpenID Connect Web SSO authentication plugin https://review.opendev.org/c/openstack/keystoneauth/+/970328 | |
| 19:35:56 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Fix docs builds on Python 3.14 https://review.opendev.org/c/openstack/keystoneauth/+/970686 | |
| 21:29:11 | opendevreview | Merged openstack/osc-lib master: Run mypy from tox https://review.opendev.org/c/openstack/osc-lib/+/970672 | |
| 21:29:12 | opendevreview | Merged openstack/osc-lib master: typing: Use Mapping, not dict https://review.opendev.org/c/openstack/osc-lib/+/970673 | |
| 21:32:52 | opendevreview | Merged openstack/python-openstackclient master: clientmanager: Remove legacy cruft https://review.opendev.org/c/openstack/python-openstackclient/+/970680 | |
| 21:42:14 | opendevreview | Merged openstack/python-openstackclient master: Add custom command classes https://review.opendev.org/c/openstack/python-openstackclient/+/970681 | |
| 21:47:58 | opendevreview | Merged openstack/cliff master: Run mypy from tox https://review.opendev.org/c/openstack/cliff/+/970670 | |
| 21:56:07 | opendevreview | Merged openstack/python-openstackclient master: typing: Add types to custom formatters https://review.opendev.org/c/openstack/python-openstackclient/+/970682 | |
| 22:21:05 | opendevreview | Merged openstack/cliff master: Remove use of ABCMeta for formatters https://review.opendev.org/c/openstack/cliff/+/970671 | |
| 22:37:20 | M0weng[m] | joelm21: This sounds like a docs problem 🤔 This is the only info I could find about OS_PASSCODE with a cursory search and it only specifies OS_AUTH_METHODS="v3totp" https://docs.openstack.org/python-openstackclient/latest/cli/authentication.html#v3totp | |
| #openstack-sdks - 2025-12-12 | |||
| 10:15:32 | stephenfin | M0weng[m]: I think you may be right, though it's also possible that there's a small bug in keystoneauth. When you don't specify OS_AUTH_TYPE keystoneauth needs to guess, and it does that by looking at available configuration and seeing what matches (so if you provide a password field, it will try password, if you provide application_credential_id, it will try v3applicationcredential etc. There's a chance this detection is | |
| 10:15:32 | stephenfin | broken for token auth, but it's also possible that was never supported so... | |
| 11:25:05 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Remove negotiation for v1 API https://review.opendev.org/c/openstack/python-openstackclient/+/970816 | |
| 11:25:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: common: Remove references to pkg_resources https://review.opendev.org/c/openstack/python-openstackclient/+/970817 | |
| 11:25:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: share: Bootstrap new service https://review.opendev.org/c/openstack/python-openstackclient/+/970818 | |
| 11:25:07 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: share: Import utils https://review.opendev.org/c/openstack/python-openstackclient/+/970819 | |
| 11:25:07 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: share: Import commands (1/n) https://review.opendev.org/c/openstack/python-openstackclient/+/970820 | |
| 11:25:08 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: share: Import commands (2/n) https://review.opendev.org/c/openstack/python-openstackclient/+/970821 | |
| 11:25:08 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: share: Add support for AZs https://review.opendev.org/c/openstack/python-openstackclient/+/970822 | |
| 11:25:09 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Enable in-tree manila support https://review.opendev.org/c/openstack/python-openstackclient/+/970823 | |
| 11:25:16 | stephenfin | carloss: gouthamr: ^ | |
| 11:28:32 | stephenfin | carloss: gouthamr: I have a migration guide put together for the remaining commands. If either of you fancy playing with claude, gemini etc. to automate the remaining migration steps I'd appreciate it https://etherpad.opendev.org/p/manilaclient-to-openstackclient-migration | |
| 11:38:56 | opendevreview | Stephen Finucane proposed openstack/openstackclient master: Migrate to pyproject.toml and ruff https://review.opendev.org/c/openstack/openstackclient/+/970619 | |
| 11:39:31 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: zuul: Make openstackclient-check-plugins voting https://review.opendev.org/c/openstack/python-openstackclient/+/970620 | |
| 11:48:05 | opendevreview | Merged openstack/openstackclient master: Respect conflict resolution https://review.opendev.org/c/openstack/openstackclient/+/970618 | |
| 12:06:02 | opendevreview | Merged openstack/openstackclient master: Migrate to pyproject.toml and ruff https://review.opendev.org/c/openstack/openstackclient/+/970619 | |
| 13:01:12 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Fixups for typed osc-lib https://review.opendev.org/c/openstack/python-openstackclient/+/970683 | |
| 13:01:38 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Run mypy from tox https://review.opendev.org/c/openstack/python-openstackclient/+/970684 | |
| 13:02:27 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Run mypy from tox https://review.opendev.org/c/openstack/python-openstackclient/+/970684 | |
| 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: Run mypy from tox https://review.opendev.org/c/openstack/python-openstackclient/+/970684 | |
| 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: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 | |