| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2026-01-22 | |||
| 17:06:44 | cardoe | frickler: I've got some others as well https://review.opendev.org/c/openstack/openstacksdk/+/955103 and https://review.opendev.org/c/openstack/openstacksdk/+/973321 both of those have 2+ ironic's +2s. | |
| 17:07:00 | cardoe | And for the permissions... https://review.opendev.org/c/openstack/project-config/+/974261 I'm not sure of the ask of me is? | |
| 17:10:31 | frickler | cardoe: you should consider stacking those changes, else they'll have merge conflicts. also gtema and stephenfin are the sdk cores most likely to approve these | |
| 17:13:19 | stephenfin | cardoe: approve those sdk changes | |
| 17:14:08 | frickler | cardoe: I have updated https://review.opendev.org/admin/groups/33c03b5e000a75ffc476c51d484c5111648d399c,members now, in theory any openstack-sdk-core could have done this. with that IMO 974261 should no longer be necessary | |
| 17:15:18 | stephenfin | frickler: infra related, but what role do the files in that gerrit/acls directory given you can modify things directly in the Gerrit UI? | |
| 17:17:21 | frickler | stephenfin: well the acl file needs to define which groups are allowed to do something. but if e.g. ironic-reviewers is already a member of openstacksdk-service-core, it is no longer necessary for the former to be explicitly added in the sdk acl | |
| 17:17:37 | frickler | same goes for all the other $project-core groups | |
| 17:18:55 | cardoe | stephenfin: thank you. | |
| 17:19:07 | cardoe | frickler: I'll stack the others and/or resolve merge conflicts. | |
| 17:23:23 | stephenfin | frickler: ack, so does https://review.opendev.org/c/openstack/project-config/+/974347 make sense? | |
| 18:47:01 | frickler | yes, that's what I suggested, will wait for gtema to +1 in case there was some reasoning behind this that I'm not aware of | |
| 18:47:07 | opendevreview | Merged openstack/openstacksdk master: Add Node.instance_name support https://review.opendev.org/c/openstack/openstacksdk/+/955103 | |
| 18:47:09 | opendevreview | Merged openstack/openstacksdk master: feat: add missing baremetal port fields https://review.opendev.org/c/openstack/openstacksdk/+/973321 | |
| 18:52:26 | gtema | Stephenfin, frickler: I think the main reason for that setup was to have a clear separation of concerns. When all service cores are included in the sdk-core group we cannot prevent them from having also workflow permissions, which we wanted to protect. I think it is better to have Gerrit acls and not to play with group nesting | |
| 18:55:36 | opendevreview | Doug Goldstein proposed openstack/openstacksdk master: Add support for filtering by `conductor_groups` https://review.opendev.org/c/openstack/openstacksdk/+/973287 | |
| 18:58:28 | opendevreview | cid proposed openstack/openstacksdk master: Add support for filtering by `conductor_groups` https://review.opendev.org/c/openstack/openstacksdk/+/973287 | |
| 18:59:33 | cardoe | cid: <spiderman meme> | |
| 18:59:48 | cid | :D | |
| 21:34:14 | opendevreview | Piotr Sipika proposed openstack/python-openstackclient master: Make --all-stores behave the same as in glanceclient. https://review.opendev.org/c/openstack/python-openstackclient/+/974374 | |
| #openstack-sdks - 2026-01-23 | |||
| 06:44:02 | frickler | gtema: the service cores are not included in the openstacksdk-core group, there is a dedicated openstacksdk-service-group, which only has CR+2 rights and lacks the W+1 | |
| 08:25:39 | opendevreview | Jacob Anders proposed openstack/openstacksdk master: Add node health field for Redfish health monitoring https://review.opendev.org/c/openstack/openstacksdk/+/973264 | |
| 10:36:36 | stephenfin | gtema: yeah, what frickler said: there are different groups | |
| 10:37:07 | gtema | yes, I got that already and recorded in the change review | |
| 10:39:06 | stephenfin | ah, didn't see that. ty | |
| 11:39:39 | opendevreview | Jacob Anders proposed openstack/openstacksdk master: Add node health field for Redfish health monitoring https://review.opendev.org/c/openstack/openstacksdk/+/973264 | |
| 14:31:08 | opendevreview | Piotr Sipika proposed openstack/python-openstackclient master: Make --all-stores behave the same as in glanceclient. https://review.opendev.org/c/openstack/python-openstackclient/+/974374 | |
| 15:24:34 | opendevreview | Merged openstack/openstacksdk master: Add node health field for Redfish health monitoring https://review.opendev.org/c/openstack/openstacksdk/+/973264 | |
| 16:10:19 | opendevreview | Johannes Beisiegel proposed openstack/openstacksdk master: fix: volume attachment list https://review.opendev.org/c/openstack/openstacksdk/+/973802 | |
| 17:33:12 | cardoe | frickler, stephenfin: if ya get a chance https://review.opendev.org/c/openstack/openstacksdk/+/973287 will let us move forward on the feature we wanted to get for 2026.1 | |
| 20:12:33 | opendevreview | Myles Penner proposed openstack/osc-lib master: Make tag help tests robust by setting prog name https://review.opendev.org/c/openstack/osc-lib/+/974558 | |
| 22:02:42 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/936279 | |
| 22:02:43 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Add --project-domain option for limits https://review.opendev.org/c/openstack/python-openstackclient/+/973325 | |
| 23:36:13 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Add --project-domain option for limits https://review.opendev.org/c/openstack/python-openstackclient/+/973325 | |
| #openstack-sdks - 2026-01-24 | |||
| 12:53:05 | opendevreview | Ivan Anfimov proposed openstack/ansible-collections-openstack master: tox: Drop basepython https://review.opendev.org/c/openstack/ansible-collections-openstack/+/974599 | |
| 12:53:29 | opendevreview | Ivan Anfimov proposed openstack/ansible-collections-openstack master: tox: Drop basepython https://review.opendev.org/c/openstack/ansible-collections-openstack/+/974599 | |
| #openstack-sdks - 2026-01-26 | |||
| 14:04:40 | opendevreview | Merged openstack/osc-lib master: Make tag help tests robust by setting prog name https://review.opendev.org/c/openstack/osc-lib/+/974558 | |
| 14:06:58 | opendevreview | Merged openstack/keystoneauth master: Enable logging related ruff checks https://review.opendev.org/c/openstack/keystoneauth/+/970587 | |
| 17:19:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add support for filtering by `conductor_groups` https://review.opendev.org/c/openstack/openstacksdk/+/973287 | |
| 21:04:59 | opendevreview | Merged openstack/openstacksdk master: Add support for filtering by `conductor_groups` https://review.opendev.org/c/openstack/openstacksdk/+/973287 | |
| #openstack-sdks - 2026-01-27 | |||
| 03:41:37 | opendevreview | Cyril Roelandt proposed openstack/openstacksdk master: upload_image: improve testing https://review.opendev.org/c/openstack/openstacksdk/+/974799 | |
| 13:11:57 | opendevreview | Juan Larriba proposed openstack/openstacksdk master: Add _query_mapping to Share resource for query parameter support https://review.opendev.org/c/openstack/openstacksdk/+/974036 | |
| 15:02:12 | opendevreview | Juan Larriba proposed openstack/openstacksdk master: Add _query_mapping to Share resource for query parameter support https://review.opendev.org/c/openstack/openstacksdk/+/974036 | |
| 15:49:36 | opendevreview | Cyril Roelandt proposed openstack/openstacksdk master: upload_image: improve testing https://review.opendev.org/c/openstack/openstacksdk/+/974799 | |
| 16:49:50 | opendevreview | Myles Penner proposed openstack/openstacksdk master: Postpone annotation evaluation for Python 3.14 https://review.opendev.org/c/openstack/openstacksdk/+/973884 | |
| 17:13:20 | opendevreview | Myles Penner proposed openstack/cliff master: Mock argv in help test for module-based runners https://review.opendev.org/c/openstack/cliff/+/974213 | |
| 19:23:56 | JayF | python-ironicclient isn't an SDKs team deliverable, but there's a change I think you all would have useful feedback for if you have a sec: https://review.opendev.org/c/openstack/python-ironicclient/+/973948/6#message-d25338b63dd378de2af2e0e0cbaf87eb8729f4b4 | |
| 19:56:32 | opendevreview | Merged openstack/openstacksdk master: Add _query_mapping to Share resource for query parameter support https://review.opendev.org/c/openstack/openstacksdk/+/974036 | |
| 20:42:32 | opendevreview | Merged openstack/python-openstackclient master: Identity: Migrate 'limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/936279 | |
| 20:42:49 | opendevreview | Merged openstack/openstacksdk master: upload_image: improve testing https://review.opendev.org/c/openstack/openstacksdk/+/974799 | |
| 20:42:53 | opendevreview | Merged openstack/openstacksdk master: Postpone annotation evaluation for Python 3.14 https://review.opendev.org/c/openstack/openstacksdk/+/973884 | |
| 21:24:05 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| 21:24:06 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Use project's domain for its parent https://review.opendev.org/c/openstack/python-openstackclient/+/974953 | |
| 21:37:20 | M0weng[m] | Quick question about projects and domains in keystone - once a project is created to be part of a domain, is it impossible to move it to a different domain? I.e., once the project's domain is set, it can't be changed. | |
| 21:58:04 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| 22:00:31 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| 22:07: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:14:41 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| 22:35:48 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'project' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/965317 | |
| 22:35:48 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Use project's domain for its parent https://review.opendev.org/c/openstack/python-openstackclient/+/974953 | |
| 22:37:39 | cardoe | stephenfin: so keystoneauth moved governance to the SDK group but it seems like the gerrit permissions still need an update. | |
| 22:37:53 | cardoe | M0weng[m]: as far as I am aware, yes. | |
| 22:51:51 | M0weng[m] | Thanks cardoe ! | |
| 23:02:35 | opendevreview | Piotr Sipika proposed openstack/python-openstackclient master: Make --all-stores behave the same as in glanceclient. https://review.opendev.org/c/openstack/python-openstackclient/+/974374 | |
| 23:10:51 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Add --project-domain option for limits https://review.opendev.org/c/openstack/python-openstackclient/+/973325 | |
| 23:15:20 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Add --project-domain option for limits https://review.opendev.org/c/openstack/python-openstackclient/+/973325 | |
| 23:16:22 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Add --project-domain option for limits https://review.opendev.org/c/openstack/python-openstackclient/+/973325 | |
| 23:25:44 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Add --project-domain option for limits https://review.opendev.org/c/openstack/python-openstackclient/+/973325 | |
| #openstack-sdks - 2026-01-28 | |||
| 10:40:35 | stephenfin | cardoe: Yeah, good point. I wonder if gtema can fix that? | |
| 10:41:51 | gtema | I did that (included openstacksdk-core) in the keystoneauth-core group | |
| 10:47:46 | stephenfin | gtema++ thanks | |
| #openstack-sdks - 2026-01-29 | |||
| 12:45:45 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Remove tags from README https://review.opendev.org/c/openstack/keystoneauth/+/972415 | |
| 13:06:26 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Fix docs builds on Python 3.14 https://review.opendev.org/c/openstack/keystoneauth/+/970686 | |
| 13:06:27 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Bump pre-commit versions https://review.opendev.org/c/openstack/keystoneauth/+/975148 | |
| 13:07:13 | opendevreview | Merged openstack/keystoneauth master: Remove tags from README https://review.opendev.org/c/openstack/keystoneauth/+/972415 | |
| 14:49:30 | opendevreview | Simon Dodsley proposed openstack/ansible-collections-openstack master: Add volume_image_metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/975168 | |
| 15:02:22 | opendevreview | Simon Dodsley proposed openstack/ansible-collections-openstack master: Add volume_image_metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/975168 | |
| 15:12:08 | opendevreview | Simon Dodsley proposed openstack/ansible-collections-openstack master: Add volume_image_metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/975168 | |
| 15:57:05 | opendevreview | Simon Dodsley proposed openstack/ansible-collections-openstack master: Add volume_image_metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/975168 | |
| 16:03:01 | opendevreview | Simon Dodsley proposed openstack/ansible-collections-openstack master: Add volume_image_metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/975168 | |
| 16:46:04 | opendevreview | Simon Dodsley proposed openstack/ansible-collections-openstack master: Add volume_image_metadata https://review.opendev.org/c/openstack/ansible-collections-openstack/+/975168 | |
| #openstack-sdks - 2026-01-30 | |||
| 13:11:29 | 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 | |
| #openstack-sdks - 2026-02-02 | |||
| 15:12:32 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Accept float value for timeout https://review.opendev.org/c/openstack/keystoneauth/+/975408 | |
| 15:13:36 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Accept float value for timeout https://review.opendev.org/c/openstack/keystoneauth/+/975408 | |
| #openstack-sdks - 2026-02-03 | |||
| 01:39:42 | opendevreview | Dong Ma proposed openstack/openstacksdk master: Fix RecursionError when deepcopy is called on CloudRegion https://review.opendev.org/c/openstack/openstacksdk/+/975482 | |
| 07:04:51 | opendevreview | Dong Ma proposed openstack/openstacksdk master: Fix RecursionError when deepcopy is called on CloudRegion https://review.opendev.org/c/openstack/openstacksdk/+/975482 | |
| 09:17:40 | opendevreview | Dong Ma proposed openstack/openstacksdk master: Fix RecursionError when deepcopy is called on CloudRegion https://review.opendev.org/c/openstack/openstacksdk/+/975482 | |
| #openstack-sdks - 2026-02-04 | |||
| 13:08:54 | opendevreview | Nam Hyeokjun proposed openstack/python-openstackclient master: identity: fall back to case-insensitive user lookup https://review.opendev.org/c/openstack/python-openstackclient/+/975631 | |
| #openstack-sdks - 2026-02-05 | |||
| 12:24:12 | opendevreview | Gorka Eguileor proposed openstack/osc-lib master: Fix shell's __init__ https://review.opendev.org/c/openstack/osc-lib/+/975784 | |
| #openstack-sdks - 2026-02-19 | |||
| 10:14:48 | opendevreview | Michal Nasiadka proposed openstack/ansible-collections-openstack master: CI: Run jobs for all non-EOL Ansible versions https://review.opendev.org/c/openstack/ansible-collections-openstack/+/977069 | |
| 11:05:55 | opendevreview | Michal Nasiadka proposed openstack/ansible-collections-openstack master: CI: Run jobs for all non-EOL Ansible versions https://review.opendev.org/c/openstack/ansible-collections-openstack/+/977069 | |
| #openstack-sdks - 2026-02-20 | |||
| 15:15:00 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Bump flake8-import-order https://review.opendev.org/c/openstack/keystoneauth/+/977449 | |
| 15:23:40 | opendevreview | Takashi Kajinami proposed openstack/codegenerator master: Bump flake8-import-order https://review.opendev.org/c/openstack/codegenerator/+/977453 | |
| 15:26:20 | opendevreview | Takashi Kajinami proposed openstack/codegenerator master: Bump flake8-import-order https://review.opendev.org/c/openstack/codegenerator/+/977453 | |
| 15:43:48 | opendevreview | Merged openstack/keystoneauth master: Bump flake8-import-order https://review.opendev.org/c/openstack/keystoneauth/+/977449 | |
| 17:22:33 | opendevreview | Merged openstack/keystoneauth master: Fix docs builds on Python 3.14 https://review.opendev.org/c/openstack/keystoneauth/+/970686 | |
| 17:22:36 | opendevreview | Merged openstack/keystoneauth master: Remove old hacking check for oslo namespace package https://review.opendev.org/c/openstack/keystoneauth/+/972213 | |
| #openstack-sdks - 2026-02-21 | |||
| 11:02:49 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Remove unused Makefile for doc https://review.opendev.org/c/openstack/keystoneauth/+/977512 | |
| 11:10:08 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Remove unused Makefile for doc https://review.opendev.org/c/openstack/python-openstackclient/+/977521 | |
| 11:11:05 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Remove unused Makefile for doc https://review.opendev.org/c/openstack/python-openstackclient/+/977521 | |