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

CI(build-and-test): move part of the pipeline to a reusable workflow #8241

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bayandin
Copy link
Member

@bayandin bayandin commented Jul 2, 2024

Problem

  • build-and-test workflow is pretty big
  • jobs that depend on the matrix job don't start before all variations are done. I.e. regress-tests depend on build-neon, but we can't start regress-tests on the release configuration until build-neon is done on release and debug configurations. This will be more visible once we add ARM to the matrix.

Summary of changes

  • Move jobs related to building (build-neon) and testing (regress-tests) to a separate job

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
  • If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

  • Do not forget to reformat commit message to not include the above checklist

Copy link

github-actions bot commented Jul 2, 2024

3042 tests run: 2927 passed, 0 failed, 115 skipped (full report)


Flaky tests (1)

Postgres 14

Code coverage* (full report)

  • functions: 32.6% (6931 of 21275 functions)
  • lines: 50.0% (54494 of 108968 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
e6bf243 at 2024-07-05T16:05:53.996Z :recycle:

@bayandin bayandin force-pushed the bayandin/build-and-test-locally branch from 87c9add to 762c483 Compare July 3, 2024 11:14
@bayandin bayandin marked this pull request as ready for review July 3, 2024 19:18
@bayandin bayandin force-pushed the bayandin/build-and-test-locally branch 2 times, most recently from 6c44378 to 481234e Compare July 3, 2024 19:21
@bayandin bayandin added the run-benchmarks Indicates to the CI that benchmarks should be run for PR marked with this label label Jul 3, 2024
@bayandin bayandin force-pushed the bayandin/build-and-test-locally branch from 481234e to 3b76151 Compare July 3, 2024 19:34
@bayandin bayandin force-pushed the bayandin/build-and-test-locally branch from 3b76151 to 2029d88 Compare July 3, 2024 22:58
@bayandin bayandin changed the base branch from main to bayandin/add-conclusion July 3, 2024 22:58
@bayandin bayandin force-pushed the bayandin/build-and-test-locally branch from 2029d88 to 020f399 Compare July 4, 2024 07:23
Base automatically changed from bayandin/add-conclusion to main July 4, 2024 08:20
@bayandin bayandin force-pushed the bayandin/build-and-test-locally branch 3 times, most recently from d5a0e3b to c26f072 Compare July 5, 2024 11:46
@bayandin bayandin removed the run-benchmarks Indicates to the CI that benchmarks should be run for PR marked with this label label Jul 5, 2024
@bayandin bayandin force-pushed the bayandin/build-and-test-locally branch 2 times, most recently from 3db9674 to b1e2379 Compare July 5, 2024 14:20
@bayandin bayandin force-pushed the bayandin/build-and-test-locally branch from b1e2379 to e6bf243 Compare July 5, 2024 15:17
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.

None yet

1 participant