[Backport v1.26] Comment out flaky test#2933
Conversation
(cherry picked from commit 039572f)
dd-octo-sts
Bot
added
backport
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: aca04e79bf
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| } | ||
| } | ||
| } | ||
| // func Test_Experiment_StateTransitions(t *testing.T) { |
There was a problem hiding this comment.
Commenting out Test_Experiment_StateTransitions removes the entire 24-scenario state-transition matrix from the runnable test suite, so regressions in restart behavior after terminal phases can ship without signal. Keeping the function registered and using t.Skip("flaky: <tracking issue>") preserves visibility in test listings, keeps the code compiling as helpers evolve, and makes it easier to re-enable targeted subcases once the flake is fixed.
Useful? React with 👍 / 👎.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## v1.26 #2933 +/- ##
==========================================
+ Coverage 40.05% 40.52% +0.46%
==========================================
Files 319 320 +1
Lines 28052 28376 +324
==========================================
+ Hits 11237 11500 +263
- Misses 15993 16032 +39
- Partials 822 844 +22
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
2 participants
Backport 039572f from #2932.
What does this PR do?
Comment out flaky test
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Minimum Agent Versions
Are there minimum versions of the Datadog Agent and/or Cluster Agent required?
Describe your test plan
Write there any instructions and details you may have to test your PR.
Checklist
bug,enhancement,refactoring,documentation,tooling, and/ordependenciesqa/skip-qalabel