| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-11-19 | |||
| 17:42:23 | opendevreview | Merged openstack/keystoneauth master: typing: Invert module settings https://review.opendev.org/c/openstack/keystoneauth/+/933001 | |
| 17:42:24 | opendevreview | Merged openstack/keystoneauth master: typing: Set remaining 'disallow_*' opts to 'true' https://review.opendev.org/c/openstack/keystoneauth/+/933002 | |
| 17:57:10 | opendevreview | Merged openstack/keystoneauth master: typing: Move configuration to pyproject.toml https://review.opendev.org/c/openstack/keystoneauth/+/933003 | |
| 17:57:11 | opendevreview | Merged openstack/keystoneauth master: Add py.typed file https://review.opendev.org/c/openstack/keystoneauth/+/933004 | |
| 17:57:14 | opendevreview | Merged openstack/keystoneauth master: typing: Make loaders broadly generic https://review.opendev.org/c/openstack/keystoneauth/+/933005 | |
| 18:21:43 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'trust' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/931694 | |
| #openstack-sdks - 2024-11-20 | |||
| 11:34:21 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Simplify some types, other TODOs https://review.opendev.org/c/openstack/keystoneauth/+/935764 | |
| 11:34:22 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused AuthMethod.get_auth_data arguments https://review.opendev.org/c/openstack/keystoneauth/+/935765 | |
| 11:34:23 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused BaseAuthPlugin.get_auth_ref kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935766 | |
| 11:34:24 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused _Rescoped.get_unscoped_auth_ref kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935767 | |
| 11:34:26 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused BaseAuthPlugin.get_headers kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935768 | |
| 11:34:28 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused BaseAuthPlugin.get_token kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935769 | |
| 11:34:30 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: WIP: typing: Deprecate use of AuthConstructor https://review.opendev.org/c/openstack/keystoneauth/+/935770 | |
| 11:34:32 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: WIP: typing: Removing remaining used kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935771 | |
| 11:38:31 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: loading.opts: Add 'argparse_envvars' helper https://review.opendev.org/c/openstack/keystoneauth/+/933110 | |
| 11:38:32 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: doc: Fix indentation https://review.opendev.org/c/openstack/keystoneauth/+/933111 | |
| 11:38:33 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: docs: Rework plugin-options doc https://review.opendev.org/c/openstack/keystoneauth/+/933112 | |
| 11:38:34 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: loading: Add blurb for most plugins https://review.opendev.org/c/openstack/keystoneauth/+/933113 | |
| 11:38:36 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: docs: Document CLI options, envvars https://review.opendev.org/c/openstack/keystoneauth/+/933114 | |
| 12:18:18 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Fix misuse of assertTrue https://review.opendev.org/c/openstack/keystoneauth/+/935777 | |
| 14:29:39 | tobias-urdin | stephenfin: can u have a look at https://review.opendev.org/c/openstack/python-openstackclient/+/932618 which for cinder v2 (my fail) is already approved | |
| 14:36:30 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 15:11:05 | stephenfin | tobias-urdin: +W | |
| 16:38:58 | opendevreview | Merged openstack/python-openstackclient master: Show Created At column for volume backups in v3 https://review.opendev.org/c/openstack/python-openstackclient/+/932618 | |
| 16:40:22 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 19:51:50 | adamcarthur5 | Hey stephenfin, feel free to close https://review.opendev.org/c/openstack/ironic/+/935697/1, Ironic is pretty good for quick review-turnover so I've just merged this change directly into the prev commit | |
| 19:59:39 | tobias-urdin | stephenfin: thx! | |
| 20:14:14 | opendevreview | Eric Harney proposed openstack/python-openstackclient master: WIP: Fix volume snapshot create on cinder mv >= 3.66 https://review.opendev.org/c/openstack/python-openstackclient/+/935829 | |
| 22:58:00 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'endpoint' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/934817 | |
| 22:59:16 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'endpoint' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/934817 | |
| 23:20:00 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'domain' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923785 | |
| 23:26:39 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate `region` commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/930937 | |
| 23:28:55 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'domain' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923785 | |
| 23:38:45 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service provider' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929961 | |
| #openstack-sdks - 2024-11-21 | |||
| 01:12:12 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 12:55:42 | stephenfin | gtema: Can you sanity check something for me? Why is this input https://paste.openstack.org/show/bY1G6R4A0pO0eZzFzmfB/ invalid against this schema? https://paste.openstack.org/show/bBWN4ljq3WoE7gTkSuS2/ | |
| 12:56:15 | gtema | gimme a sec to look | |
| 13:01:18 | stephenfin | gtema: got it: turns out additionalProperties and oneOf do not play nicely together. That's a fairly nasty gotcha https://github.com/python-jsonschema/jsonschema/issues/193 | |
| 13:01:36 | gtema | yeah, I slowly came to the same conclusion | |
| 13:02:17 | gtema | if you put whole impls into the oneOf kinds and drop everything else on the base it goes fine | |
| 13:02:42 | stephenfin | yup, that's what I've ended up doing. Bit of duplication but at least it works | |
| 15:17:02 | frickler | gtema: stephenfin: I've seen two gate failures today in reqs sdk job, both times the two tests after openstack.tests.functional.compute.v2.test_service.TestService.test_update were failing, even though the latter is now skipped, so the skip doesn't seem to have helped much :( | |
| 15:17:29 | gtema | :( | |
| 15:17:34 | frickler | maybe some resource issue still, not sure if one of you can look into it or if I should rather try to check with nova folks | |
| 15:17:48 | frickler | https://zuul.opendev.org/t/openstack/build/2862593e22bd41718da98e504fe67c6d is one of them | |
| 15:18:33 | gtema | today I am completely booked for other stuff | |
| 17:51:37 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Allow passing None for username in v2.Password (redux) https://review.opendev.org/c/openstack/keystoneauth/+/935897 | |
| 18:34:54 | opendevreview | OpenStack Release Bot proposed openstack/keystoneauth master: reno: Update master for unmaintained/2023.1 https://review.opendev.org/c/openstack/keystoneauth/+/935961 | |
| #openstack-sdks - 2024-11-22 | |||
| 11:56:59 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Skip disabling compute service https://review.opendev.org/c/openstack/openstacksdk/+/936016 | |
| 11:58:51 | frickler | gtema: ^^ seems there is another slightly destructive test ... :-( | |
| 12:13:36 | gtema | Damn | |
| 12:20:07 | frickler | well I'll be glad if it really turns out to be this and not some weird, deep nova issue | |
| 16:20:50 | stephenfin | M0weng[m]: Sorry, I forgot about the limits thing | |
| 16:21:27 | M0weng[m] | No worries :D I'll let you know if I figure it out with the local keystone setup | |
| 16:21:44 | stephenfin | wdym but "local" setup? | |
| 16:21:55 | stephenfin | As in, running on your localhost rather than in DevStack? | |
| 16:22:34 | stephenfin | If so, you can do that but I never have. Instead, I tend to make changes on the devstack VM and see what happens there | |
| 16:23:48 | stephenfin | DevStack installs all projects into '/opt/stack' by default. You'll find e.g. the keystone repo there and can modify things as you see fit. When you've made a modification, you need to restart the relevant service with systemctl. | |
| 16:23:57 | M0weng[m] | Ohh I think I misunderstood... where in devstack would I make those changes? I tried modifying the code in /opt/stack and it didn't make any difference | |
| 16:24:05 | M0weng[m] | Ohhh I forgot to restart it haha | |
| 16:24:09 | M0weng[m] | That would make sense | |
| 16:24:25 | M0weng[m] | Ok cool thank you, I'll try that :D | |
| 16:24:35 | stephenfin | Ah, that'd do it :) For Keystone you want 'systemctl restart devstack@keystone.service' | |
| 16:25:38 | stephenfin | Also, everything uses mysql. You hopefully set a password in your local.conf. If so, you can run the 'mysql' CLI with e.g. 'mysql -ppassword' | |
| 16:26:34 | stephenfin | M0weng[m]: When I do that, I see the following for the 'limit' table: https://paste.openstack.org/show/b0i8rWQDsNJhS2K7nd8X/ | |
| 16:26:44 | stephenfin | So I'm not really sure what's going on without further investigation | |
| 16:26:47 | opendevreview | Merged openstack/openstacksdk master: Skip disabling compute service https://review.opendev.org/c/openstack/openstacksdk/+/936016 | |
| 16:31:33 | M0weng[m] | stephenfin: Yeah I'm getting the same result 😔 | |
| 16:40:16 | stephenfin | Hmm, I think there might be a bug in keystone. When I apply the following patch https://paste.openstack.org/show/bIis4CjDiJZJv3McQIIm/ | |
| 16:41:01 | stephenfin | I see the following in the logs https://paste.openstack.org/show/badFhL59v9AZ37S4iSkN/ | |
| 16:43:08 | stephenfin | So it's attempting to filter on two project IDs at the same time. One of those is the one I passed via '--project-id' but idk where the other one is coming from | |
| 16:43:28 | stephenfin | Adding more debug logging gives me the following SQL query which proves that https://paste.openstack.org/show/bWmfbv6KcjTfnp5VKaGN/ | |
| 16:52:24 | opendevreview | Merged openstack/keystoneauth master: Allow passing None for username in v2.Password (redux) https://review.opendev.org/c/openstack/keystoneauth/+/935897 | |
| 17:08:16 | frickler | stephenfin: is ^^ enough to warrant another release or do you have more fixes in the pipeline? | |
| 17:12:50 | stephenfin | nope, let's cut a release | |
| 17:13:06 | stephenfin | there are a few other patches up there but nothing is urgent | |
| 17:59:58 | stephenfin | M0weng[m]: Looks like it's expected behavior https://github.com/openstack/keystone/blob/3e056ba8bd62dd16f941b05dd238a2a380642de7/keystone/tests/unit/test_limits.py#L1241-L1248 | |
| 18:02:16 | stephenfin | M0weng[m]: So if you want to list against a different role, you need to use a system-scoped token. You can get one of those by using 'devstack-system-admin' instead of 'devstack-admin' | |
| 18:03:05 | stephenfin | We should probably add a check that the token is system-scoped or the project_id field matches that in the token in OSC, given the absence of one in keystone | |
| 18:03:10 | stephenfin | https://bugs.launchpad.net/keystone/+bug/2089403 | |
| 18:03:11 | M0weng[m] | Ohhhh ok, that makes sense! | |
| 18:03:27 | stephenfin | very poor API design IMO but now we know | |
| 18:03:37 | M0weng[m] | Thank you for the help! :D | |
| #openstack-sdks - 2024-11-24 | |||
| 15:42:05 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Remove ceilometer service overrides https://review.opendev.org/c/openstack/python-openstackclient/+/936105 | |
| #openstack-sdks - 2024-11-26 | |||
| 12:06:29 | opendevreview | Merged openstack/python-openstackclient master: Remove ceilometer service overrides https://review.opendev.org/c/openstack/python-openstackclient/+/936105 | |
| 15:57:01 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add octavia query parameters https://review.opendev.org/c/openstack/codegenerator/+/936274 | |
| 16:38:04 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate limit commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/936279 | |
| 16:46:22 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Temporary disable keystone code generation https://review.opendev.org/c/openstack/codegenerator/+/936280 | |
| 16:49:35 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Re-enable building of nova openapi specs as a voting job https://review.opendev.org/c/openstack/codegenerator/+/936281 | |
| 17:04:08 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 17:20:29 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Temporary replace appcreds schema https://review.opendev.org/c/openstack/codegenerator/+/936280 | |
| 19:22:20 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Temporary replace appcreds schema https://review.opendev.org/c/openstack/codegenerator/+/936280 | |
| 19:32:25 | opendevreview | Merged openstack/openstacksdk master: identity: Add tags, options to Domain https://review.opendev.org/c/openstack/openstacksdk/+/934991 | |
| 19:32:27 | opendevreview | Merged openstack/openstacksdk master: Timing tests should use a threshold. https://review.opendev.org/c/openstack/openstacksdk/+/934519 | |
| 19:32:29 | opendevreview | Merged openstack/openstacksdk master: Add port bindings to SDK https://review.opendev.org/c/openstack/openstacksdk/+/915902 | |
| 19:32:31 | opendevreview | Merged openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 19:32:33 | opendevreview | Merged openstack/openstacksdk master: identity: Add functional tests for projects https://review.opendev.org/c/openstack/openstacksdk/+/934226 | |
| 19:32:35 | opendevreview | Merged openstack/openstacksdk master: compute: Add additional migration parameters https://review.opendev.org/c/openstack/openstacksdk/+/934228 | |
| 19:32:36 | opendevreview | Merged openstack/openstacksdk master: block storage: Add user_id query param for Volume https://review.opendev.org/c/openstack/openstacksdk/+/934229 | |
| 19:32:38 | opendevreview | Merged openstack/openstacksdk master: doc: Add multiple examples of authentication configuration https://review.opendev.org/c/openstack/openstacksdk/+/934230 | |