Skip to content

Fix flaky aws remote sampler test #2051

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

Merged
merged 1 commit into from
Jul 24, 2025

Conversation

laurit
Copy link
Contributor

@laurit laurit commented Jul 23, 2025

Sampler update time is set up in a background thread. By the time the removed assertion is run it is usually not set, but sometimes is.

@laurit laurit requested a review from a team July 23, 2025 12:00
@trask
Copy link
Member

trask commented Jul 23, 2025

Closing and re-opening to retrigger component owner reviews since I've just fixed that (hopefully)

@trask trask closed this Jul 23, 2025
@trask trask reopened this Jul 23, 2025
@github-actions github-actions bot requested review from srprash and wangzlei July 23, 2025 18:12
@laurit laurit added this pull request to the merge queue Jul 24, 2025
Merged via the queue into open-telemetry:main with commit 87de708 Jul 24, 2025
59 checks passed
@laurit laurit deleted the aws-sampler-flaky-test branch July 24, 2025 06:10
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.

4 participants