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

chore: Update CI to improve throughput and caching #61

Merged
merged 65 commits into from
Feb 8, 2023

Conversation

swlynch99
Copy link
Contributor

@swlynch99 swlynch99 commented Feb 7, 2023

This PR is meant to improve the throughput of CI and reduce its cache footprint. This way, if multiple PRs are being updated at the same time then they'll spend less time waiting. It does not reduce the E2E latency of the cache pipline, instead it bumps jobs that are less important to the end and focuses on reducing total compute time.

Detailed changes:

  • Avoid using any actions from actions-rs because they have not been maintained in years
  • Move all builds for fuzz targets to a single job and cache entry (fuzzing still runs in separate jobs)
  • Share caches between jobs in the regular cargo workflow where possible
  • Capture clippy annotations and expose them via the GH UI using clippy-sarif
  • Smoketest jobs no longer build their binaries, instead they use binaries compiled by the ubuntu-latest build job
  • I've removed bench-check since it does nothing that isn't covered by build or check

@swlynch99 swlynch99 marked this pull request as draft February 7, 2023 19:56
@swlynch99 swlynch99 marked this pull request as ready for review February 7, 2023 20:37
@swlynch99 swlynch99 enabled auto-merge (squash) February 7, 2023 21:09
Copy link
Collaborator

@brayniac brayniac left a comment

Choose a reason for hiding this comment

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

LGTM

@swlynch99 swlynch99 enabled auto-merge (squash) February 7, 2023 21:30
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

3 participants