Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop use of owner references on job resources due to race condition #186

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

mkjpryor
Copy link
Collaborator

@mkjpryor mkjpryor commented Oct 16, 2024

Since Azimuth switched to foreground deletion, the cascading deletion on CaaS clusters can result in resources required for the remove job being deleted between the operator checking if they are available and creating the remove job.

@mkjpryor mkjpryor changed the title Drop use of owner references due to race condition Drop use of owner references on job resourcesdue to race condition Oct 16, 2024
@mkjpryor mkjpryor changed the title Drop use of owner references on job resourcesdue to race condition Drop use of owner references on job resources due to race condition Oct 16, 2024
Copy link
Contributor

@sd109 sd109 left a comment

Choose a reason for hiding this comment

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

LGTM

@mkjpryor mkjpryor merged commit 2f0de74 into main Oct 16, 2024
8 of 15 checks passed
@mkjpryor mkjpryor added the bug Something isn't working label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants