| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2026-01-15 | |||
| 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 | |
| 12:32:41 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/2025.1: zuul: Remove osc-upload-image, osc-promote-image jobs https://review.opendev.org/c/openstack/python-openstackclient/+/973816 | |
| 12:33:12 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/2025.1: identity: Fix 'user list --project' option https://review.opendev.org/c/openstack/python-openstackclient/+/962778 | |
| 13:30:35 | opendevreview | Jakub Darmach proposed openstack/ansible-collections-openstack master: Add BGP support https://review.opendev.org/c/openstack/ansible-collections-openstack/+/973662 | |
| 14:10:44 | opendevreview | Johannes Beisiegel proposed openstack/openstacksdk master: fix: volume attachment list https://review.opendev.org/c/openstack/openstacksdk/+/973802 | |
| 14:46:09 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Mention OS_CLIENT_SECURE_FILE env var in docs https://review.opendev.org/c/openstack/openstacksdk/+/973842 | |
| 15:05:38 | opendevreview | Merged openstack/python-openstackclient stable/2025.1: zuul: Remove osc-upload-image, osc-promote-image jobs https://review.opendev.org/c/openstack/python-openstackclient/+/973816 | |
| 15:05:40 | opendevreview | Merged openstack/python-openstackclient stable/2025.1: identity: Fix 'user list --project' option https://review.opendev.org/c/openstack/python-openstackclient/+/962778 | |
| 15:08:49 | opendevreview | Stephen Finucane proposed openstack/cliff master: Revert "Deprecate CommandManager namespace argument" https://review.opendev.org/c/openstack/cliff/+/973844 | |
| 16:20:18 | opendevreview | Jakub Darmach proposed openstack/ansible-collections-openstack master: Add BGP support https://review.opendev.org/c/openstack/ansible-collections-openstack/+/973662 | |
| 16:27:20 | opendevreview | Jacob Anders proposed openstack/openstacksdk master: Add node health field for Redfish health monitoring https://review.opendev.org/c/openstack/openstacksdk/+/973264 | |
| 17:52:36 | opendevreview | Merged openstack/cliff master: Revert "Deprecate CommandManager namespace argument" https://review.opendev.org/c/openstack/cliff/+/973844 | |
| 18:39:44 | opendevreview | Myles Penner proposed openstack/openstacksdk master: Postpone annotation evaluation for Python 3.14 https://review.opendev.org/c/openstack/openstacksdk/+/973884 | |
| 18:39:44 | opendevreview | Myles Penner proposed openstack/openstacksdk master: Modernize type hints after enabling future annotations https://review.opendev.org/c/openstack/openstacksdk/+/973885 | |
| 19:55:35 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient stable/2025.1: Fix incorrect warning with --password-prompt option https://review.opendev.org/c/openstack/python-openstackclient/+/951844 | |
| 19:55:39 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient stable/2025.1: Fix incorrect warning with --password-prompt option https://review.opendev.org/c/openstack/python-openstackclient/+/951844 | |
| 19:55:59 | opendevreview | Rajesh Tailor proposed openstack/python-openstackclient stable/2025.1: Fix microversion 2.96 https://review.opendev.org/c/openstack/python-openstackclient/+/958758 | |
| 22:16:06 | opendevreview | Merged openstack/python-openstackclient stable/2025.1: Fix openstack quota show without cinder https://review.opendev.org/c/openstack/python-openstackclient/+/962841 | |
| #openstack-sdks - 2026-01-20 | |||
| 01:15:14 | opendevreview | Merged openstack/python-openstackclient stable/2025.1: Fix microversion 2.96 https://review.opendev.org/c/openstack/python-openstackclient/+/958758 | |
| 09:56:48 | opendevreview | Jan Ueberacker proposed openstack/python-openstackclient master: Fix: Make server resize options required https://review.opendev.org/c/openstack/python-openstackclient/+/973919 | |
| 12:40:54 | opendevreview | Jacob Anders proposed openstack/openstacksdk master: Add node health field for Redfish health monitoring https://review.opendev.org/c/openstack/openstacksdk/+/973264 | |
| 12:57:02 | opendevreview | Jacob Anders proposed openstack/openstacksdk master: Add node health field for Redfish health monitoring https://review.opendev.org/c/openstack/openstacksdk/+/973264 | |
| 14:24:13 | opendevreview | Takashi Kajinami proposed openstack/codegenerator master: Remove unused libpcre3-dev https://review.opendev.org/c/openstack/codegenerator/+/973945 | |
| 15:59:40 | opendevreview | Masahito Muroi proposed openstack/openstacksdk master: Add microversion to tag operation API request https://review.opendev.org/c/openstack/openstacksdk/+/973956 | |
| 17:42:00 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Adapt bindep for debian-trixie https://review.opendev.org/c/openstack/python-openstackclient/+/973973 | |
| 17:42:07 | stephenfin | tkajinam: Can you sanity check ^ | |
| 17:42:10 | stephenfin | ? | |
| 23:48:22 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/python-openstackclient/+/974019 | |
| 23:50:12 | tkajinam | stephenfin, whereto was already bumped in global requirements so I think this is more straightforward ^^^ | |
| #openstack-sdks - 2026-01-21 | |||
| 08:43:31 | opendevreview | Jan Ueberacker proposed openstack/python-openstackclient master: Fix: Make server resize options required https://review.opendev.org/c/openstack/python-openstackclient/+/973919 | |
| 11:06:24 | 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 | |
| 11:54:30 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/python-openstackclient/+/974045 | |
| 14:53:51 | opendevreview | Merged openstack/python-openstackclient master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/python-openstackclient/+/974019 | |
| 14:53:54 | opendevreview | Merged openstack/python-openstackclient master: tests: Update functional test for image metadef object https://review.opendev.org/c/openstack/python-openstackclient/+/961213 | |
| 14:53:57 | opendevreview | Merged openstack/python-openstackclient master: Add functional test for volume snapshot https://review.opendev.org/c/openstack/python-openstackclient/+/958067 | |
| 15:51:29 | opendevreview | Masahito Muroi proposed openstack/openstacksdk master: Add microversion to tag operation API request https://review.opendev.org/c/openstack/openstacksdk/+/973956 | |
| 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] | 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: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: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: | |
| 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. | |
| 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 | |