| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-02-26 | |||
| 17:31:37 | gtema | totally weird is that osc debug log also states: Using auth plugin: password Using parameters {'auth_url': 'http://192.168.1.3/identity', 'system_scope': 'all', 'project_domain_id': 'default', 'username': 'admin', 'user_domain_id': 'default', 'password': '***'} | |
| 17:32:19 | stephenfin | Okay, something is regressed here so. I see: | |
| 17:32:20 | stephenfin | Using parameters {'auth_url': 'http://10.0.110.125/identity', 'system_scope': 'all', 'username': 'admin', 'password': '***'} | |
| 17:32:44 | stephenfin | no project_domain_id or user_domain_id fields | |
| 17:32:55 | stephenfin | and openstackclient on the devstack host works just fine | |
| 17:33:39 | gtema | yeah, this is how I test it now - osc directly on the devstack host | |
| 17:42:13 | gtema | stephenfin, at least in the osc-lib I see setting of the default_domain | |
| 17:43:04 | gtema | https://opendev.org/openstack/osc-lib/src/branch/master/osc_lib/shell.py#L44 | |
| 17:55:42 | stephenfin | gtema: Got to the bottom of it :( My local copy was missing the `identity_api_version: '3'` line | |
| 17:55:59 | gtema | he, very weird | |
| 17:56:01 | stephenfin | Looks like osc-lib only generates a fake domain if that's present | |
| 17:57:17 | gtema | that's anyway totally insane how complex is it to find what and how sets you defaults | |
| 17:57:28 | gtema | and moreover: why? | |
| 17:57:41 | stephenfin | I think the clouds.yaml entry is invalid, tbh | |
| 17:57:56 | gtema | technically - yes | |
| 17:57:56 | stephenfin | user_domain_id should be mandatory if using a username | |
| 17:58:16 | gtema | or user_domain_name - that is one of the other creepy things | |
| 17:59:12 | stephenfin | same thing, right? | |
| 17:59:24 | stephenfin | well, for Default | |
| 17:59:39 | gtema | well, one of user_domain_id or user_domain_name must be present | |
| 18:00:01 | gtema | in reality even default domain id and name may differ | |
| 18:00:12 | gtema | it is "default" vs "Default" in devstack | |
| 18:00:30 | gtema | sorry, need to run away again. | |
| 18:00:33 | stephenfin | o/ | |
| 18:12:49 | stephenfin | gtema: https://review.opendev.org/c/openstack/devstack/+/942840 jfyi | |
| 18:13:12 | opendevreview | Merged openstack/codegenerator master: Hardcode barbican secret_store schemas https://review.opendev.org/c/openstack/codegenerator/+/941374 | |
| 18:13:31 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Fix the Ironic functional tests to use the correct scope https://review.opendev.org/c/openstack/openstacksdk/+/942705 | |
| 18:40:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: identity: Fix limit, registered limit creation https://review.opendev.org/c/openstack/openstacksdk/+/942818 | |
| 18:40:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: identity: Add functional tests for limits, registered limits https://review.opendev.org/c/openstack/openstacksdk/+/942819 | |
| 18:40:13 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: tests: Add reference to system admin credentials https://review.opendev.org/c/openstack/openstacksdk/+/942820 | |
| 18:41:41 | opendevreview | Manuel Osorio proposed openstack/openstacksdk master: Add share transfer to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/925653 | |
| 18:54:39 | opendevreview | Merged openstack/codegenerator master: Hardcode barbican container schemas https://review.opendev.org/c/openstack/codegenerator/+/941406 | |
| 18:54:40 | opendevreview | Merged openstack/codegenerator master: Hardcode barbican order schema https://review.opendev.org/c/openstack/codegenerator/+/941542 | |
| 18:57:39 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: exceptions: Deprecate http_status, request_id params https://review.opendev.org/c/openstack/openstacksdk/+/929017 | |
| 19:01:51 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Rename cloud name variables https://review.opendev.org/c/openstack/openstacksdk/+/942712 | |
| 19:01:51 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: test: Use specific cloud in functional test https://review.opendev.org/c/openstack/openstacksdk/+/942713 | |
| 20:45:48 | opendevreview | Merged openstack/openstacksdk master: cloud: Switch remaining compute functions to proxy https://review.opendev.org/c/openstack/openstacksdk/+/923426 | |
| 20:45:50 | opendevreview | Merged openstack/openstacksdk master: cloud: Add missing parameter docs https://review.opendev.org/c/openstack/openstacksdk/+/923427 | |
| 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: tests: Rename cloud name variables https://review.opendev.org/c/openstack/openstacksdk/+/942712 | |
| 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: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:16 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Use correct cloud https://review.opendev.org/c/openstack/openstacksdk/+/942892 | |
| 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.proxy https://review.opendev.org/c/openstack/openstacksdk/+/942598 | |
| 11:41:48 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.connection https://review.opendev.org/c/openstack/openstacksdk/+/942599 | |
| 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 dib-nodepool-functional-openstack-ubuntu-noble-src voting https://review.opendev.org/c/openstack/openstacksdk/+/942628 | |
| 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: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: pre-commit: Bump versions https://review.opendev.org/c/openstack/python-openstackclient/+/942275 | |
| 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: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:55 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: typing: Use consistent types https://review.opendev.org/c/openstack/python-openstackclient/+/942278 | |
| 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: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: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: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: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. | |