Skip to content

Commit

Permalink
[chore] Do more steps in parallel in build-and-test workflow (open-te…
Browse files Browse the repository at this point in the history
…lemetry#30891)

The build-and-test workflow takes a long time. Part of the issue is that
we spend a lot of time waiting for long jobs, such as integration tests,
to finish before moving onto more long jobs.

This PR attempts to parallelize more steps by removing some existing
ordering we've restricted ourselves to. Specifically this moves the
cross-compile chain to start in parallel with the unit tests, lint, and
integration tests. My reasoning is that if a unit tests or lint is bad
it will still get caught and we don't need unit tests to pass to attempt
to compile the code. I have update the publish-check step to still
depend on all these steps.

Related to
open-telemetry#30880
  • Loading branch information
TylerHelmuth authored and cparkins committed Feb 1, 2024
1 parent 10bf0ec commit e53c25f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build-and-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -395,7 +395,7 @@ jobs:

cross-compile:
runs-on: ubuntu-latest
needs: [unittest, integration-tests, lint]
needs: [setup-environment]
strategy:
matrix:
os:
Expand Down Expand Up @@ -543,7 +543,7 @@ jobs:

publish-check:
runs-on: ubuntu-latest
needs: [build-package]
needs: [lint, unittest, integration-tests, build-package]
steps:
- uses: actions/checkout@v4
- name: Download Binaries
Expand Down

0 comments on commit e53c25f

Please sign in to comment.