| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2026-08-20 | |||
| 13:37:30 | opendevreview | Merged openstack/keystoneauth master: pre-commit: Bump versions https://review.opendev.org/c/openstack/keystoneauth/+/1001538 | |
| 14:59:57 | opendevreview | Merged openstack/openstacksdk master: Identity: Add support for endpoint groups https://review.opendev.org/c/openstack/openstacksdk/+/995762 | |
| 15:27:53 | opendevreview | Thomas Sell proposed openstack/ansible-collections-openstack master: Fix subnet module when subnet already exists https://review.opendev.org/c/openstack/ansible-collections-openstack/+/1001685 | |
| 15:30:26 | opendevreview | Thomas Sell proposed openstack/ansible-collections-openstack master: Fix subnet module when subnet already exists https://review.opendev.org/c/openstack/ansible-collections-openstack/+/1001685 | |
| 15:39:15 | opendevreview | Merged openstack/openstacksdk master: Identity: Add support for endpoint group/endpoint and endpoint group/project associations https://review.opendev.org/c/openstack/openstacksdk/+/999040 | |
| 16:27:16 | stephenfin | cardoe: jfyi stevedore 5.9.1 is out and should be in u-c soon https://review.opendev.org/c/openstack/releases/+/1001666 | |
| 16:27:24 | stephenfin | https://review.opendev.org/c/openstack/releases/+/1001666 | |
| 16:27:46 | stephenfin | that adds docs for and fixes the conflict resolution stuff, so you should be able to rely on it in KSA | |
| 16:27:46 | cardoe | stephenfin: you got any examples of how to use or drive it? | |
| 16:28:13 | stephenfin | https://docs.openstack.org/stevedore/latest/user/conflict_resolution.html | |
| 16:28:18 | cardoe | perfect | |
| 16:28:54 | cardoe | stephenfin: as an aside... can you merge keystonemiddleware too? | |
| 16:29:11 | stephenfin | this can probably be another standalone precursor patch. easier to merge that way | |
| 16:29:16 | stephenfin | cardoe: afraid not :( | |
| 16:29:31 | stephenfin | gtema can though | |
| 16:30:05 | cardoe | well there's a lot of cleanups that tkajinam and I have pitched out and gtema says its a lack of reviewers | |
| 16:30:18 | cardoe | a lot of its cleanups to pyproject.toml and such | |
| 16:30:21 | cardoe | but that's an aside | |
| 16:30:49 | opendevreview | Vladislav Terekhov proposed openstack/openstacksdk master: compute: Fix server create attribute https://review.opendev.org/c/openstack/openstacksdk/+/1001700 | |
| 16:31:16 | cardoe | I'll get https://review.opendev.org/c/openstack/keystoneauth/+/1001419 cleaned up today and I saw it's parent you added David and gtema... I assume getting their +2 is the blocker. | |
| 16:31:46 | stephenfin | I can review middleware changes, but I can't commit to reviewing keystone itself and I think the two go hand-in-hand currently | |
| 16:32:17 | stephenfin | and unlike ksa it doesn't have a natural home in SDK | |
| 16:33:42 | stephenfin | cardoe: Not a blocker, no. I'd just like to give them some chance to weigh in | |
| 16:34:07 | cardoe | okay sounds good. | |
| 16:34:36 | cardoe | My plan is to follow up on the WebSSO protocol and formalize that a bit better with a spec. I plan on fixing the CSRF issue by allowing a nonce in the protocol. | |
| 16:35:11 | opendevreview | Vladislav Terekhov proposed openstack/openstacksdk master: compute: Fix server create attribute https://review.opendev.org/c/openstack/openstacksdk/+/1001700 | |
| 16:38:12 | stephenfin | makes sense 👌 | |
| 16:38:20 | cardoe | Another aside... "openstack subnet pool set" and "openstack subnet pool unset" aren't complementary. You have to delete a subnet pool to change most of the settings. I've got a patch series coming to fix that up. | |
| 16:39:15 | cardoe | Just lemme know how I can help. I'd like to see gophercloud and openstacksdk be pretty feature parity and us to clean up the typing and api stuff. | |
| 16:39:35 | cardoe | One area I'd appreciate you input on is going to be some of the scope changed I'd like to do. | |
| 16:40:06 | cardoe | So maybe sometime when you have 30 minutes we can setup a meetpad and talk? | |
| 16:40:39 | cardoe | Cause I'd like to propose some specs and I'm not sure of the right places for that either. | |
| 16:43:02 | stephenfin | Sure. This week is almost over and I've got some Mythos findings to finish working through, but I could do something towards the end of next week | |
| 16:43:30 | stephenfin | though I will say async might be better if we starting getting too into the weeds, since it affords time to actually go and research things | |
| 16:43:36 | stephenfin | s/if/once/ | |
| 16:44:30 | stephenfin | I had never really considered the fact that unscoped tokens are a thing and that the shape of clouds.yaml entires isn't necessarily as good as it should be | |
| 16:44:46 | stephenfin | ...before starting on reviews for this websso work | |
| 16:53:18 | cardoe | yeah so that was actually one of the struggles in the original scope adoption failure review. | |
| 16:53:40 | cardoe | That people had to flip flop between three OS_CLOUD entries to be able to run commands. | |
| 16:57:22 | cardoe | I'm happy to post something async but it's a bit more than can fit into a few lines of IRC | |
| 16:57:35 | cardoe | How would you like to get that? email? etherpad? | |
| 16:57:56 | stephenfin | etherpad? At least to start | |
| 16:58:53 | cardoe | Will do. | |
| 17:06:59 | cardoe | I'll share that tomorrow or so. | |
| 17:20:22 | opendevreview | Merged openstack/openstacksdk master: identity: Add federated attribute support to User https://review.opendev.org/c/openstack/openstacksdk/+/1001638 | |
| 17:37:45 | cardoe | rm_work: so your identities are tied back to the same identity_provider and protocol values in clouds.yaml? | |
| 17:38:26 | cardoe | rm_work: cause I've got other identities too but they have different values for one of those. | |
| 18:19:27 | opendevreview | Doug Goldstein proposed openstack/keystoneauth master: Expose the unscoped token of a federated plugin https://review.opendev.org/c/openstack/keystoneauth/+/1001419 | |
| 18:44:36 | cardoe | rm_work: I have a solution. | |
| 18:53:41 | opendevreview | Doug Goldstein proposed openstack/keystoneauth master: Expose the unscoped token of a federated plugin https://review.opendev.org/c/openstack/keystoneauth/+/1001419 | |
| 18:54:28 | cardoe | rm_work: ^ updates the cache_id elements to behave like the Password plugin which includes username/password in the hash. I'm not necessarily a fan of password being in there. But it does invalidate the cache when the password changes. | |
| 18:55:16 | cardoe | Now when I rebase the v3websso plugin, I'll include username in the elements... which while not used at all by the plugin... will allow you to have two different cache entries. | |
| 19:19:38 | opendevreview | Doug Goldstein proposed openstack/keystoneauth master: Add a v3websso authentication plugin https://review.opendev.org/c/openstack/keystoneauth/+/1000759 | |
| 19:35:25 | opendevreview | Doug Goldstein proposed openstack/keystoneauth master: Add a v3websso authentication plugin https://review.opendev.org/c/openstack/keystoneauth/+/1000759 | |
| 19:36:29 | cardoe | stephenfin, rm_work: assuming you give me the grace of doing the conflict resolution changes with stevedore in a separate patch ^ should address all the feedback now. | |
| 23:40:42 | rm_work | oh yeah I guess that works since it's just unused in the actual auth process with websso 😛 | |
| #openstack-sdks - 2026-08-21 | |||
| 09:59:07 | opendevreview | Merged openstack/openstacksdk master: compute: Fix server create attribute https://review.opendev.org/c/openstack/openstacksdk/+/1001700 | |
| 11:55:26 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Migrate schemas to JSON Schema Draft 2020-12 https://review.opendev.org/c/openstack/openstacksdk/+/1001785 | |
| 11:55:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Drop defaults https://review.opendev.org/c/openstack/openstacksdk/+/1001787 | |
| 11:55:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Initial rework of schema https://review.opendev.org/c/openstack/openstacksdk/+/1001786 | |
| 11:55:28 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Add schema for clouds.yaml files https://review.opendev.org/c/openstack/openstacksdk/+/1001789 | |
| 11:55:28 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Expand schema.json https://review.opendev.org/c/openstack/openstacksdk/+/1001788 | |
| 12:55:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Address various typing issues https://review.opendev.org/c/openstack/openstacksdk/+/1001796 | |
| 12:55:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: load_balancer: Change cascade delete https://review.opendev.org/c/openstack/openstacksdk/+/1001795 | |
| 12:55:28 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Don't assign to methods https://review.opendev.org/c/openstack/openstacksdk/+/1001798 | |
| 12:55:28 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Improve network proxy tests https://review.opendev.org/c/openstack/openstacksdk/+/1001797 | |
| 12:55:29 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Add some missing asserts, type hints https://review.opendev.org/c/openstack/openstacksdk/+/1001800 | |
| 12:55:29 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Use a module-level base class in resource action tests https://review.opendev.org/c/openstack/openstacksdk/+/1001799 | |
| 12:55:30 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable checks on unit tests https://review.opendev.org/c/openstack/openstacksdk/+/1001801 | |
| 12:55:31 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (2/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001803 | |
| 12:55:31 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (1/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001802 | |
| 12:55:33 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (4/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001805 | |
| 12:55:33 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (3/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001804 | |
| 12:55:35 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (6/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001807 | |
| 12:55:35 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (5/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001806 | |
| 12:58:04 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Add schema for clouds.yaml files https://review.opendev.org/c/openstack/openstacksdk/+/1001789 | |
| 14:23:38 | cardoe | stephenfin: schema for clouds.yaml!? | |
| 14:24:01 | stephenfin | A first pass at one anyway | |
| 14:24:09 | cardoe | I think it's great. | |
| 14:24:11 | stephenfin | I want it for gophercloud | |
| 14:24:42 | cardoe | I've been posting updates to docs around the 'auth' block. A lot of places have examples with fields at the top level and not in the auth block. | |
| 14:25:08 | cardoe | Some of those are patches hanging out against keystonemiddleware. | |
| 14:25:55 | cardoe | The only outstanding item on the websso work is your comment on https://review.opendev.org/c/openstack/keystoneauth/+/1001419 if you wanted me to make that change or not since you said it was an idea. | |
| 14:27:21 | stephenfin | No, no change needed. I haven't thought about that enough to have an actionable suggestions and it's private so we can change it later | |
| 14:28:33 | stephenfin | where I was going with that was potentially rejigging ksa so we *always* fetch unscoped tokens first, then rescope as required. If we did that, we'd need a way to cache the different scopes | |
| 14:29:26 | stephenfin | ...but that's putting the cart before the horse. Lots of other, much more complicated things to figure out first (including an actual design/plan) | |
| 14:32:13 | cardoe | yeah I think it's a good idea | |
| 14:32:19 | cardoe | but yeah need to design it. | |
| 22:06:34 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Fix EndpointGroupProject resource base path https://review.opendev.org/c/openstack/openstacksdk/+/1001902 | |
| 22:24:03 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Fix EndpointGroupProject resource base path https://review.opendev.org/c/openstack/openstacksdk/+/1001902 | |
| 22:25:18 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Fix EndpointGroupProject resource base path https://review.opendev.org/c/openstack/openstacksdk/+/1001902 | |
| #openstack-sdks - 2026-08-22 | |||
| 02:11:08 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Fix EndpointGroupProject resource base path https://review.opendev.org/c/openstack/openstacksdk/+/1001902 | |
| 23:02:13 | opendevreview | Merged openstack/openstacksdk master: load_balancer: Change cascade delete https://review.opendev.org/c/openstack/openstacksdk/+/1001795 | |
| 23:08:45 | opendevreview | Merged openstack/openstacksdk master: typing: Address various typing issues https://review.opendev.org/c/openstack/openstacksdk/+/1001796 | |
| 23:31:26 | opendevreview | Merged openstack/openstacksdk master: tests: Improve network proxy tests https://review.opendev.org/c/openstack/openstacksdk/+/1001797 | |
| 23:38:07 | opendevreview | Merged openstack/openstacksdk master: tests: Don't assign to methods https://review.opendev.org/c/openstack/openstacksdk/+/1001798 | |
| 23:38:10 | opendevreview | Merged openstack/openstacksdk master: tests: Use a module-level base class in resource action tests https://review.opendev.org/c/openstack/openstacksdk/+/1001799 | |
| 23:38:12 | opendevreview | Merged openstack/openstacksdk master: tests: Add some missing asserts, type hints https://review.opendev.org/c/openstack/openstacksdk/+/1001800 | |
| 23:38:14 | opendevreview | Merged openstack/openstacksdk master: mypy: Enable checks on unit tests https://review.opendev.org/c/openstack/openstacksdk/+/1001801 | |
| #openstack-sdks - 2026-08-23 | |||
| 10:18:20 | opendevreview | Yash proposed openstack/openstacksdk master: Deprecate get_server_meta() method https://review.opendev.org/c/openstack/openstacksdk/+/997626 | |