| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2026-01-05 | |||
| 19:20:45 | opendevreview | Callum Dickinson proposed openstack/openstacksdk master: Do not use the global Prometheus collector registry https://review.opendev.org/c/openstack/openstacksdk/+/972237 | |
| 20:17:24 | opendevreview | Callum Dickinson proposed openstack/openstacksdk master: Do not use the global Prometheus collector registry https://review.opendev.org/c/openstack/openstacksdk/+/972237 | |
| 21:03:30 | opendevreview | Callum Dickinson proposed openstack/openstacksdk master: Do not use the global Prometheus collector registry https://review.opendev.org/c/openstack/openstacksdk/+/972237 | |
| 21:08:54 | opendevreview | Callum Dickinson proposed openstack/openstacksdk master: Do not use the global Prometheus collector registry https://review.opendev.org/c/openstack/openstacksdk/+/972237 | |
| 21:45:25 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Only clear out security groups if the parameter is set https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972247 | |
| #openstack-sdks - 2026-01-06 | |||
| 00:27:58 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Only clear out security groups if the parameter is set https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972247 | |
| 09:49:50 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Only clear out security groups if the parameter is set https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972247 | |
| 10:40:04 | stephenfin | gtema: Would appreciate your thoughts on https://review.opendev.org/c/openstack/openstacksdk/+/972237 at some point, just because you're more familiar with that stuff than I am | |
| 11:21:13 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient stable/2025.1: Set NEUTRON_DEPLOY_MOD_WSGI to true for func jobs https://review.opendev.org/c/openstack/python-openstackclient/+/972272 | |
| 12:52:13 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Only clear out security groups if the parameter is set https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972247 | |
| 14:15:55 | opendevreview | Merged openstack/openstacksdk master: Do not use the global Prometheus collector registry https://review.opendev.org/c/openstack/openstacksdk/+/972237 | |
| 18:34:59 | Callum027 | stephenfin: Thanks for reviewing 972237. Do you think it would be worth backporting to supported releases? IIRC the person who reported it is running 2024.1 so they would still be experiencing it even if they upgraded to 2025.1 | |
| 18:39:03 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Only clear out security groups if the parameter is set https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972247 | |
| 18:41:54 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Only clear out security groups if the parameter is set https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972247 | |
| 19:03:21 | M0weng[m] | croelandt: croelandt: python-openstackclient has unit and functional tests, not sure if that would fit what you are looking for? | |
| 19:16:18 | croelandt | M0weng[m]: I need to remember what I asked :D | |
| 19:16:45 | M0weng[m] | :) | |
| 19:17:03 | M0weng[m] | > But I'm not aware of it | |
| 19:17:03 | M0weng[m] | > There must be an easy way to say "hey, here, this is a fake SDK/Auth/whatever" | |
| 19:17:03 | M0weng[m] | > 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:17:54 | croelandt | yeah so I could always copy what the openstackclient does | |
| 19:18:10 | croelandt | I just wish there were some kind of "fake SDK":D | |
| 19:18:15 | croelandt | also I'm bad at testing | |
| 19:18:38 | croelandt | and especially in OpenStack I always feel like I'm drowning in mocks/fixtures/fake stuff and not really testing real code :D | |
| 19:22:58 | M0weng[m] | Yep I get that feeling too, there are so many fakes | |
| 19:26:31 | M0weng[m] | The openstackclient unit tests do use fakes instead of the actual SDK I believe, for example see openstackclient/tests/unit/identity/v3/test_user.py | |
| 19:27:04 | M0weng[m] | They are imported from openstack.test.fakes | |
| 19:31:24 | croelandt | oh you can generate fake resources | |
| 21:33:02 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| 21:39:54 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| 22:57:43 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| 23:07:55 | opendevreview | Ivan Anfimov proposed openstack/keystoneauth master: Remove tags from README https://review.opendev.org/c/openstack/keystoneauth/+/972415 | |
| 23:09:18 | opendevreview | Ivan Anfimov proposed openstack/keystoneauth master: Remove tags from README https://review.opendev.org/c/openstack/keystoneauth/+/972415 | |
| 23:42:22 | opendevreview | Ivan Anfimov proposed openstack/keystoneauth master: Remove tags from README https://review.opendev.org/c/openstack/keystoneauth/+/972415 | |
| #openstack-sdks - 2026-01-07 | |||
| 10:18:18 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Run mypy from tox https://review.opendev.org/c/openstack/keystoneauth/+/970461 | |
| 10:18:23 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: WIP: typing: Add hints to fixtures https://review.opendev.org/c/openstack/keystoneauth/+/970462 | |
| 11:45:01 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Only clear out security groups if the parameter is set https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972247 | |
| 11:51:44 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Add bootable volume support https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972458 | |
| 11:59:15 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Run mypy from tox https://review.opendev.org/c/openstack/keystoneauth/+/970461 | |
| 11:59:16 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: WIP: typing: Add hints to fixtures https://review.opendev.org/c/openstack/keystoneauth/+/970462 | |
| 12:30:16 | opendevreview | Andriy Kurilin proposed openstack/python-openstackclient master: Fix quota usage and reservation display https://review.opendev.org/c/openstack/python-openstackclient/+/972468 | |
| 14:45:57 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Add support for NDP proxies https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972496 | |
| 15:17:12 | opendevreview | James Hewitt proposed openstack/ansible-collections-openstack master: Only clear out security groups if the parameter is set https://review.opendev.org/c/openstack/ansible-collections-openstack/+/972247 | |
| 15:40:34 | opendevreview | Merged openstack/python-openstackclient master: Fix quota usage and reservation display https://review.opendev.org/c/openstack/python-openstackclient/+/972468 | |
| #openstack-sdks - 2026-01-08 | |||
| 01:02:45 | joelm21 | Thanks M0weng[m] and stephenfin, appreciate the responses. POC of the method I'm using for the client, if you are interested. | |
| 01:02:46 | joelm21 | https://github.com/micron21/totp-capable-openstack-client-for-mcloud/ | |
| 01:11:17 | joelm21 | I do note that keystone supports an "Expires at", according to the source. | |
| 01:11:19 | joelm21 | https://docs.openstack.org/keystone/latest/_modules/keystone/token/provider.html#Manager.issue_token | |
| 01:13:03 | joelm21 | I just can't see anything in the openstackclient source that might parse that parameter, i.e. openstack token issue --expires-at YYYY-MM-DD:HH:MM:SS | |
| 01:20:49 | joelm21 | (that would be because it doesn't) | |
| 01:20:50 | joelm21 | usage: openstack token issue [-h] [-f {json,shell,table,value,yaml}] [-c COLUMN] [--noindent] [--prefix PREFIX] [--max-width <integer>] [--fit-width] [--print-empty]openstack token issue: error: unrecognized arguments: --expires-at 2026-01-08:00:00:00 | |
| 01:22:53 | joelm21 | as far as I can tell, expires_at is supported by the model: https://opendev.org/openstack/keystone/src/commit/a903b505429410435aab4c96db6e89413929fcf3/keystone/token/provider.py#L272 | |
| 01:29:00 | joelm21 | it looks like there is no supported mechanism to specify an expiry time for a token generated by the openstack client; https://github.com/openstack/python-openstackclient/blob/0a937332932b2a0263b12d307026fee43d125418/openstackclient/identity/v3/token.py#L196 looks like it generates one that is used, and cannot be modified. | |
| #openstack-sdks - 2026-01-09 | |||
| 13:59:32 | opendevreview | Stephen Finucane proposed openstack/os-api-ref master: Defer log arg parsing https://review.opendev.org/c/openstack/os-api-ref/+/972875 | |
| 13:59:33 | opendevreview | Stephen Finucane proposed openstack/os-api-ref master: Add ruff https://review.opendev.org/c/openstack/os-api-ref/+/972876 | |
| 13:59:34 | opendevreview | Stephen Finucane proposed openstack/os-api-ref master: Add typing https://review.opendev.org/c/openstack/os-api-ref/+/972877 | |
| 14:42:13 | opendevreview | Merged openstack/os-api-ref master: Defer log arg parsing https://review.opendev.org/c/openstack/os-api-ref/+/972875 | |
| 15:07:57 | opendevreview | Merged openstack/os-api-ref master: Add ruff https://review.opendev.org/c/openstack/os-api-ref/+/972876 | |
| 15:08:00 | opendevreview | Merged openstack/os-api-ref master: Add typing https://review.opendev.org/c/openstack/os-api-ref/+/972877 | |
| #openstack-sdks - 2026-01-10 | |||
| 07:39:23 | frickler | gtema: stephenfin: https://review.opendev.org/c/openstack/python-openstackclient/+/972272 would unblock the stable/2025.1 branch until neutron fixes the cited bug | |
| #openstack-sdks - 2026-01-12 | |||
| 16:50:55 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Try to improve handling for '{}' schemas https://review.opendev.org/c/openstack/codegenerator/+/973122 | |
| #openstack-sdks - 2026-01-13 | |||
| 15:03:12 | opendevreview | Jacob Anders proposed openstack/openstacksdk master: Add node health field for Redfish health monitoring https://review.opendev.org/c/openstack/openstacksdk/+/973264 | |
| 18:31:45 | opendevreview | cid proposed openstack/openstacksdk master: Add support for filtering by `conductor_groups` https://review.opendev.org/c/openstack/openstacksdk/+/973287 | |
| 21:51:35 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| 23:22:20 | opendevreview | Doug Goldstein proposed openstack/openstacksdk master: feat: add missing baremetal port fields https://review.opendev.org/c/openstack/openstacksdk/+/973321 | |
| 23:24:57 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/936279 | |
| 23:30:22 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: WIP: Identity: Add --project-domain option for limits https://review.opendev.org/c/openstack/python-openstackclient/+/973325 | |
| #openstack-sdks - 2026-01-14 | |||
| 07:18:29 | opendevreview | Inyong Hong proposed openstack/python-openstackclient master: mig with cluster https://review.opendev.org/c/openstack/python-openstackclient/+/973349 | |
| 07:20:43 | opendevreview | Inyong Hong proposed openstack/python-openstackclient master: Add --cluster option to volume migration https://review.opendev.org/c/openstack/python-openstackclient/+/973349 | |
| 11:16:40 | opendevreview | Inyong Hong proposed openstack/python-openstackclient master: Add --cluster option to volume migration https://review.opendev.org/c/openstack/python-openstackclient/+/973349 | |
| 11:35:52 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Try to improve handling for '{}' schemas https://review.opendev.org/c/openstack/codegenerator/+/973122 | |
| 15:35:45 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Try to improve handling for '{}' schemas https://review.opendev.org/c/openstack/codegenerator/+/973122 | |
| 18:26:45 | opendevreview | Merged openstack/codegenerator master: Try to improve handling for '{}' schemas https://review.opendev.org/c/openstack/codegenerator/+/973122 | |
| 18:58:50 | JayF | https://review.opendev.org/c/openstack/openstacksdk/+/955103 has 2x+2 from two ironic cores, if an SDK core gets a chance | |
| 18:59:25 | JayF | https://review.opendev.org/c/openstack/openstacksdk/+/973287 only 1x+2 but trivial | |
| #openstack-sdks - 2026-01-15 | |||
| 00:35:10 | opendevreview | Inyong Hong proposed openstack/python-openstackclient master: Add --cluster option to volume migration https://review.opendev.org/c/openstack/python-openstackclient/+/973349 | |
| 09:43:25 | frickler | gtema: fyi the secret for osc-image-upload needs rotation, not sure if you have access for that or where this comes from https://review.opendev.org/c/openstack/python-openstackclient/+/962778 | |
| 09:44:26 | gtema | @fricker: - no, I do not | |
| 09:59:22 | frickler | gtema: looks like mordred pushed this in 2000 https://review.opendev.org/714502, so you can either try to get access from him (still around in #opendev afaict), or drop that secret and job | |
| 10:00:32 | gtema | I personally do not rely on this job. The question is rather - is someone using that? | |
| 10:00:48 | gtema | wasn't the opendev itself using it? | |
| 10:01:29 | gtema | I barely remember seeing that (or similar) somewhere in the bridge provisioning playbooks | |
| 10:06:35 | frickler | I'm not aware of that, I'll ask in the #opendev channel | |
| 15:33:43 | opendevreview | Cyril Roelandt proposed openstack/openstacksdk master: image: Add support for downloading from a specific store https://review.opendev.org/c/openstack/openstacksdk/+/969839 | |
| #openstack-sdks - 2026-01-16 | |||
| 05:11:06 | opendevreview | Pranali Deore proposed openstack/openstacksdk master: Adding support for new add/get location apis https://review.opendev.org/c/openstack/openstacksdk/+/883271 | |
| 10:13:12 | opendevreview | Pranali Deore proposed openstack/openstacksdk master: Adding support for new add/get location apis https://review.opendev.org/c/openstack/openstacksdk/+/883271 | |
| 12:38:51 | opendevreview | Johannes Beisiegel proposed openstack/openstacksdk master: fix Proxy.server_actions signature to support kwargs https://review.opendev.org/c/openstack/openstacksdk/+/973606 | |
| 12:47:32 | opendevreview | Johannes Beisiegel proposed openstack/openstacksdk master: fix Proxy.server_actions signature to support kwargs https://review.opendev.org/c/openstack/openstacksdk/+/973606 | |
| 16:21:36 | cardoe | if someone has some +W cycles, we've got two changes that have 2 or more Ironic cores giving +2.. | |
| 16:21:46 | cardoe | https://review.opendev.org/c/openstack/openstacksdk/+/955103 and https://review.opendev.org/c/openstack/openstacksdk/+/973321 | |
| 16:22:02 | cardoe | I'm technically an Ironic core but I'm unable to give +2 on the sdk changes. | |
| 16:35:26 | frickler | cardoe: JayF: the reference for sdk has still the (I assume outdated now) ironic-core group https://review.opendev.org/admin/groups/33c03b5e000a75ffc476c51d484c5111648d399c,members should that be updated to ironic-approvers or ironic-reviewers? | |
| 16:36:04 | cardoe | ah yeah maybe. | |
| 17:46:09 | opendevreview | Jakub Darmach proposed openstack/ansible-collections-openstack master: Add support for BGP speaker https://review.opendev.org/c/openstack/ansible-collections-openstack/+/973662 | |
| 17:48:43 | opendevreview | Jakub Darmach proposed openstack/ansible-collections-openstack master: Add BGP support https://review.opendev.org/c/openstack/ansible-collections-openstack/+/973662 | |
| 17:49:40 | opendevreview | Jakub Darmach proposed openstack/ansible-collections-openstack master: Add BGP support https://review.opendev.org/c/openstack/ansible-collections-openstack/+/973662 | |
| 18:54:58 | JayF | frickler: I'd suggest -reviewers, but I understand if -approvers if preferred | |
| #openstack-sdks - 2026-01-18 | |||
| 19:47:12 | opendevreview | Valentin Chassignol proposed openstack/python-openstackclient master: Add retrieve password with private key decryption https://review.opendev.org/c/openstack/python-openstackclient/+/923621 | |
| 21:35:09 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: DNM: testing bindep fix https://review.opendev.org/c/openstack/python-openstackclient/+/973745 | |
| #openstack-sdks - 2026-01-19 | |||
| 11:39:04 | opendevreview | Johannes Beisiegel proposed openstack/openstacksdk master: fix: volume attachment list https://review.opendev.org/c/openstack/openstacksdk/+/973802 | |