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.

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

impr(ci): move some kinds of tests to PR runs only #11272

Merged
merged 2 commits into from
Mar 18, 2025

Conversation

jcgruenhage
Copy link
Member

Problem

The pipelines after release merges are slower than they need to be at the moment. This is because some kinds of tests/checks run on all kinds of pipelines, even though they only matter in some of those.

Summary of changes

Run check-codestyle-{rust,python,jsonnet}, build-and-test-locally and trigger-e2e-tests only on regular PRs, not release PR or pushes to main or release branches.

@jcgruenhage jcgruenhage requested review from a team as code owners March 17, 2025 09:34
@jcgruenhage jcgruenhage requested a review from bayandin March 17, 2025 09:34
@jcgruenhage jcgruenhage force-pushed the jc/move-checks-to-pr-only branch 3 times, most recently from a26f308 to 7e356ec Compare March 17, 2025 10:23
Copy link

github-actions bot commented Mar 17, 2025

7964 tests run: 7580 passed, 0 failed, 384 skipped (full report)


Flaky tests (1)

Postgres 16

Code coverage* (full report)

  • functions: 32.3% (8739 of 27025 functions)
  • lines: 48.4% (74886 of 154737 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
096d37f at 2025-03-18T14:02:54.306Z :recycle:

@jcgruenhage jcgruenhage force-pushed the jc/move-checks-to-pr-only branch 2 times, most recently from 6d82d56 to 6653472 Compare March 17, 2025 13:57
@jcgruenhage jcgruenhage requested a review from bayandin March 17, 2025 13:57
@jcgruenhage jcgruenhage force-pushed the jc/move-checks-to-pr-only branch from 6653472 to b4518d2 Compare March 17, 2025 14:06
@jcgruenhage jcgruenhage force-pushed the jc/move-checks-to-pr-only branch from b4518d2 to 096d37f Compare March 18, 2025 12:34
@jcgruenhage jcgruenhage enabled auto-merge March 18, 2025 12:34
@jcgruenhage jcgruenhage added this pull request to the merge queue Mar 18, 2025
Merged via the queue into main with commit eb6efda Mar 18, 2025
114 checks passed
@jcgruenhage jcgruenhage deleted the jc/move-checks-to-pr-only branch March 18, 2025 14:00
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.

2 participants