| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-02-28 | |||
| 21:52:14 | opendevreview | Manuel Osorio proposed openstack/openstacksdk master: Add share transfer to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/925653 | |
| 21:56:16 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] Identity: Migrate 'limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/936279 | |
| 22:03:00 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] Identity: Migrate 'limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/936279 | |
| 22:03:13 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] Identity: Migrate 'limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/936279 | |
| #openstack-sdks - 2025-03-01 | |||
| 05:23:19 | frickler | gtema: stephenfin: after https://review.opendev.org/c/openstack/python-openstackclient/+/929712 the "openstack role show" command is case-sensitive, which it wasn't before. arguably this could be considered a regression, if you call it a bug fix I think at least a reno might be needed | |
| 05:24:00 | frickler | amongst others this is breaking keystone jobs as in https://zuul.opendev.org/t/openstack/build/46f9132008204e3b9910b9d826af20ad as M0weng[m] discovered | |
| 05:28:40 | frickler | https://paste.opendev.org/show/bsDrVMhANQmJomtHRGms/ | |
| 09:02:43 | gtema | frickler - I noticed that also last night. But I need to object - OSC was and is case sensitive. I also checked that creation of the role with osc 7.2.0 and master work the same. I assume something has changed in the osc wrapper or wherelse. To not to spend too much time I proposed https://review.opendev.org/c/openstack/keystone/+/943091 fixing the test which is not correct, but was working before | |
| 09:06:19 | gtema | actually a keystone is throwing exception claiming that the role with this name already exists, so it doesnt look to me like a osc issue | |
| 15:38:22 | frickler | gtema: did you check my paste? that's pure OSC without a wrapper and with 7.2.1 or earlier the "role show" command is not case sensitive. both "role show Member" and "role show member" or even "mEmBeR" give the same role as result | |
| 15:40:56 | frickler | checking with debug, the keystone API request also is case-insensitive, the same "member" role is returned, but despite that result osc/sdk says "no result found" for 7.3.0 | |
| 16:13:40 | gtema | frickler - no I missed the paste. But I am speechless since that works for me (perhaps one more reason to dislike Apfel devices). Technically what has changed is that between 7.2.1 and 7.3.0 role methods are switched to sdk from old python-keystoneclient+osc-lib, but I do not see where case would be altered | |
| #openstack-sdks - 2025-03-03 | |||
| 10:10:16 | stephenfin | frickler: gtema: TIL that keystone is case-insensitive. The difference in behavior appears to be because Resource.find is case sensitive https://paste.opendev.org/show/bkHaAZHo7xaUKu2I5WwW/ | |
| 10:11:23 | stephenfin | Changing that globally is likely to be a large change that could have knock-on effects for other services that may not (a) allow filtering by name query string parameters and/or (b) are case sensitive. idk if we want to do that | |
| 10:11:46 | gtema | neah, we must stick to case sensitivity | |
| 10:12:22 | gtema | services themselves are free to be case sensitive or insensitive. SDK must respect case | |
| 10:12:26 | stephenfin | This is also presumably a change we've seen in the other identity resources that have switched to using SDK in recent releases. I'm guessing this is just the first one we've noticied | |
| 10:12:29 | stephenfin | *noticed | |
| 10:13:33 | stephenfin | If this is important enough as a regression, we probably need to temporarily reimplement Resource.find in OSC and issue warnings on case-insensitive matches | |
| 10:14:27 | stephenfin | s/as a/to be classified as a/ | |
| 10:14:28 | gtema | wrt keystone I see additional undocumented "case_sensitive" parameters support in the code | |
| 11:57:56 | frickler | stephenfin: gtema: how about defining this to be a feature instead of a regression, but being nice to consumers and add a reno with a big warning? | |
| 11:58:50 | gtema | I would also prefer feature rather than regression | |
| 13:09:11 | opendevreview | yatin proposed openstack/openstacksdk stable/2024.1: [Stable Only] Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/943177 | |
| 14:39:10 | opendevreview | yatin proposed openstack/openstacksdk stable/2024.1: [Stable Only] Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/943177 | |
| 14:39:36 | opendevreview | yatin proposed openstack/openstacksdk stable/2024.2: [Stable Only] Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/943182 | |
| 19:44:56 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Bootstrap magnum OpenAPI build https://review.opendev.org/c/openstack/codegenerator/+/943058 | |
| #openstack-sdks - 2025-03-04 | |||
| 10:23:01 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/943236 | |
| 11:21:12 | opendevreview | Merged openstack/openstacksdk stable/2024.2: [Stable Only] Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/943182 | |
| 12:09:29 | opendevreview | Merged openstack/openstacksdk stable/2024.1: [Stable Only] Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/943177 | |
| 12:59:15 | opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Fix DNS secondary zone creation https://review.opendev.org/c/openstack/openstacksdk/+/943245 | |
| 13:10:23 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Bump hacking in unit tests https://review.opendev.org/c/openstack/openstacksdk/+/943249 | |
| 14:08:43 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Bootstrap magnum OpenAPI build https://review.opendev.org/c/openstack/codegenerator/+/943058 | |
| 16:49:47 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] Identity: Migrate 'registered limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/942735 | |
| 16:50:04 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] Identity: Migrate 'registered limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/942735 | |
| 17:01:33 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] Identity: Migrate 'limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/936279 | |
| 17:02:43 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] Identity: Migrate 'registered limit' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/942735 | |
| 18:26:16 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Fix 'openstack keypair list --project |
|
| #openstack-sdks - 2025-03-05 | |||
| 12:01:09 | opendevreview | Merged openstack/codegenerator master: Bootstrap magnum OpenAPI build https://review.opendev.org/c/openstack/codegenerator/+/943058 | |
| 14:30:16 | opendevreview | Rajesh Tailor proposed openstack/python-openstackclient master: Fix missing space in help messages https://review.opendev.org/c/openstack/python-openstackclient/+/943398 | |
| 14:30:34 | opendevreview | Rajesh Tailor proposed openstack/openstacksdk master: Add support for showing finish_time https://review.opendev.org/c/openstack/openstacksdk/+/930562 | |
| 15:59:59 | opendevreview | Rajesh Tailor proposed openstack/python-openstackclient master: Add support for showing finish_time https://review.opendev.org/c/openstack/python-openstackclient/+/943407 | |
| 16:56:03 | opendevreview | Merged openstack/openstacksdk master: Bump hacking in unit tests https://review.opendev.org/c/openstack/openstacksdk/+/943249 | |
| 17:22:30 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add magnum metadata https://review.opendev.org/c/openstack/codegenerator/+/943416 | |
| 18:03:10 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add magnum metadata https://review.opendev.org/c/openstack/codegenerator/+/943416 | |
| #openstack-sdks - 2025-03-06 | |||
| 06:58:31 | opendevreview | Merged openstack/codegenerator master: Add magnum metadata https://review.opendev.org/c/openstack/codegenerator/+/943416 | |
| 08:54:10 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix network external_gateway_info schema https://review.opendev.org/c/openstack/codegenerator/+/943036 | |
| 10:13:20 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix network external_gateway_info schema https://review.opendev.org/c/openstack/codegenerator/+/943036 | |
| 10:33:42 | veith4f | Hello. Just stumbled over openstack project cleanup ... https://bugs.launchpad.net/openstacksdk/+bug/2100958. Seems like the fix is to just delete security groups after networks. | |
| 10:34:30 | veith4f_ | Hello. Just stumbled over openstack project cleanup ... https://bugs.launchpad.net/openstacksdk/+bug/2100958. Seems like the fix is to just delete security groups after networks. | |
| 11:22:08 | opendevreview | Fabian Schuetz proposed openstack/openstacksdk master: delete security groups after network stuff https://review.opendev.org/c/openstack/openstacksdk/+/943585 | |
| 17:45:11 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix network external_gateway_info schema https://review.opendev.org/c/openstack/codegenerator/+/943036 | |
| 17:52:51 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Actually start building magnum code https://review.opendev.org/c/openstack/codegenerator/+/943627 | |
| #openstack-sdks - 2025-03-07 | |||
| 08:11:30 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Actually start building magnum code https://review.opendev.org/c/openstack/codegenerator/+/943627 | |
| 09:50:01 | opendevreview | Merged openstack/codegenerator master: Fix network external_gateway_info schema https://review.opendev.org/c/openstack/codegenerator/+/943036 | |
| 10:33:50 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: resource: Drop 'action' arg for '_get_microversion' https://review.opendev.org/c/openstack/openstacksdk/+/943668 | |
| 10:33:50 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: resource: Make _assert_microversion_for a classmethod https://review.opendev.org/c/openstack/openstacksdk/+/943667 | |
| 10:33:51 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Add Service to volume v2 API https://review.opendev.org/c/openstack/openstacksdk/+/943670 | |
| 10:33:51 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Add service 'set-log', 'get-log' actions https://review.opendev.org/c/openstack/openstacksdk/+/943669 | |
| 10:33:52 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Sort proxy docs alphabetically https://review.opendev.org/c/openstack/openstacksdk/+/943671 | |
| 13:20:22 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Add support for cascaded deletions https://review.opendev.org/c/openstack/openstacksdk/+/943686 | |
| 13:20:23 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Use force delete param when possible https://review.opendev.org/c/openstack/openstacksdk/+/943687 | |
| 14:23:27 | opendevreview | OpenStack Release Bot proposed openstack/cliff stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/cliff/+/943693 | |
| 14:23:29 | opendevreview | OpenStack Release Bot proposed openstack/cliff stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/cliff/+/943694 | |
| 14:23:43 | opendevreview | OpenStack Release Bot proposed openstack/openstacksdk stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/openstacksdk/+/943695 | |
| 14:23:45 | opendevreview | OpenStack Release Bot proposed openstack/openstacksdk stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/openstacksdk/+/943696 | |
| 14:23:48 | opendevreview | OpenStack Release Bot proposed openstack/openstacksdk master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/openstacksdk/+/943697 | |
| 14:23:59 | opendevreview | OpenStack Release Bot proposed openstack/osc-lib stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/osc-lib/+/943698 | |
| 14:24:01 | opendevreview | OpenStack Release Bot proposed openstack/osc-lib stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/osc-lib/+/943699 | |
| 14:24:03 | opendevreview | OpenStack Release Bot proposed openstack/osc-lib master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/osc-lib/+/943700 | |
| 14:24:54 | opendevreview | OpenStack Release Bot proposed openstack/keystoneauth stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/keystoneauth/+/943704 | |
| 14:24:55 | opendevreview | OpenStack Release Bot proposed openstack/keystoneauth stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/keystoneauth/+/943705 | |
| 14:24:56 | opendevreview | OpenStack Release Bot proposed openstack/keystoneauth master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/keystoneauth/+/943706 | |
| 15:36:34 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Add missing fields to Volume https://review.opendev.org/c/openstack/openstacksdk/+/943800 | |
| 16:36:58 | opendevreview | Merged openstack/codegenerator master: Actually start building magnum code https://review.opendev.org/c/openstack/codegenerator/+/943627 | |
| 17:56:25 | opendevreview | Pierre Riteau proposed openstack/keystoneauth master: Use releases.openstack.org instead of opendev.org https://review.opendev.org/c/openstack/keystoneauth/+/943809 | |
| #openstack-sdks - 2025-03-10 | |||
| 11:45:15 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Use consistent reset_*_state functions https://review.opendev.org/c/openstack/openstacksdk/+/943919 | |
| 11:45:16 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Add backup metadata helpers https://review.opendev.org/c/openstack/openstacksdk/+/943920 | |
| 12:39:16 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Add backup metadata helpers https://review.opendev.org/c/openstack/openstacksdk/+/943920 | |
| 14:36:42 | opendevreview | Merged openstack/openstacksdk master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/openstacksdk/+/943697 | |
| 14:36:43 | opendevreview | Merged openstack/openstacksdk stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/openstacksdk/+/943695 | |
| 16:36:39 | opendevreview | Vladimir Kozhukalov proposed openstack/python-openstackclient master: identity: Fix 'trust' commands to work with SDK https://review.opendev.org/c/openstack/python-openstackclient/+/943972 | |
| 16:54:35 | opendevreview | Vladimir Kozhukalov proposed openstack/python-openstackclient master: identity: Fix 'trust' commands to work with SDK https://review.opendev.org/c/openstack/python-openstackclient/+/943972 | |
| 17:14:15 | opendevreview | Vladimir Kozhukalov proposed openstack/python-openstackclient master: identity: Fix 'trust' commands to work with SDK https://review.opendev.org/c/openstack/python-openstackclient/+/943972 | |
| 18:41:16 | opendevreview | Vladimir Kozhukalov proposed openstack/python-openstackclient master: identity: Fix 'trust' commands to work with SDK https://review.opendev.org/c/openstack/python-openstackclient/+/943972 | |
| #openstack-sdks - 2025-03-11 | |||
| 13:05:30 | oschwart | hello o/ can anyone please review this one? https://review.opendev.org/c/openstack/openstacksdk/+/943245 | |
| 13:05:33 | oschwart | thanks | |
| 16:23:42 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Fix token-based auth https://review.opendev.org/c/openstack/openstacksdk/+/944057 | |
| 17:31:55 | opendevreview | Vladimir Kozhukalov proposed openstack/python-openstackclient master: identity: Fix 'trust' commands to work with SDK https://review.opendev.org/c/openstack/python-openstackclient/+/943972 | |
| #openstack-sdks - 2025-03-12 | |||
| 00:52:53 | opendevreview | Michael Still proposed openstack/openstacksdk master: Add support for spice-direct console types. https://review.opendev.org/c/openstack/openstacksdk/+/940479 | |
| 00:55:39 | opendevreview | Michael Still proposed openstack/python-openstackclient master: Add support for spice-direct console types. https://review.opendev.org/c/openstack/python-openstackclient/+/940480 | |
| 09:52:49 | opendevreview | Rajesh Tailor proposed openstack/openstacksdk master: Add support for showing finish_time https://review.opendev.org/c/openstack/openstacksdk/+/930562 | |
| 11:24:40 | stephenfin | gtema: Are we able to merge stuff again, now that the 2025.1 branch has been created? | |
| 11:27:07 | gtema | Stephenfin: yeah, I wasn't following when branch creation was finished | |
| 14:38:09 | opendevreview | Andrew Bonney proposed openstack/ansible-collections-openstack master: Add support for managing network segments https://review.opendev.org/c/openstack/ansible-collections-openstack/+/944121 | |
| 14:50:46 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Refactor and expand loader backward compat tests https://review.opendev.org/c/openstack/openstacksdk/+/944122 | |
| 14:50:46 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Fix token-based auth loading from config https://review.opendev.org/c/openstack/openstacksdk/+/944057 | |
| 14:50:47 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Refactor loader magic fixes https://review.opendev.org/c/openstack/openstacksdk/+/944123 | |
| 16:03:51 | opendevreview | Tobias Urdin proposed openstack/python-openstackclient master: Add device ID and device owner to port unset https://review.opendev.org/c/openstack/python-openstackclient/+/944130 | |
| 19:39:04 | opendevreview | Tobias Urdin proposed openstack/python-openstackclient master: Add device ID and device owner to port unset https://review.opendev.org/c/openstack/python-openstackclient/+/944130 | |