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

fix: race conditions for object status updates #2448

Merged
merged 1 commit into from
May 2, 2022
Merged

Conversation

shaneutt
Copy link
Contributor

@shaneutt shaneutt commented May 2, 2022

What this PR does / why we need it:

This resolves race conditions that can occur when updating the status of an object in a controller shortly after caching it in the dataplane client.

Which issue this PR fixes

Resolves #2446

PR Readiness Checklist:

  • the CHANGELOG.md release notes have been updated

@shaneutt shaneutt added bug Something isn't working priority/medium labels May 2, 2022
@shaneutt shaneutt added this to the Gateway API - Milestone 2 milestone May 2, 2022
@shaneutt shaneutt self-assigned this May 2, 2022
@shaneutt shaneutt temporarily deployed to Configure ci May 2, 2022 14:11 Inactive
@shaneutt shaneutt temporarily deployed to Configure ci May 2, 2022 14:12 Inactive
@shaneutt shaneutt temporarily deployed to Configure ci May 2, 2022 14:13 Inactive
@shaneutt shaneutt temporarily deployed to Configure ci May 2, 2022 14:13 Inactive
@shaneutt shaneutt temporarily deployed to Configure ci May 2, 2022 14:33 Inactive
@shaneutt shaneutt marked this pull request as ready for review May 2, 2022 14:36
@shaneutt shaneutt requested a review from a team as a code owner May 2, 2022 14:36
@shaneutt shaneutt enabled auto-merge (rebase) May 2, 2022 15:28
@shaneutt shaneutt merged commit c1cdff9 into main May 2, 2022
@shaneutt shaneutt deleted the shaneutt/fix-2446 branch May 2, 2022 17:32
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.

HTTPRoute translation and status updates may result in race condition
2 participants