Skip to content

Fix up ownership resolution for cluster-scoped resources#924

Closed
andrewstucki wants to merge 2 commits intomainfrom
as/cluster-scoped-ownership-changes
Closed

Fix up ownership resolution for cluster-scoped resources#924
andrewstucki wants to merge 2 commits intomainfrom
as/cluster-scoped-ownership-changes

Conversation

@andrewstucki
Copy link
Contributor

Need to add tests still, but this is to fix an oddity (that has no real side-effects yet), where ClusterRole or some other cluster-scoped resource was triggering a reconciliation on a Redpanda object using a keda helmrelease in prod due to Azure leveraging flux internally for their cluster setups.

@github-actions
Copy link

This PR is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@chrisseto
Copy link
Contributor

@andrewstucki looks like this is good to merge?

@github-actions
Copy link

github-actions bot commented Jul 2, 2025

This PR is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the stale label Jul 2, 2025
@github-actions
Copy link

github-actions bot commented Jul 7, 2025

This PR was closed because it has been stalled for 5 days with no activity.

@github-actions github-actions bot closed this Jul 7, 2025
@chrisseto chrisseto removed the stale label Jul 8, 2025
@chrisseto
Copy link
Contributor

@andrewstucki can this be revived?

@chrisseto chrisseto reopened this Jul 8, 2025
@github-actions
Copy link

This PR is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the stale label Jul 14, 2025
@github-actions
Copy link

This PR was closed because it has been stalled for 5 days with no activity.

@github-actions github-actions bot closed this Jul 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants