| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-02-26 | |||
| 17:10:29 | gtema | it's a pure osc venv | |
| 17:10:57 | gtema | I just reinstalled devstack today, so it should be fresh | |
| 17:13:18 | stephenfin | okay, and if you create a wholly new venv and just install OSC from pypi, it works? | |
| 17:14:34 | gtema | yupp, still works | |
| 17:14:40 | stephenfin | that's the log I see https://paste.opendev.org/show/b410BGwb97j6BUXKicSE/ though this is an old devstack from last october | |
| 17:14:41 | stephenfin | weird | |
| 17:14:53 | stephenfin | okay, something with my devstack so | |
| 17:15:22 | gtema | this just states that user_domain_id is missing in the clouds.yaml | |
| 17:16:25 | stephenfin | right, but it's system-scoped: there's no domain | |
| 17:17:13 | stephenfin | this is the clouds.yaml entry I'm using. I don't think I modified it since devstack generated it https://paste.opendev.org/show/bcvFHACtwhUadXDv1mHK/ | |
| 17:19:52 | gtema | no domain for the scope, but not for the user | |
| 17:20:07 | gtema | user_domain_id has nothing to do with the scope - it is info about user's home | |
| 17:20:52 | gtema | ok, in my clouds.yaml it is also missing | |
| 17:21:28 | stephenfin | Fair. You said you just deployed devstack: does your devstack-system-admin entry have a user_domain_id field? | |
| 17:21:46 | stephenfin | Looking at tools/update_clouds_yaml.py and I can't see how that would get set | |
| 17:22:16 | stephenfin | (tools/update_clouds_yaml.py in the devstack repo) | |
| 17:23:41 | gtema | nope, as just said it doesn't | |
| 17:24:10 | gtema | do you have fresh keystone log? | |
| 17:24:40 | stephenfin | sorry, missed that message | |
| 17:25:00 | stephenfin | yes, I just pulled from master and restarted the devstack@keystone service | |
| 17:27:40 | stephenfin | ...but it's also an issue on 24.0.0 from 1.5 years ago | |
| 17:30:07 | gtema | with hacking keystone a bit to add debug I see that it tries to validate {'user': {'password': 'secret', 'name': 'admin', 'domain': {'id': 'default'}}} as input | |
| 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 | stephenfin | user_domain_id should be mandatory if using a username | |
| 17:57:56 | gtema | technically - yes | |
| 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: Add functional tests for limits, registered limits https://review.opendev.org/c/openstack/openstacksdk/+/942819 | |
| 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: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: test: Use specific cloud in functional test https://review.opendev.org/c/openstack/openstacksdk/+/942713 | |
| 19:01:51 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Rename cloud name variables https://review.opendev.org/c/openstack/openstacksdk/+/942712 | |
| 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: 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. | |