Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update test durations automatically based on latest durations from master #12748

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
Merged
merged 6 commits into from
Jun 17, 2025
Merged

Update test durations automatically based on latest durations from master #12748

merged 6 commits into from
Jun 17, 2025

Conversation

k-a-il
Copy link
Contributor

@k-a-il k-a-il commented Jun 12, 2025 •
edited
Loading

Motivation

While switching from CircleCI to GitHub Actions, I saw that the test durations file hadn’t been updated in about a year. Since then, we’ve added a lot of new tests. This pull request adds a workflow that downloads test durations from the latest scheduled run on the master branch, combines them, and creates a PR with the updated test durations file. New workflow is scheduled to run every month.

Changes

  • Added a workflow to download and merge the test durations file, and create a pull request with the updates

Testing

Tested in a forked repostiory.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2025 •
edited
Loading

Test Results - Preflight, Unit

21 621 tests  +8   19 964 ✅ +6   6m 11s ⏱️ -49s
     1 suites ±0    1 657 💤 +2 
     1 files   ±0        0 ❌ ±0 

Results for commit c4975e8. ± Comparison against base commit 3bbf944.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2025 •
edited
Loading

Test Results (MA/MR) - Preflight, Unit

21 621 tests  +8   19 964 ✅ +6   6m 14s ⏱️ -7s
     1 suites ±0    1 657 💤 +2 
     1 files   ±0        0 ❌ ±0 

Results for commit c4975e8. ± Comparison against base commit 3bbf944.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2025 •
edited
Loading

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 8s ⏱️ -1s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit c4975e8. ± Comparison against base commit 3bbf944.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2025 •
edited
Loading

Test Results (amd64, MA/MR) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 5s ⏱️ -14s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit c4975e8. ± Comparison against base commit 3bbf944.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2025 •
edited
Loading

Test Results (MA/MR) - Alternative Providers

987 tests  ±0   589 ✅ ±0   31m 35s ⏱️ + 1m 34s
  4 suites ±0   398 💤 ±0 
  4 files   ±0     0 ❌ ±0 

Results for commit c4975e8. ± Comparison against base commit 3bbf944.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2025 •
edited
Loading

Test Results - Alternative Providers

987 tests  ±0   589 ✅ ±0   30m 37s ⏱️ +15s
  4 suites ±0   398 💤 ±0 
  4 files   ±0     0 ❌ ±0 

Results for commit c4975e8. ± Comparison against base commit 3bbf944.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2025 •
edited
Loading

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 21m 30s ⏱️ + 1m 9s
5 231 tests +1  4 302 ✅ +1  929 💤 ±0  0 ❌ ±0 
5 237 runs  +1  4 302 ✅ +1  935 💤 ±0  0 ❌ ±0 

Results for commit c4975e8. ± Comparison against base commit 3bbf944.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2025 •
edited
Loading

Test Results (amd64, MA/MR) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 23m 14s ⏱️ +48s
5 231 tests +1  4 302 ✅ +1  929 💤 ±0  0 ❌ ±0 
5 237 runs  +1  4 302 ✅ +1  935 💤 ±0  0 ❌ ±0 

Results for commit c4975e8. ± Comparison against base commit 3bbf944.

♻️ This comment has been updated with latest results.

@k-a-il k-a-il self-assigned this Jun 13, 2025
@k-a-il k-a-il added the semver: patch Non-breaking changes which can be included in patch releases label Jun 13, 2025
@k-a-il k-a-il marked this pull request as ready for review June 13, 2025 06:55
alexrashed
alexrashed approved these changes Jun 13, 2025
View reviewed changes
Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! This is really cool! The update-test-durations.yml might be something we want to consider generalizing and moving to our meta repository in the future to use it in other repositories as well 🤩

silv-io
silv-io approved these changes Jun 13, 2025
View reviewed changes
Copy link
Member

@silv-io silv-io left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Just some minor non-blocking suggestions :)

@k-a-il k-a-il merged commit 21c4d5d into master Jun 17, 2025
52 checks passed
@k-a-il k-a-il deleted the ci/update-test-durations-automatically branch June 17, 2025 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexrashed alexrashed alexrashed approved these changes

@silv-io silv-io silv-io approved these changes

@dfangl dfangl Awaiting requested review from dfangl dfangl is a code owner

@dominikschubert dominikschubert Awaiting requested review from dominikschubert dominikschubert is a code owner

Assignees

@k-a-il k-a-il

Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@k-a-il @alexrashed @silv-io