Skip to content

Keystone: Implement RegisteredLimit Controller#782

Open
gndrmnn wants to merge 10 commits into
k-orc:mainfrom
gndrmnn:implement_registeredlimit
Open

Keystone: Implement RegisteredLimit Controller#782
gndrmnn wants to merge 10 commits into
k-orc:mainfrom
gndrmnn:implement_registeredlimit

Conversation

@gndrmnn

@gndrmnn gndrmnn commented May 11, 2026

Copy link
Copy Markdown
Contributor

WIP

  • No support for RegionID as k-orc does not have that resource currently
  • RegisteredLimits are unnamed, but have a unique constraint over ResourceName, RegionID, and DefaultValue leading to some interesting challengens. E.g. Declaring two yaml files with two RegisteredLimits with the same RegionID and ResourceName will race to change the same entry.

@github-actions

Copy link
Copy Markdown

Failed to assess the semver bump. See logs for details.

@gndrmnn gndrmnn force-pushed the implement_registeredlimit branch from 22a08f7 to e46ac56 Compare May 12, 2026 14:31
@github-actions github-actions Bot added the semver:major Breaking change label May 12, 2026
@gndrmnn gndrmnn force-pushed the implement_registeredlimit branch from e46ac56 to 4964e1d Compare May 13, 2026 13:40
@gndrmnn gndrmnn force-pushed the implement_registeredlimit branch 4 times, most recently from 8379469 to 027f4aa Compare June 1, 2026 14:23
@gndrmnn gndrmnn force-pushed the implement_registeredlimit branch 4 times, most recently from b6de206 to e46d4a0 Compare June 29, 2026 14:10
@gndrmnn gndrmnn force-pushed the implement_registeredlimit branch from e46d4a0 to d91dcf9 Compare July 13, 2026 09:25
gndrmnn added 6 commits July 13, 2026 11:43
$ go run ./cmd/scaffold-controller -interactive=false \
    -kind=RegisteredLimit \
    -gophercloud-client=NewIdentityV3 \
    -gophercloud-module=github.com/gophercloud/gophercloud/v2/openstack/identity/v3/registeredlimits \
    -gophercloud-type=RegisteredLimit \
    -openstack-json-object=registered_limits \
    -required-create-dependency=Service

On-behalf-of: SAP nils.gondermann@sap.com
Register with the resource generator

On-behalf-of: SAP nils.gondermann@sap.com
Add the OpenStack client to scope

On-behalf-of: SAP nils.gondermann@sap.com
Register the controller

On-behalf-of: SAP nils.gondermann@sap.com
On-behalf-of: SAP nils.gondermann@sap.com
On-behalf-of: SAP nils.gondermann@sap.com
@gndrmnn gndrmnn force-pushed the implement_registeredlimit branch 9 times, most recently from 767c683 to 4c2f1fe Compare July 14, 2026 09:55
gndrmnn added 2 commits July 14, 2026 14:01
On-behalf-of: SAP nils.gondermann@sap.com
On-behalf-of: SAP nils.gondermann@sap.com
gndrmnn added 2 commits July 14, 2026 14:01
On-behalf-of: SAP nils.gondermann@sap.com
Generate the OLM bundle

On-behalf-of: SAP nils.gondermann@sap.com
@gndrmnn gndrmnn force-pushed the implement_registeredlimit branch from 4c2f1fe to 8c90198 Compare July 14, 2026 12:02
@gndrmnn

gndrmnn commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

@mandre

Hi, this PR is more or less good to be reviewed. I am currently running into a strange problem though: In the registeredlimit-import test we are creating a mandatory Service to use in the import filter. In some cases, the CI pipeline will fail with Waiting for service/<service name> to be ready. Other times, it successfully proceeds to Waiting for OpenStack resource to be created externally. As you can see right now, two CI tests failed while the epoxy one passed.

I have already setup a local keystone instance to test this against, and I can not reproduce the Waiting for service/<service name> to be ready problem. So it looks like this is some kind of timing problem in the test maybe?

What makes this especially weird is, I have looked at the Endpoint actuator and used it as inspiration for this PR. The Endpoint also requires a mandatory ServiceRef and even declares it the same way as I do in the endpoint-import CI test. That test however does not flap at all. So I seem to be doing something wrong, I can not put my finger on right now.

@gndrmnn gndrmnn marked this pull request as ready for review July 15, 2026 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:major Breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant