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.

Sign up for GitHub

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

Jump to bottom

Handle race condition where events are reported out of order #4115

Merged
merged 1 commit into from
Oct 31, 2023
Merged

Handle race condition where events are reported out of order #4115

merged 1 commit into from
Oct 31, 2023

Conversation

sgalsaleh
Copy link
Member

@sgalsaleh sgalsaleh commented Oct 31, 2023 •
edited
Loading

What this PR does / why we need it:

Fixes an issue where in rare scenarios, telemetry events that closely followed each other were reported out of order.

Which issue(s) this PR fixes:

Fixes SC-91640

Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Fixes an issue where in rare scenarios, telemetry events that closely followed each other were reported out of order.

Does this PR require documentation?

NONE

@sgalsaleh sgalsaleh added type::bug Something isn't working bug::normal labels Oct 31, 2023
@sgalsaleh sgalsaleh requested a review from cbodonnell October 31, 2023 19:06
@sgalsaleh sgalsaleh merged commit e048c7b into main Oct 31, 2023
231 of 238 checks passed
@sgalsaleh sgalsaleh deleted the salah/sc-91640/race-condition-when-kots-and-the-sdk-report branch October 31, 2023 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cbodonnell cbodonnell cbodonnell approved these changes

Assignees
No one assigned
Labels
bug::normal type::bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sgalsaleh @cbodonnell