| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-02-28 | |||
| 12:21:48 | stephenfin | Yeah, I'll come up with something. I first trying to figure out why it has that impact though | |
| 12:22:16 | gtema | great, I am afk for now, but feel free to ping me | |
| 12:23:18 | stephenfin | As for why I changed that, we're asserting that if you give e.g. data_type=str, the output will a str | |
| 12:23:57 | stephenfin | If we just returned the value the user provided without doing anything to it, you do e.g.: | |
| 12:24:14 | stephenfin | _conver_type(123, data_type=list) | |
| 12:24:26 | stephenfin | but the result will be the int 123 | |
| 12:24:41 | stephenfin | which is obviously wrong, and mypy complained as such | |
| 12:25:00 | stephenfin | hence the change. and the horrific type hints :) | |
| 12:36:50 | stephenfin | gtema: Okay, there's a bigger issue with _convert_type here. It popped up in older releases if you passed list_type also. For example https://paste.opendev.org/show/bZOYPdh75lIJLal2P65T/ | |
| 12:37:23 | stephenfin | idk if we should/can support this, at least while continuing to offer type conversion :-\ | |
| 12:41:48 | stephenfin | tbc, the issue is that we're using this inside '__getattribute__', and '_convert_type' is occasionally returning a _copy_ of a field, rather than the field itself | |
| 12:42:13 | stephenfin | Thus, attempts to modify a field in-place modify the copy rather than the "source" | |
| 12:42:40 | stephenfin | To fix this properly, I think we need | |
| 12:42:47 | stephenfin | ...to kill _convert_type | |
| 12:44:12 | gtema | We should rather redesign everything from OpenApi anyway | |
| 13:22:19 | stephenfin | https://review.opendev.org/c/openstack/horizon/+/943001 | |
| 13:52:06 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Support server unshelve to specific availability zone https://review.opendev.org/c/openstack/openstacksdk/+/942877 | |
| 15:02:13 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: fields: Save converted attributes before returning https://review.opendev.org/c/openstack/openstacksdk/+/943009 | |
| 15:02:25 | stephenfin | gtema: ^ | |
| 15:19:58 | opendevreview | Merged openstack/cliff master: command: Filter on empty epilog, not empty hooks https://review.opendev.org/c/openstack/cliff/+/942992 | |
| 15:19:59 | opendevreview | Merged openstack/cliff master: hooks: Update type hints to reflect reality https://review.opendev.org/c/openstack/cliff/+/942993 | |
| 16:26:18 | opendevreview | Douglas Viroel proposed openstack/openstacksdk master: Bump compute max microversion to 2.100 https://review.opendev.org/c/openstack/openstacksdk/+/938833 | |
| 16:54:52 | priteau | stephenfin: Thanks a lot for the quick fix. It looks like our CI is green again. | |
| 17:34:39 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix network external_gateway_info schema https://review.opendev.org/c/openstack/codegenerator/+/943036 | |
| 20:03:12 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Bootstrap magnum OpenAPI build https://review.opendev.org/c/openstack/codegenerator/+/943058 | |
| 20:11:58 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Bootstrap magnum OpenAPI build https://review.opendev.org/c/openstack/codegenerator/+/943058 | |
| 20:13:51 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Bootstrap magnum OpenAPI build https://review.opendev.org/c/openstack/codegenerator/+/943058 | |
| 20:25:15 | opendevreview | Manuel Osorio proposed openstack/openstacksdk master: Add share transfer to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/925653 | |
| 20:43:34 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Bootstrap magnum OpenAPI build https://review.opendev.org/c/openstack/codegenerator/+/943058 | |
| 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 | |