| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2026-08-20 | |||
| 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: Initial rework of schema https://review.opendev.org/c/openstack/openstacksdk/+/1001786 | |
| 11:55:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Drop defaults https://review.opendev.org/c/openstack/openstacksdk/+/1001787 | |
| 11:55:28 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: config: Expand schema.json https://review.opendev.org/c/openstack/openstacksdk/+/1001788 | |
| 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 | |
| 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:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Address various typing issues https://review.opendev.org/c/openstack/openstacksdk/+/1001796 | |
| 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:28 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Don't assign to methods https://review.opendev.org/c/openstack/openstacksdk/+/1001798 | |
| 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: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: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 (1/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001802 | |
| 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: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: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: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: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: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 | |
| 10:19:54 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (1/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001802 | |
| 10:19:55 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (2/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001803 | |
| 10:19:55 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (3/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001804 | |
| 10:19:56 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (4/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001805 | |
| 10:19:56 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (5/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001806 | |
| 10:19:57 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Fix type issue with service tests (6/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001807 | |
| 13:16:44 | opendevreview | Merged openstack/openstacksdk master: tests: Fix type issue with service tests (1/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001802 | |
| 13:21:32 | opendevreview | Merged openstack/openstacksdk master: tests: Fix type issue with service tests (2/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001803 | |
| 13:33:19 | opendevreview | Merged openstack/openstacksdk master: Deprecate get_server_meta() method https://review.opendev.org/c/openstack/openstacksdk/+/997626 | |
| 19:03:15 | opendevreview | Merged openstack/openstacksdk master: tests: Fix type issue with service tests (3/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001804 | |
| #openstack-sdks - 2026-08-24 | |||
| 00:31:29 | opendevreview | Merged openstack/openstacksdk master: tests: Fix type issue with service tests (4/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001805 | |
| 00:46:35 | opendevreview | Merged openstack/openstacksdk master: tests: Fix type issue with service tests (5/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001806 | |
| 00:46:37 | opendevreview | Merged openstack/openstacksdk master: tests: Fix type issue with service tests (6/6) https://review.opendev.org/c/openstack/openstacksdk/+/1001807 | |
| 08:07:40 | opendevreview | Vladislav Terekhov proposed openstack/openstacksdk master: resource: Avoid consuming attributes twice https://review.opendev.org/c/openstack/openstacksdk/+/1002021 | |
| 12:42:40 | opendevreview | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: reduce likelyhood of races around allocations https://review.opendev.org/c/openstack/openstacksdk/+/1002079 | |
| 13:04:04 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Disable import-untyped, var-annotated rule for tests https://review.opendev.org/c/openstack/openstacksdk/+/1002088 | |
| 13:04:05 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Add hints to cloud._utils, exc https://review.opendev.org/c/openstack/openstacksdk/+/1002089 | |
| 13:04:05 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Add hints to cloud.cmd, cloud.inventory https://review.opendev.org/c/openstack/openstacksdk/+/1002090 | |
| 13:04:06 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Add hints to cloud.meta https://review.opendev.org/c/openstack/openstacksdk/+/1002091 | |
| 13:04:06 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Add hints to cloud.openstackcloud https://review.opendev.org/c/openstack/openstacksdk/+/1002092 | |