| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2026-01-21 | |||
| 16:33:35 | opendevreview | Myles Penner proposed openstack/cliff master: Relax help usage assertion for module-based runners https://review.opendev.org/c/openstack/cliff/+/974213 | |
| 16:38:11 | opendevreview | Myles Penner proposed openstack/cliff master: Postpone annotations in cliff.complete https://review.opendev.org/c/openstack/cliff/+/974214 | |
| 22:44:45 | cardoe | What can we do to get https://review.opendev.org/c/openstack/openstacksdk/+/973287 into gazpacho so we can finish this feature off? | |
| #openstack-sdks - 2026-01-22 | |||
| 05:56:58 | opendevreview | Takashi Kajinami proposed openstack/codegenerator master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/codegenerator/+/973945 | |
| 07:14:30 | frickler | cardoe: well it currently isn't mergeable due to the wrong depends-on. once that is fixed, you could specifically ping the PTL and maybe some other cores | |
| 08:33:55 | opendevreview | Merged openstack/openstacksdk master: Add microversion to tag operation API request https://review.opendev.org/c/openstack/openstacksdk/+/973956 | |
| 11:41:35 | PavloKostianov[m] | Are osc_lib.utils (as I understand this is what it is about) really deprecated? I see osc_lib.tests.utils are deprecated, but can't find anything about osc_lib_utils deprecation. | |
| 11:41:35 | PavloKostianov[m] | Hey there! I am Pavlo - new here. I was reviewing release patch (https://review.opendev.org/c/openstack/releases/+/974030) which suggest releasing "replacement of deprecated formatter from osc_utils" in python-ironicclient (https://review.opendev.org/c/openstack/python-ironicclient/+/949704). My question is: | |
| 11:42:43 | PavloKostianov[m] | Are osc\_lib.utils (as I understand this is what it is about) really deprecated? I see osc\_lib.tests.utils are deprecated, but can't find anything about osc\_lib\_utils deprecation. | |
| 11:42:43 | PavloKostianov[m] | * Hey there! I am Pavlo - new here. I was reviewing release patch (https://review.opendev.org/c/openstack/releases/+/974030) which suggests releasing "replacement of deprecated formatter from osc\_utils" in python-ironicclient (https://review.opendev.org/c/openstack/python-ironicclient/+/949704). My question is: | |
| 14:59:20 | stephenfin | PavloKostianov[m]: https://github.com/openstack/osc-lib/commit/0e7dada02b825b2f97ac3eb68544bdd438a3a099 | |
| 15:00:33 | stephenfin | PavloKostianov[m]: osc_lib.utils *isn't* deprecated. Using the functions from there (or anything other than a formatter class *was* deprecated and is not removed | |
| 15:01:37 | stephenfin | correct: *(or anything other than a formatter class) with the get_dict_properties or get_item_properties functions | |
| 15:46:53 | PavloKostianov[m] | I see, thanks stephenfin! When you say "formatter class" you mean all the functions related to formatting? So let's say format_dict() is not deprecated, is it? No need to replace it in another dependency then? | |
| 15:51:55 | stephenfin | PavloKostianov[m]: correct | |
| 16:40:11 | opendevreview | cid proposed openstack/openstacksdk master: Add support for filtering by `conductor_groups` https://review.opendev.org/c/openstack/openstacksdk/+/973287 | |
| 16:53:48 | opendevreview | Jacob Anders proposed openstack/openstacksdk master: Add node health field for Redfish health monitoring https://review.opendev.org/c/openstack/openstacksdk/+/973264 | |
| 16:55:51 | opendevreview | Jacob Anders proposed openstack/openstacksdk master: Add node health field for Redfish health monitoring https://review.opendev.org/c/openstack/openstacksdk/+/973264 | |
| 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: Use project's domain for its parent https://review.opendev.org/c/openstack/python-openstackclient/+/974953 | |
| 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: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 | |