-
Notifications
You must be signed in to change notification settings - Fork 397
DEBUG-4548 Increase number of iterations for flakiness #5184
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
Conversation
|
✅ Tests 🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage 🔗 Commit SHA: bfc4f90 | Docs | Datadog PR Page | Was this helpful? Give us feedback! |
ivoanjo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looking at this test it looks quite brittle so in some future we should probably review it, but for now agree with holding our noses and try to get rid of the flakiness.
* master: Ignore "leaked" pipe file descriptors in JRuby, improve diagnostics (#5188) debug-4548 Increase number of iterations for flakiness (#5184) [🤖] Update System Tests: https://github.com/DataDog/dd-trace-rb/actions/runs/20684824141 [🤖] Update System Tests: https://github.com/DataDog/dd-trace-rb/actions/runs/20616292456 (#5190)
* master: Ignore "leaked" pipe file descriptors in JRuby, improve diagnostics (#5188) debug-4548 Increase number of iterations for flakiness (#5184) [🤖] Update System Tests: https://github.com/DataDog/dd-trace-rb/actions/runs/20684824141
* master: (129 commits) Transports: remove api_version (#5164) DEBUG-4548 Telemetry: send events in forked children + telemetry metrics reset after fork (#5159) Ignore "leaked" pipe file descriptors in JRuby, improve diagnostics (#5188) debug-4548 Increase number of iterations for flakiness (#5184) [🤖] Update System Tests: https://github.com/DataDog/dd-trace-rb/actions/runs/20684824141 [🤖] Update System Tests: https://github.com/DataDog/dd-trace-rb/actions/runs/20616292456 (#5190) downgrade ffi for ruby 4.0 & 2.5 (#5189) Fix ruby warnings when accessing undefined instance variables (#5178) DEBUG-3499 DI: fix accounting when intrumenting upon class definition, add instr… (#5168) DEBUG-3499 RC: add diagnostics for invalid values (#5167) DEBUG-4548 Core: fix worker shutdown race (#5176) Retry system-test build (#5181) Fix Baggage type check (#5182) [🤖] Update System Tests: https://github.com/DataDog/dd-trace-rb/actions/runs/20487829791 (#5183) [🤖] Update Latest Dependency: https://github.com/DataDog/dd-trace-rb/actions/runs/20401889084 (#5180) DEBUG-3499 DI: do not instrument when there is already an installed probe with the same id (#5169) DEBUG-3499 DI: rework RC interface (#5165) set DI test duration upper bound to 1000 seconds (#5161) add missing supported config default value [🤖] Update System Tests: https://github.com/DataDog/dd-trace-rb/actions/runs/20401907816 (#5179) ...
* master: Transports: remove api_version (#5164) DEBUG-4548 Telemetry: send events in forked children + telemetry metrics reset after fork (#5159) Ignore "leaked" pipe file descriptors in JRuby, improve diagnostics (#5188) debug-4548 Increase number of iterations for flakiness (#5184) [🤖] Update System Tests: https://github.com/DataDog/dd-trace-rb/actions/runs/20684824141
What does this PR do?
Increases number of iterations in CI for the worker race test added in #5176
Motivation:
CI is flaking: https://github.com/DataDog/dd-trace-rb/actions/runs/20491855096/job/58885380113?pr=5159
Change log entry
None
Additional Notes:
There is no guarantee that any reasonable bound will suffice...
I would like to retain the test.
How to test the change?
Existing CI not failing