Skip to content

Conversation

@vpellan
Copy link
Contributor

@vpellan vpellan commented Oct 7, 2025

What does this PR do?

This PR adds a seed input to the CI Unit tests workflow, which should give the same batches output, and same RSpec seeds.

Motivation:

It should be easier to reproduce flaky tests

Change log entry

None.

Additional Notes:

How to test the change?

Once merged, workflow_dispatch

@datadog-official
Copy link

datadog-official bot commented Oct 7, 2025

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage
Patch Coverage: 100.00%
Total Coverage: 101.96% (-0.01%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 89b7193 | Docs | Was this helpful? Give us feedback!

@pr-commenter
Copy link

pr-commenter bot commented Oct 7, 2025

Benchmarks

Benchmark execution time: 2025-10-07 12:41:53

Comparing candidate commit 89b7193 in PR branch vpellan/make-ci-reproducible with baseline commit 0a32491 in branch master.

Found 1 performance improvements and 0 performance regressions! Performance is the same for 43 metrics, 2 unstable metrics.

scenario:profiling - stack collector (ruby frames - native filenames enabled)

  • 🟩 throughput [+183.430op/s; +185.427op/s] or [+5.631%; +5.692%]

@vpellan vpellan force-pushed the vpellan/make-ci-reproducible branch from bd73b48 to 89b7193 Compare October 7, 2025 12:11
@vpellan vpellan marked this pull request as ready for review October 7, 2025 12:16
@vpellan vpellan requested a review from a team as a code owner October 7, 2025 12:16
@vpellan vpellan merged commit d865382 into master Oct 7, 2025
478 of 480 checks passed
@vpellan vpellan deleted the vpellan/make-ci-reproducible branch October 7, 2025 14:38
@github-actions github-actions bot added this to the 2.22.0 milestone Oct 7, 2025
vpellan added a commit that referenced this pull request Oct 13, 2025
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.

3 participants