| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-02-26 | |||
| 20:45:52 | opendevreview | Merged openstack/openstacksdk master: cloud: Deprecate jmespath-style filters https://review.opendev.org/c/openstack/openstacksdk/+/942609 | |
| 20:45:54 | opendevreview | Merged openstack/openstacksdk master: typing: Add (more) missing proxy arguments https://review.opendev.org/c/openstack/openstacksdk/+/942782 | |
| 20:45:56 | opendevreview | Merged openstack/openstacksdk master: proxy: Remove '_check_resource' decorator https://review.opendev.org/c/openstack/openstacksdk/+/942597 | |
| 21:04:47 | opendevreview | Merged openstack/openstacksdk master: deps: Bump minimum keystoneauth https://review.opendev.org/c/openstack/openstacksdk/+/942610 | |
| 21:04:48 | opendevreview | Merged openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-manila voting https://review.opendev.org/c/openstack/openstacksdk/+/942624 | |
| 21:04:50 | opendevreview | Merged openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-masakari voting https://review.opendev.org/c/openstack/openstacksdk/+/942626 | |
| #openstack-sdks - 2025-02-27 | |||
| 03:34:42 | opendevreview | Sina Sadeghi proposed openstack/openstacksdk master: Support server unshelve to specific availability zone https://review.opendev.org/c/openstack/openstacksdk/+/942877 | |
| 06:53:18 | opendevreview | Rajesh Tailor proposed openstack/openstacksdk master: Add support for image properties in server show https://review.opendev.org/c/openstack/openstacksdk/+/940798 | |
| 11:38:15 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: test: Use specific cloud in functional test https://review.opendev.org/c/openstack/openstacksdk/+/942713 | |
| 11:38:15 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Rename cloud name variables https://review.opendev.org/c/openstack/openstacksdk/+/942712 | |
| 11:38:16 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Use correct cloud https://review.opendev.org/c/openstack/openstacksdk/+/942892 | |
| 11:38:16 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Remove redundant _set_operator_cloud calls https://review.opendev.org/c/openstack/openstacksdk/+/942891 | |
| 11:38:17 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Streamline keypair tests https://review.opendev.org/c/openstack/openstacksdk/+/942893 | |
| 11:41:48 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.connection https://review.opendev.org/c/openstack/openstacksdk/+/942599 | |
| 11:41:48 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.proxy https://review.opendev.org/c/openstack/openstacksdk/+/942598 | |
| 11:41:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: typing: Annotate openstack.service_description https://review.opendev.org/c/openstack/openstacksdk/+/942600 | |
| 13:19:58 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-ironic voting https://review.opendev.org/c/openstack/openstacksdk/+/942705 | |
| 13:19:59 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-magnum voting https://review.opendev.org/c/openstack/openstacksdk/+/942627 | |
| 13:19:59 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: zuul: Make dib-nodepool-functional-openstack-ubuntu-noble-src voting https://review.opendev.org/c/openstack/openstacksdk/+/942628 | |
| 13:31:00 | opendevreview | Rajesh Tailor proposed openstack/openstacksdk master: Add support for image properties in server show https://review.opendev.org/c/openstack/openstacksdk/+/940798 | |
| 13:40:24 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Drop support for testing identity v2 https://review.opendev.org/c/openstack/openstacksdk/+/942898 | |
| 13:58:35 | ralonsoh | stephenfin, hello! If you have less than one minute: https://review.opendev.org/c/openstack/python-openstackclient/+/942175 | |
| 13:58:40 | ralonsoh | thanks in advance | |
| 14:14:53 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Prepare for ruff bump https://review.opendev.org/c/openstack/python-openstackclient/+/942274 | |
| 14:14:54 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Indicate tuples to be extended https://review.opendev.org/c/openstack/python-openstackclient/+/942276 | |
| 14:14:54 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: pre-commit: Bump versions https://review.opendev.org/c/openstack/python-openstackclient/+/942275 | |
| 14:14:55 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Use consistent types https://review.opendev.org/c/openstack/python-openstackclient/+/942278 | |
| 14:14:55 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Add types for empty dicts, tuples https://review.opendev.org/c/openstack/python-openstackclient/+/942277 | |
| 14:14:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Correct type for missing attributes https://review.opendev.org/c/openstack/python-openstackclient/+/942280 | |
| 14:14:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Remove use of optional imports https://review.opendev.org/c/openstack/python-openstackclient/+/942279 | |
| 14:14:57 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Resolve incompatible operand issues https://review.opendev.org/c/openstack/python-openstackclient/+/942282 | |
| 14:14:57 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Correct type for missing attributes https://review.opendev.org/c/openstack/python-openstackclient/+/942281 | |
| 14:14:58 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: pre-commit: Enable mypy https://review.opendev.org/c/openstack/python-openstackclient/+/942283 | |
| 16:10:58 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: BS volume bootable parameter is a string https://review.opendev.org/c/openstack/codegenerator/+/942920 | |
| 16:26:18 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: identity: Add functional tests for limits, registered limits https://review.opendev.org/c/openstack/openstacksdk/+/942819 | |
| 17:16:42 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add required to compute service responses https://review.opendev.org/c/openstack/codegenerator/+/942934 | |
| 22:14:33 | M0weng[m] | <stephenfin> "0weng: mea culpa: looks like I..." <- Thank you!! 🙏 | |
| 22:17:39 | 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 | |
| 23:06:48 | M0weng[m] | Uh oh, I got an internal server error when trying to update a federation protocol's remote_id_attribute only:... (full message at <https://matrix.org/oftc/media/v1/media/download/AcXQUdcnE8LuNEEjXHuiKeEWrFA55TiLOD29oGhp3ZbkicDx4zxI1G4im0Zt-fQMtYG0Le1RpPpuGMKdswrfz29CeVkcKpUwAG1hdHJpeC5vcmcvTFJuTVRqbGpSTkNybnZQQ2VBTXZtb211>) | |
| 23:10:09 | M0weng[m] | I have also been able to successfully edit just the mapping_id or both the mapping_id and the remote_id_attribute together, just not remote_id_attribute by itself. | |
| 23:49:53 | opendevreview | Merged openstack/codegenerator master: Hardcode barbican quota schemas https://review.opendev.org/c/openstack/codegenerator/+/941543 | |
| #openstack-sdks - 2025-02-28 | |||
| 01:43:28 | opendevreview | Merged openstack/python-openstackclient master: Add four new network agent types to the list command filter https://review.opendev.org/c/openstack/python-openstackclient/+/942175 | |
| 01:43:29 | opendevreview | Merged openstack/openstacksdk master: exceptions: Deprecate http_status, request_id params https://review.opendev.org/c/openstack/openstacksdk/+/929017 | |
| 02:14:02 | opendevreview | Merged openstack/openstacksdk master: tests: Rename cloud name variables https://review.opendev.org/c/openstack/openstacksdk/+/942712 | |
| 02:14:04 | opendevreview | Merged openstack/openstacksdk master: identity: Fix limit, registered limit creation https://review.opendev.org/c/openstack/openstacksdk/+/942818 | |
| 08:08:49 | opendevreview | Merged openstack/codegenerator master: BS volume bootable parameter is a string https://review.opendev.org/c/openstack/codegenerator/+/942920 | |
| 08:09:33 | opendevreview | Merged openstack/codegenerator master: Add required to compute service responses https://review.opendev.org/c/openstack/codegenerator/+/942934 | |
| 08:40:13 | opendevreview | Rajesh Tailor proposed openstack/python-openstackclient master: Add support for image properties in server show https://review.opendev.org/c/openstack/python-openstackclient/+/940799 | |
| 08:41:26 | opendevreview | Rajesh Tailor proposed openstack/openstacksdk master: Add support for image properties in server show https://review.opendev.org/c/openstack/openstacksdk/+/940798 | |
| 08:46:26 | priteau | Hello stephenfin. The kayobe CLI broke yesterday following the release of cliff 4.9.0. I have opened a bug with more details here: https://bugs.launchpad.net/kayobe/+bug/2100579. I don't know yet if this is a regression in cliff or if kayobe is using it wrongly. | |
| 10:50:13 | stephenfin | priteau: Thanks. I'll take a look now | |
| 11:13:30 | stephenfin | priteau: So two things. First, it's a definite logic mistake in cliff: the filter and list comprehensions are not the same thing. I'll fix that now. | |
| 11:14:46 | stephenfin | Secondly: the CommandHook class stated that get_parser method should return an ArgumentParser instance, while get_epilog should return a string. Kayobe's 'HookDispatcher' is returning None for both. | |
| 11:15:34 | stephenfin | But, seeing as that's been tolerated before and there was the filter for the epilog, I'm inclined to think that cliff's own docstrings were wrong | |
| 11:38:36 | opendevreview | Stephen Finucane proposed openstack/cliff master: command: Filter on empty epilog, not empty hooks https://review.opendev.org/c/openstack/cliff/+/942992 | |
| 11:38:37 | opendevreview | Stephen Finucane proposed openstack/cliff master: hooks: Update type hints to reflect reality https://review.opendev.org/c/openstack/cliff/+/942993 | |
| 11:38:46 | stephenfin | gtema, priteau: ^ | |
| 11:50:48 | priteau | stephenfin: I have just tested 942992, it resolves the issue in kayobe | |
| 11:51:15 | priteau | Would you be able to push a new release soon? | |
| 11:51:48 | stephenfin | Yes, I'd just like gtema to sanity check it, then we can merge and cut a 4.9.1 release | |
| 11:52:06 | gtema | sure | |
| 11:53:34 | stephenfin | https://review.opendev.org/c/openstack/releases/+/942996 There's a release patch. We'll just recheck it once those patches are merged. | |
| 12:02:05 | priteau | stephenfin: Would you still recommend we fix the HookDispatcher class? I can look into it once our CI is green again. | |
| 12:04:25 | stephenfin | priteau: You can fix your CI now by returning '' from get_epilog https://paste.opendev.org/show/bgLM8MsGNB0sWVVtjzsS/ | |
| 12:04:51 | stephenfin | As an alternative to capping and uncapping | |
| 12:06:26 | priteau | Any CI fix we do ourselves involves fixing the oldest branches first, because we have upgrade jobs that are failing. We don't use upper constraints in kayobe so a new release would magically fix all our branches :) | |
| 12:07:47 | priteau | Then I can fix get_epilog in master and backport the usual way | |
| 12:18:07 | stephenfin | priteau: Okay, then maybe hold tight until this evening and we'll hopefully be able to get that fix out sharpish | |
| 12:18:52 | stephenfin | otherwise, IMO you might as well apply the fix to all branches rather than a cap followed by an uncap. It's literally half the work :) | |
| 12:19:01 | gtema | stephefin - I assume we may want to add explicit test like I posted in releases channel for capturing regressions | |
| 12:19:33 | stephenfin | sorry, what explicit test? | |
| 12:19:39 | gtema | https://paste.openstack.org/show/byZpxP2y2ujhv4TCKehT/ | |
| 12:19:43 | gtema | something like that | |
| 12:19:51 | gtema | just convert print to assert | |
| 12:19:54 | stephenfin | ah, yes | |
| 12:20:52 | gtema | I assume you have idea how to fix the regression assuming you introduced this change? You should have had reasons for that | |
| 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 | |