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

test flake: oximeter-producer tests::test_producer_registration_task #5645

Closed
sunshowers opened this issue Apr 27, 2024 · 0 comments · Fixed by #5661
Closed

test flake: oximeter-producer tests::test_producer_registration_task #5645

sunshowers opened this issue Apr 27, 2024 · 0 comments · Fixed by #5661
Labels
Test Flake Tests that work. Wait, no. Actually yes. Hang on. Something is broken.

Comments

@sunshowers
Copy link
Contributor

Saw it here: https://buildomat.eng.oxide.computer/wg/0/details/01HWF3FQYGD9FXPMNK03VQWRMS/TKYHnLgC2vm5idnTWgPNdLA4jCSSluaR4V1HCKdLAjo5OYYC/01HWF3G1G84300N8BW78V8MJ5M#S5495
@sunshowers sunshowers added the Test Flake Tests that work. Wait, no. Actually yes. Hang on. Something is broken. label Apr 27, 2024
bnaecker added a commit that referenced this issue Apr 29, 2024
@bnaecker
Fix flaky oximeter reregistration test
f3cfd15 
- Fixes #5645
- Reworks the registration task to use `wait_for_condition`. This
  extends the duration of the test, but is much less flaky when the
  system is loaded.
@bnaecker bnaecker mentioned this issue Apr 29, 2024
Merged
bnaecker added a commit that referenced this issue Apr 29, 2024
@bnaecker
Fix flaky oximeter reregistration test (#5661)
161f5ea 
- Fixes #5645
- Reworks the registration task to use `wait_for_condition`. This
extends the duration of the test, but is much less flaky when the system
is loaded.
@sunshowers sunshowers mentioned this issue May 14, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Test Flake Tests that work. Wait, no. Actually yes. Hang on. Something is broken.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix flaky oximeter reregistration test oxidecomputer/omicron
1 participant
@sunshowers