| 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 | 00:25 |
|---|---|---|
| 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 | 00:28 |
| 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 | 00:29 |
| opendevreview | Manuel Osorio proposed openstack/openstacksdk master: Add share transfer to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/925653 | 02:00 |
| Yanks | Hello team! I've got this streaming image download support PR: https://review.opendev.org/c/openstack/openstacksdk/+/927778 I've resolved all the comments as well, may I know if this could be merged? :) | 02:23 |
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: Ensure BS responses have some properties marked as required https://review.opendev.org/c/openstack/codegenerator/+/942631 | 07:54 |
| opendevreview | Rajesh Tailor proposed openstack/openstacksdk master: Add support for image properties in server show https://review.opendev.org/c/openstack/openstacksdk/+/940798 | 10:05 |
| 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 | 10:36 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: proxy: Remove '_check_resource' decorator https://review.opendev.org/c/openstack/openstacksdk/+/942597 | 11:26 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.proxy https://review.opendev.org/c/openstack/openstacksdk/+/942598 | 11:26 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.connection https://review.opendev.org/c/openstack/openstacksdk/+/942599 | 11:26 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: typing: Annotate openstack.service_description https://review.opendev.org/c/openstack/openstacksdk/+/942600 | 11:26 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Add (more) missing proxy arguments https://review.opendev.org/c/openstack/openstacksdk/+/942782 | 11:26 |
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: Ensure BS responses have some properties marked as required https://review.opendev.org/c/openstack/codegenerator/+/942631 | 11:34 |
| opendevreview | Merged openstack/openstacksdk master: typing: Don't abuse Proxy._update https://review.opendev.org/c/openstack/openstacksdk/+/942685 | 11:45 |
| opendevreview | Merged openstack/openstacksdk master: typing: Remove duplicate TypeVar https://review.opendev.org/c/openstack/openstacksdk/+/942596 | 11:55 |
| opendevreview | Merged openstack/codegenerator master: Ensure BS responses have some properties marked as required https://review.opendev.org/c/openstack/codegenerator/+/942631 | 14:10 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: identity: Fix limit, registered limit creation https://review.opendev.org/c/openstack/openstacksdk/+/942818 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: identity: Add functional tests for limits, registered limits https://review.opendev.org/c/openstack/openstacksdk/+/942819 | 15:33 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: tests: Add reference to system admin credentials https://review.opendev.org/c/openstack/openstacksdk/+/942820 | 15:33 |
| stephenfin | M0weng[m]: mea culpa: looks like I broke you. 👆 should fix things | 15:34 |
| opendevreview | Merged openstack/codegenerator master: Add barbican secret schemas https://review.opendev.org/c/openstack/codegenerator/+/941234 | 15:44 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Add (more) missing proxy arguments https://review.opendev.org/c/openstack/openstacksdk/+/942782 | 15:56 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: proxy: Remove '_check_resource' decorator https://review.opendev.org/c/openstack/openstacksdk/+/942597 | 15:56 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.proxy https://review.opendev.org/c/openstack/openstacksdk/+/942598 | 15:56 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.connection https://review.opendev.org/c/openstack/openstacksdk/+/942599 | 15:56 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: typing: Annotate openstack.service_description https://review.opendev.org/c/openstack/openstacksdk/+/942600 | 15:56 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.proxy https://review.opendev.org/c/openstack/openstacksdk/+/942598 | 16:22 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.connection https://review.opendev.org/c/openstack/openstacksdk/+/942599 | 16:22 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: typing: Annotate openstack.service_description https://review.opendev.org/c/openstack/openstacksdk/+/942600 | 16:22 |
| stephenfin | gtema: If I run 'openstack --os-cloud devstack-system-admin registered limit list', I get the following error | 17:06 |
| stephenfin | xpecting to find domain in user. The server could not comply with the request since it is either malformed or otherwise incorrect. The client is assumed to be in error. (HTTP 400) | 17:06 |
| stephenfin | Is that expected? How does one do system-scoped operations otherwise? | 17:06 |
| * stephenfin thought he'd broken something with the recent typing changes to ksa, but it reproduces on ksa 5.8.0 which predates those | 17:07 | |
| gtema | stephenfin - it works for me on devstack now | 17:07 |
| gtema | at least with 7.2.1 as I see now | 17:07 |
| gtema | potentially we broke it with some of the changes recently | 17:08 |
| stephenfin | I'm seeing it fail on dtantsur's change here https://zuul.opendev.org/t/openstack/build/e745767e1b0d4e6d86a0dee43238478a | 17:08 |
| stephenfin | is that OSC in a virtualenv? could you share your `pip freeze` output? | 17:09 |
| gtema | hmm, even with 7.3.2 on devstack it works | 17:09 |
| gtema | https://paste.openstack.org/show/bHixZI149prOY0IyMEW7/ | 17:10 |
| stephenfin | it could be my devstack, but as I said I also see it on Dmitry's patch | 17:10 |
| gtema | it's a pure osc venv | 17:10 |
| gtema | I just reinstalled devstack today, so it should be fresh | 17:10 |
| stephenfin | okay, and if you create a wholly new venv and just install OSC from pypi, it works? | 17:13 |
| gtema | yupp, still works | 17:14 |
| stephenfin | that's the log I see https://paste.opendev.org/show/b410BGwb97j6BUXKicSE/ though this is an old devstack from last october | 17:14 |
| stephenfin | weird | 17:14 |
| stephenfin | okay, something with my devstack so | 17:14 |
| gtema | this just states that user_domain_id is missing in the clouds.yaml | 17:15 |
| stephenfin | right, but it's system-scoped: there's no domain | 17:16 |
| 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:17 |
| gtema | no domain for the scope, but not for the user | 17:19 |
| gtema | user_domain_id has nothing to do with the scope - it is info about user's home | 17:20 |
| gtema | ok, in my clouds.yaml it is also missing | 17:20 |
| stephenfin | Fair. You said you just deployed devstack: does your devstack-system-admin entry have a user_domain_id field? | 17:21 |
| stephenfin | Looking at tools/update_clouds_yaml.py and I can't see how that would get set | 17:21 |
| stephenfin | (tools/update_clouds_yaml.py in the devstack repo) | 17:22 |
| gtema | nope, as just said it doesn't | 17:23 |
| gtema | do you have fresh keystone log? | 17:24 |
| stephenfin | sorry, missed that message | 17:24 |
| stephenfin | yes, I just pulled from master and restarted the devstack@keystone service | 17:25 |
| stephenfin | ...but it's also an issue on 24.0.0 from 1.5 years ago | 17:27 |
| 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:30 |
| 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:31 |
| stephenfin | Okay, something is regressed here so. I see: | 17:32 |
| stephenfin | Using parameters {'auth_url': 'http://10.0.110.125/identity', 'system_scope': 'all', 'username': 'admin', 'password': '***'} | 17:32 |
| stephenfin | no project_domain_id or user_domain_id fields | 17:32 |
| stephenfin | and openstackclient on the devstack host works just fine | 17:32 |
| gtema | yeah, this is how I test it now - osc directly on the devstack host | 17:33 |
| gtema | stephenfin, at least in the osc-lib I see setting of the default_domain | 17:42 |
| gtema | https://opendev.org/openstack/osc-lib/src/branch/master/osc_lib/shell.py#L44 | 17:43 |
| stephenfin | gtema: Got to the bottom of it :( My local copy was missing the `identity_api_version: '3'` line | 17:55 |
| gtema | he, very weird | 17:55 |
| stephenfin | Looks like osc-lib only generates a fake domain if that's present | 17:56 |
| gtema | that's anyway totally insane how complex is it to find what and how sets you defaults | 17:57 |
| gtema | and moreover: why? | 17:57 |
| stephenfin | I think the clouds.yaml entry is invalid, tbh | 17:57 |
| gtema | technically - yes | 17:57 |
| stephenfin | user_domain_id should be mandatory if using a username | 17:57 |
| gtema | or user_domain_name - that is one of the other creepy things | 17:58 |
| stephenfin | same thing, right? | 17:59 |
| stephenfin | well, for Default | 17:59 |
| gtema | well, one of user_domain_id or user_domain_name must be present | 17:59 |
| gtema | in reality even default domain id and name may differ | 18:00 |
| gtema | it is "default" vs "Default" in devstack | 18:00 |
| gtema | sorry, need to run away again. | 18:00 |
| stephenfin | o/ | 18:00 |
| stephenfin | gtema: https://review.opendev.org/c/openstack/devstack/+/942840 jfyi | 18:12 |
| opendevreview | Merged openstack/codegenerator master: Hardcode barbican secret_store schemas https://review.opendev.org/c/openstack/codegenerator/+/941374 | 18:13 |
| 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:13 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: identity: Fix limit, registered limit creation https://review.opendev.org/c/openstack/openstacksdk/+/942818 | 18:40 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: tests: Add reference to system admin credentials https://review.opendev.org/c/openstack/openstacksdk/+/942820 | 18:40 |
| opendevreview | Manuel Osorio proposed openstack/openstacksdk master: Add share transfer to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/925653 | 18:41 |
| opendevreview | Merged openstack/codegenerator master: Hardcode barbican container schemas https://review.opendev.org/c/openstack/codegenerator/+/941406 | 18:54 |
| opendevreview | Merged openstack/codegenerator master: Hardcode barbican order schema https://review.opendev.org/c/openstack/codegenerator/+/941542 | 18:54 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: exceptions: Deprecate http_status, request_id params https://review.opendev.org/c/openstack/openstacksdk/+/929017 | 18:57 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Rename cloud name variables https://review.opendev.org/c/openstack/openstacksdk/+/942712 | 19:01 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: test: Use specific cloud in functional test https://review.opendev.org/c/openstack/openstacksdk/+/942713 | 19:01 |
| opendevreview | Merged openstack/openstacksdk master: cloud: Switch remaining compute functions to proxy https://review.opendev.org/c/openstack/openstacksdk/+/923426 | 20:45 |
| opendevreview | Merged openstack/openstacksdk master: cloud: Add missing parameter docs https://review.opendev.org/c/openstack/openstacksdk/+/923427 | 20:45 |
| opendevreview | Merged openstack/openstacksdk master: cloud: Deprecate jmespath-style filters https://review.opendev.org/c/openstack/openstacksdk/+/942609 | 20:45 |
| opendevreview | Merged openstack/openstacksdk master: typing: Add (more) missing proxy arguments https://review.opendev.org/c/openstack/openstacksdk/+/942782 | 20:45 |
| opendevreview | Merged openstack/openstacksdk master: proxy: Remove '_check_resource' decorator https://review.opendev.org/c/openstack/openstacksdk/+/942597 | 20:45 |
| opendevreview | Merged openstack/openstacksdk master: deps: Bump minimum keystoneauth https://review.opendev.org/c/openstack/openstacksdk/+/942610 | 21:04 |
| opendevreview | Merged openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-manila voting https://review.opendev.org/c/openstack/openstacksdk/+/942624 | 21:04 |
| opendevreview | Merged openstack/openstacksdk master: zuul: Make openstacksdk-functional-devstack-masakari voting https://review.opendev.org/c/openstack/openstacksdk/+/942626 | 21:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!