Skip to content

Conversation

Diaphteiros
Copy link
Contributor

What this PR does / why we need it:
Adds an AccessRequest controller to the openmcp-operator. This controller reconciles only AccessRequests that don't have a label containing the provider name and adds this label to the resource. This simplifies the logic in the individual ClusterProviders and boosts their performance, since not every ClusterProvider has to check on its own whether it is responsible for the AccessRequest.

@Diaphteiros Diaphteiros marked this pull request as draft May 21, 2025 11:16
@Diaphteiros Diaphteiros marked this pull request as ready for review May 21, 2025 11:50
@Diaphteiros Diaphteiros marked this pull request as draft May 21, 2025 11:54
@Diaphteiros Diaphteiros marked this pull request as ready for review May 21, 2025 12:59
Copy link
Contributor

@reshnm reshnm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some minor nit.

@Diaphteiros Diaphteiros requested a review from reshnm May 22, 2025 10:40
@Diaphteiros Diaphteiros merged commit d99e51b into main May 22, 2025
4 checks passed
@Diaphteiros Diaphteiros deleted the accessrequest branch May 22, 2025 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants