| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-11-18 | |||
| 09:26:20 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: make sort_key and sort_dir in neutron array https://review.opendev.org/c/openstack/codegenerator/+/935489 | |
| 11:40:46 | opendevreview | Josephine Seifert proposed openstack/python-openstackclient master: WIP: Add admin-editable metadata field to volume types https://review.opendev.org/c/openstack/python-openstackclient/+/928132 | |
| 13:42:43 | opendevreview | Merged openstack/openstacksdk master: Remove reference to removed nova services https://review.opendev.org/c/openstack/openstacksdk/+/935477 | |
| 15:47:12 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.identity.generic https://review.opendev.org/c/openstack/keystoneauth/+/932988 | |
| 15:49:57 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.identity.v3 https://review.opendev.org/c/openstack/keystoneauth/+/932989 | |
| 15:49:58 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.extras (1/2) https://review.opendev.org/c/openstack/keystoneauth/+/932990 | |
| 15:49:59 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.base https://review.opendev.org/c/openstack/keystoneauth/+/932991 | |
| 15:50:01 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.opts https://review.opendev.org/c/openstack/keystoneauth/+/932992 | |
| 16:09:15 | opendevreview | Merged openstack/openstacksdk master: Fix missing 'f' prefix in exception message https://review.opendev.org/c/openstack/openstacksdk/+/933375 | |
| 16:54:48 | opendevreview | Merged openstack/codegenerator master: make sort_key and sort_dir in neutron array https://review.opendev.org/c/openstack/codegenerator/+/935489 | |
| 19:44:42 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Use Array for serialized param instead of Set https://review.opendev.org/c/openstack/codegenerator/+/935559 | |
| 20:54:21 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.cli https://review.opendev.org/c/openstack/keystoneauth/+/932993 | |
| 20:54:22 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.conf https://review.opendev.org/c/openstack/keystoneauth/+/932994 | |
| 20:54:24 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.adapter https://review.opendev.org/c/openstack/keystoneauth/+/932995 | |
| 20:54:25 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.session https://review.opendev.org/c/openstack/keystoneauth/+/932996 | |
| 20:54:27 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.identity https://review.opendev.org/c/openstack/keystoneauth/+/932997 | |
| 20:54:28 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading._utils https://review.opendev.org/c/openstack/keystoneauth/+/932998 | |
| 20:54:30 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading._plugins https://review.opendev.org/c/openstack/keystoneauth/+/932999 | |
| #openstack-sdks - 2024-11-19 | |||
| 08:52:46 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Use Array for serialized param instead of Set https://review.opendev.org/c/openstack/codegenerator/+/935559 | |
| 11:07:42 | opendevreview | Merged openstack/codegenerator master: Use Array for serialized param instead of Set https://review.opendev.org/c/openstack/codegenerator/+/935559 | |
| 12:50:43 | frickler | gtema: please have a look at these when you have a moment https://review.opendev.org/c/openstack/releases/+/934606 https://review.opendev.org/c/openstack/releases/+/934597 | |
| 12:51:11 | gtema | ack frickler, thanks for reminder | |
| 13:32:42 | opendevreview | Merged openstack/openstacksdk stable/2024.2: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932401 | |
| 13:32:44 | opendevreview | Merged openstack/openstacksdk stable/2024.1: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932402 | |
| 13:33:40 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.extras (2/2) https://review.opendev.org/c/openstack/keystoneauth/+/933000 | |
| 13:39:02 | opendevreview | Merged openstack/openstacksdk stable/2023.2: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932403 | |
| 17:05:15 | M0weng[m] | stephenfin: test ping | |
| 17:05:31 | stephenfin | M0weng[m]: Now you appear. Matrix weirdness | |
| 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/ | |