| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-11-20 | |||
| 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 | |
| 22:34:12 | opendevreview | Merged openstack/codegenerator master: Temporary replace appcreds schema https://review.opendev.org/c/openstack/codegenerator/+/936280 | |
| #openstack-sdks - 2024-11-27 | |||
| 08:39:52 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add octavia query parameters https://review.opendev.org/c/openstack/codegenerator/+/936274 | |
| 10:28:50 | opendevreview | Merged openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 12:20:56 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Update authentication configuration examples https://review.opendev.org/c/openstack/openstacksdk/+/936350 | |
| 15:19:37 | opendevreview | Merged openstack/codegenerator master: Add octavia query parameters https://review.opendev.org/c/openstack/codegenerator/+/936274 | |
| #openstack-sdks - 2024-11-28 | |||
| 09:39:28 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add loadbalancer_id QP to octavia pools https://review.opendev.org/c/openstack/codegenerator/+/936432 | |
| 12:26:37 | opendevreview | Merged openstack/codegenerator master: Add loadbalancer_id QP to octavia pools https://review.opendev.org/c/openstack/codegenerator/+/936432 | |
| 13:40:57 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Parse schema for validation https://review.opendev.org/c/openstack/codegenerator/+/936458 | |
| 13:56:13 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Only compile generated rust code https://review.opendev.org/c/openstack/codegenerator/+/936460 | |
| 14:18:31 | opendevreview | JaeSeong-Shin proposed openstack/openstacksdk master: Adding nameserver get for a zone from the DNS SDK https://review.opendev.org/c/openstack/openstacksdk/+/926133 | |
| 15:03:30 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Parse schema for validation https://review.opendev.org/c/openstack/codegenerator/+/936458 | |
| 15:10:06 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Use single quotes for single char https://review.opendev.org/c/openstack/codegenerator/+/936629 | |
| 16:38:48 | opendevreview | Merged openstack/codegenerator master: Only compile generated rust code https://review.opendev.org/c/openstack/codegenerator/+/936460 | |
| 17:04:46 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Parse schema for validation https://review.opendev.org/c/openstack/codegenerator/+/936458 | |
| 17:20:07 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Release 2.3.0 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/936648 | |
| 18:44:38 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Parse schema for validation https://review.opendev.org/c/openstack/codegenerator/+/936458 | |
| 18:49:53 | opendevreview | Merged openstack/codegenerator master: Use single quotes for single char https://review.opendev.org/c/openstack/codegenerator/+/936629 | |
| #openstack-sdks - 2024-11-29 | |||
| 05:05:44 | opendevreview | JaeSeong-Shin proposed openstack/openstacksdk master: Adding nameserver get for a zone from the DNS SDK https://review.opendev.org/c/openstack/openstacksdk/+/926133 | |
| 07:08:26 | opendevreview | JaeSeong-Shin proposed openstack/openstacksdk master: Adding nameserver get for a zone from the DNS SDK https://review.opendev.org/c/openstack/openstacksdk/+/926133 | |
| 09:50:27 | opendevreview | Merged openstack/ansible-collections-openstack master: Release 2.3.0 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/936648 | |
| 10:33:59 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Add server create --no-security-group option https://review.opendev.org/c/openstack/python-openstackclient/+/936715 | |
| 10:34:00 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: network: Make better use of argparse https://review.opendev.org/c/openstack/python-openstackclient/+/936716 | |
| 11:31:16 | opendevreview | Merged openstack/codegenerator master: Parse schema for validation https://review.opendev.org/c/openstack/codegenerator/+/936458 | |
| 17:44:01 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add pagination QP to octavia resources https://review.opendev.org/c/openstack/codegenerator/+/936763 | |
| 20:04:16 | opendevreview | Merged openstack/codegenerator master: Add pagination QP to octavia resources https://review.opendev.org/c/openstack/codegenerator/+/936763 | |
| #openstack-sdks - 2024-12-02 | |||
| 14:34:18 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Skip lifetime in rust sdk https://review.opendev.org/c/openstack/codegenerator/+/936836 | |
| 18:00:25 | opendevreview | Merged openstack/codegenerator master: Skip lifetime in rust sdk https://review.opendev.org/c/openstack/codegenerator/+/936836 | |