| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2026-09-18 | |||
| 17:44:16 | opendevreview | Merged openstack/python-openstackclient stable/2026.2: vpnaas: Remove deprecated sha1 and 3des algorithms https://review.opendev.org/c/openstack/python-openstackclient/+/1006264 | |
| #openstack-sdks - 2026-09-19 | |||
| 00:30:48 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Migrate accelerator commands from python-cyborgclient https://review.opendev.org/c/openstack/python-openstackclient/+/1004707 | |
| 00:30:49 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Add functional tests and Zuul job for accelerator commands https://review.opendev.org/c/openstack/python-openstackclient/+/996698 | |
| #openstack-sdks - 2026-09-21 | |||
| 08:15:57 | opendevreview | Pranali Deore proposed openstack/python-openstackclient master: Add cached image clean and prune commands https://review.opendev.org/c/openstack/python-openstackclient/+/1004414 | |
| 11:41:40 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Add missing service guides https://review.opendev.org/c/openstack/openstacksdk/+/1006267 | |
| 11:41:47 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: trivial: Remove dead TODO https://review.opendev.org/c/openstack/openstacksdk/+/1006268 | |
| 11:41:56 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: trivial: Remove dead TODO https://review.opendev.org/c/openstack/openstacksdk/+/1006268 | |
| 11:45:00 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Bump up deprecation timeline for deprecated cloud filter kwarg https://review.opendev.org/c/openstack/openstacksdk/+/1006193 | |
| 11:45:07 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Bump up deprecation timeline for passing dicts to cloud APIs https://review.opendev.org/c/openstack/openstacksdk/+/1006194 | |
| 12:52:55 | opendevreview | Merged openstack/openstacksdk master: docs: Resolve shade TODOs https://review.opendev.org/c/openstack/openstacksdk/+/1006265 | |
| 12:52:57 | opendevreview | Merged openstack/openstacksdk master: docs: Add proxy, auth plugin guides https://review.opendev.org/c/openstack/openstacksdk/+/1006266 | |
| 13:00:50 | opendevreview | Merged openstack/openstacksdk master: zuul: Remove unused jobs https://review.opendev.org/c/openstack/openstacksdk/+/1006278 | |
| 13:22:47 | opendevreview | Merged openstack/openstacksdk master: Bump up deprecation timeline for CloudConfig class https://review.opendev.org/c/openstack/openstacksdk/+/1006191 | |
| 13:22:49 | opendevreview | Merged openstack/openstacksdk master: Bump up deprecation timeline for deprecated proxy, cloud methods https://review.opendev.org/c/openstack/openstacksdk/+/1006192 | |
| 14:14:43 | cardoe | stephenfin: https://review.opendev.org/q/project:openstack/keystoneauth+status:open has 4 patches that should be good to +W | |
| 14:15:17 | stephenfin | Do you not have +W there yet? | |
| 14:16:19 | stephenfin | Wait, I forgot to make the openstacksdk-core changes promised 🤦 | |
| 14:16:55 | stephenfin | cardoe: You should be able to +2/+W those yourself now | |
| 14:22:09 | cardoe | heh indeed now I can | |
| 14:22:16 | cardoe | working through openstacksdk open stuff now as well. | |
| 14:54:04 | opendevreview | Merged openstack/openstacksdk master: trivial: Typo fix https://review.opendev.org/c/openstack/openstacksdk/+/1006219 | |
| 14:54:07 | opendevreview | Merged openstack/openstacksdk master: proxy: Group methods https://review.opendev.org/c/openstack/openstacksdk/+/1006220 | |
| 14:58:43 | opendevreview | Merged openstack/python-openstackclient master: volume: Replace server arg for volume attachment create https://review.opendev.org/c/openstack/python-openstackclient/+/1002622 | |
| 15:03:15 | opendevreview | Merged openstack/openstacksdk master: docs: Add missing service guides https://review.opendev.org/c/openstack/openstacksdk/+/1006267 | |
| 15:03:20 | opendevreview | Merged openstack/openstacksdk master: Bump up deprecation timeline for deprecated cloud filter kwarg https://review.opendev.org/c/openstack/openstacksdk/+/1006193 | |
| 15:03:24 | opendevreview | Merged openstack/openstacksdk master: Bump up deprecation timeline for passing dicts to cloud APIs https://review.opendev.org/c/openstack/openstacksdk/+/1006194 | |
| 15:11:12 | opendevreview | Merged openstack/openstacksdk master: trivial: Remove dead TODO https://review.opendev.org/c/openstack/openstacksdk/+/1006268 | |
| 15:11:15 | opendevreview | Merged openstack/openstacksdk master: zuul: Remove more dead jobs https://review.opendev.org/c/openstack/openstacksdk/+/1006279 | |
| 15:11:18 | opendevreview | Merged openstack/openstacksdk master: zuul: Use proper boolean values https://review.opendev.org/c/openstack/openstacksdk/+/1006280 | |
| 15:20:12 | opendevreview | Merged openstack/python-openstackclient master: Add router option to floating IP create and set commands https://review.opendev.org/c/openstack/python-openstackclient/+/1005826 | |
| 15:50:46 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Stop ignoring .0 microversions https://review.opendev.org/c/openstack/openstacksdk/+/1006234 | |
| 15:50:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: placement: Add support for bulk update, delete resource provider inventories https://review.opendev.org/c/openstack/openstacksdk/+/1006242 | |
| 15:50:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: placement: Add support for deleting resource provider traits https://review.opendev.org/c/openstack/openstacksdk/+/1006241 | |
| 15:50:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: placement: Add support for allocation candidates https://review.opendev.org/c/openstack/openstacksdk/+/1006240 | |
| 15:50:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: placement: Add support for resource provider allocations https://review.opendev.org/c/openstack/openstacksdk/+/1006239 | |
| 15:50:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: placement: Add support for bulk create of allocations https://review.opendev.org/c/openstack/openstacksdk/+/1006238 | |
| 15:50:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: placement: Add support for allocations https://review.opendev.org/c/openstack/openstacksdk/+/1006237 | |
| 15:50:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: placement: Allow empty commit on resource class https://review.opendev.org/c/openstack/openstacksdk/+/1006236 | |
| 15:50:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: placement: Fix trait create and exists https://review.opendev.org/c/openstack/openstacksdk/+/1006235 | |
| 15:50:50 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: placement: Add support for usages https://review.opendev.org/c/openstack/openstacksdk/+/1006243 | |
| 16:34:54 | opendevreview | Brian Haley proposed openstack/openstacksdk master: Make openstacksdk-functional-devstack-networking-ext OVN-based https://review.opendev.org/c/openstack/openstacksdk/+/1006592 | |
| 16:44:41 | opendevreview | Brian Haley proposed openstack/openstacksdk master: Make openstacksdk-functional-devstack-networking-ext OVN-based https://review.opendev.org/c/openstack/openstacksdk/+/1006592 | |
| 16:49:15 | opendevreview | Merged openstack/keystoneauth master: Include TOTP passcode in auth cache key elements https://review.opendev.org/c/openstack/keystoneauth/+/1005216 | |
| 16:49:17 | opendevreview | Merged openstack/keystoneauth master: Default OIDC device authorization polling interval to 5s https://review.opendev.org/c/openstack/keystoneauth/+/1005688 | |
| 16:49:19 | opendevreview | Merged openstack/keystoneauth master: Redact OTP and MFA fields in OIDC debug sanitization https://review.opendev.org/c/openstack/keystoneauth/+/1005685 | |
| 16:58:51 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Deprecate legacy option names in accelerator commands https://review.opendev.org/c/openstack/python-openstackclient/+/1006597 | |
| 18:40:37 | opendevreview | Brian Haley proposed openstack/openstacksdk master: Make openstacksdk-functional-devstack-networking-ext OVN-based https://review.opendev.org/c/openstack/openstacksdk/+/1006592 | |
| 19:11:28 | opendevreview | Merged openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-networking-ext voting https://review.opendev.org/c/openstack/openstacksdk/+/1005450 | |
| 19:13:25 | opendevreview | Merged openstack/openstacksdk master: Fix typo https://review.opendev.org/c/openstack/openstacksdk/+/1006291 | |
| 19:56:35 | opendevreview | Brian Haley proposed openstack/openstacksdk master: Make openstacksdk-functional-devstack-networking-ext OVN-based https://review.opendev.org/c/openstack/openstacksdk/+/1006592 | |
| 20:04:46 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Validate SAML2 ECP consumer fault and redirect URLs https://review.opendev.org/c/openstack/keystoneauth/+/1005682 | |
| 20:04:47 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Resolve relative ACS redirect Location in SAML2 ECP https://review.opendev.org/c/openstack/keystoneauth/+/1005690 | |
| 20:04:48 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Handle optional RelayState in SAML2 ECP responses https://review.opendev.org/c/openstack/keystoneauth/+/1005691 | |
| 22:45:03 | opendevreview | Merged openstack/openstacksdk master: config: Stop ignoring .0 microversions https://review.opendev.org/c/openstack/openstacksdk/+/1006234 | |
| 22:45:06 | opendevreview | Merged openstack/openstacksdk master: placement: Fix trait create and exists https://review.opendev.org/c/openstack/openstacksdk/+/1006235 | |
| 22:45:08 | opendevreview | Merged openstack/openstacksdk master: placement: Allow empty commit on resource class https://review.opendev.org/c/openstack/openstacksdk/+/1006236 | |
| 23:21:54 | opendevreview | Merged openstack/keystoneauth master: Validate SAML2 ECP consumer fault and redirect URLs https://review.opendev.org/c/openstack/keystoneauth/+/1005682 | |
| 23:27:11 | opendevreview | Merged openstack/keystoneauth master: Reject unsupported system_scope values in v3 auth requests https://review.opendev.org/c/openstack/keystoneauth/+/1005238 | |
| 23:27:13 | opendevreview | Merged openstack/keystoneauth master: Resolve relative ACS redirect Location in SAML2 ECP https://review.opendev.org/c/openstack/keystoneauth/+/1005690 | |
| #openstack-sdks - 2026-09-22 | |||
| 00:17:21 | opendevreview | Merged openstack/openstacksdk master: placement: Add support for allocations https://review.opendev.org/c/openstack/openstacksdk/+/1006237 | |
| 00:17:26 | opendevreview | Merged openstack/openstacksdk master: placement: Add support for bulk create of allocations https://review.opendev.org/c/openstack/openstacksdk/+/1006238 | |
| 00:26:19 | opendevreview | Merged openstack/openstacksdk master: placement: Add support for resource provider allocations https://review.opendev.org/c/openstack/openstacksdk/+/1006239 | |
| 01:41:02 | opendevreview | Brian Haley proposed openstack/openstacksdk master: Make openstacksdk-functional-devstack-networking-ext OVN-based https://review.opendev.org/c/openstack/openstacksdk/+/1006592 | |
| 02:18:23 | opendevreview | Brian Haley proposed openstack/openstacksdk master: Make openstacksdk-functional-devstack-networking-ext OVN-based https://review.opendev.org/c/openstack/openstacksdk/+/1006592 | |
| 02:22:38 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Migrate accelerator commands from python-cyborgclient https://review.opendev.org/c/openstack/python-openstackclient/+/1004707 | |
| 02:22:39 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Rename and deprecate legacy options in accelerator commands https://review.opendev.org/c/openstack/python-openstackclient/+/1006597 | |
| 02:22:39 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Add functional tests and Zuul job for accelerator commands https://review.opendev.org/c/openstack/python-openstackclient/+/996698 | |
| 08:25:48 | opendevreview | Maximilian Stinsky proposed openstack/ansible-collections-openstack master: identity_user: support Keystone user resource options https://review.opendev.org/c/openstack/ansible-collections-openstack/+/1006688 | |
| 11:15:36 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tox: Run mypy over package https://review.opendev.org/c/openstack/openstacksdk/+/1006703 | |
| 11:45:58 | opendevreview | Inyong Hong proposed openstack/python-openstackclient master: volume: Add 'volume group add/remove volume' commands https://review.opendev.org/c/openstack/python-openstackclient/+/995941 | |
| 15:21:14 | opendevreview | Merged openstack/python-openstackclient master: volume: Add 'volume group add/remove volume' commands https://review.opendev.org/c/openstack/python-openstackclient/+/995941 | |
| 16:50:48 | opendevreview | Brian Haley proposed openstack/openstacksdk master: Make openstacksdk-functional-devstack-networking-ext OVN-based https://review.opendev.org/c/openstack/openstacksdk/+/1006592 | |
| 17:56:14 | opendevreview | Brian Haley proposed openstack/openstacksdk master: Make openstacksdk-functional-devstack-networking-ext OVN-based https://review.opendev.org/c/openstack/openstacksdk/+/1006592 | |
| 18:11:39 | opendevreview | Merged openstack/openstacksdk master: tox: Run mypy over package https://review.opendev.org/c/openstack/openstacksdk/+/1006703 | |
| 18:37:02 | opendevreview | Merged openstack/openstacksdk master: placement: Add support for allocation candidates https://review.opendev.org/c/openstack/openstacksdk/+/1006240 | |
| 18:37:07 | opendevreview | Merged openstack/openstacksdk master: placement: Add support for deleting resource provider traits https://review.opendev.org/c/openstack/openstacksdk/+/1006241 | |
| #openstack-sdks - 2026-09-23 | |||
| 04:55:03 | opendevreview | Merged openstack/openstacksdk master: heat: Fix stack query tag parameters https://review.opendev.org/c/openstack/openstacksdk/+/1005901 | |
| 04:57:25 | opendevreview | Merged openstack/openstacksdk master: placement: Add support for bulk update, delete resource provider inventories https://review.opendev.org/c/openstack/openstacksdk/+/1006242 | |
| 04:57:26 | opendevreview | Merged openstack/openstacksdk master: placement: Add support for usages https://review.opendev.org/c/openstack/openstacksdk/+/1006243 | |
| 06:29:38 | opendevreview | Pranali Deore proposed openstack/openstacksdk master: image: Parse hw_vif_multiqueue_enabled as BoolStr https://review.opendev.org/c/openstack/openstacksdk/+/1006895 | |
| 07:16:53 | opendevreview | Moritz Tanner proposed openstack/ansible-collections-openstack master: image: Perform a web-download import when uri is given https://review.opendev.org/c/openstack/ansible-collections-openstack/+/1006899 | |
| 07:19:36 | opendevreview | Moritz Tanner proposed openstack/ansible-collections-openstack master: image: do a web-download import when uri is given https://review.opendev.org/c/openstack/ansible-collections-openstack/+/1006899 | |
| 08:33:21 | opendevreview | Matt Crees proposed openstack/ansible-collections-openstack master: Fix: Include target_project_id in RBAC creation https://review.opendev.org/c/openstack/ansible-collections-openstack/+/964684 | |
| 08:35:36 | opendevreview | Matt Crees proposed openstack/ansible-collections-openstack master: Fix: Include target_project_id in RBAC creation https://review.opendev.org/c/openstack/ansible-collections-openstack/+/964684 | |
| 11:47:25 | opendevreview | Vladislav Terekhov proposed openstack/openstacksdk master: resource: Fix CSV query parameter formatting for strings https://review.opendev.org/c/openstack/openstacksdk/+/1005733 | |
| 13:23:49 | andreykurilin | Hi folks! Is anyone available to talk about `atexit.register(self.close)` in the connection init method? It seems like a problem for horizon + urllib3 2+ . I can raise a bug report, but want to discuss it first | |
| 13:53:10 | andreykurilin | atexit.register | |
| 13:53:10 | andreykurilin | The problem - Horizon of 2026.1 started to leak memory. Quite a lot in comparison to 2025.1 . After digging into an affected process, I found ~3,5k of openstacksdk objects in the memory. Each one stores a connection pool with a complete chain of other objects. Horizon never calls close for most of openstack connection it creates, which means that all of them (as far as I understand) keeps to be attached to uwsgi process due to | |
| 13:56:10 | andreykurilin | https://github.com/psf/requests/pull/6767 changed the behavior of default ssl context. Every pooled connection now owns a CA-loaded SSLContext which makes the difference in memory consumption by leaked objects | |
| 14:38:25 | stephenfin | andreykurilin: What do you think is the issue from the SDK side? | |
| 14:41:03 | andreykurilin | `atexit.register` call which makes openstack conn “immortal” :) I’m doing some experiments right now, it seems like switching to weakref.finalize should solve the problem - it would fire close “as soon” as openstacksdk object is no longer used | |
| 14:43:52 | andreykurilin | @stephenfin https://github.com/openstack/openstacksdk/blob/stable/2026.1/openstack/cloud/openstackcloud.py#L279-L280 this part I’m talking about | |
| 14:45:05 | stephenfin | as you likely know, the whole point of that is to ensure things are correctly closed before the Python process exits. gtema is out this week (I think) but he added that some time back https://github.com/openstack/openstacksdk/commit/2b4aeff6d3972761da9c4a6a8b696103664a758b | |
| 14:46:01 | andreykurilin | Yes, it has been there since 2023. I believe it has not caused much problems as leaked requests connections were cheap | |
| 14:46:56 | andreykurilin | Having thousands of connections in memory was taking like couple megabytes as they were sharing the same ssl context | |
| 14:49:16 | andreykurilin | Horizon (uwsgi) is long running process which means that atexit may never fire until redeploy or OOM | |
| 14:51:24 | stephenfin | Right. My reason for pointing that out wasn't to say that this isn't an issue because it's old code. Rather, it was merely to point out that the code is there for a reason | |
| 14:52:32 | stephenfin | I wonder how practical it would be to allow Connection to be used as a context manager and to use that in-tree in Horizon. Or just have horizon ensure it calls `close()` when it's done? | |
| 14:53:01 | andreykurilin | Considering the review speed of horizon changes - near impossible | |
| 14:54:35 | andreykurilin | + it is hard to ensure that all horizon dashboards follow this. For example, octavia-dashboard may create dozens openstacksdk objs for a single user request, no memorize usage | |