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.

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

Improve CI caching of fuzz dependencies #14469

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Improve CI caching of fuzz dependencies #14469

wants to merge 2 commits into from
+1 −0

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Nov 20, 2024 •
edited
Loading

cargo fuzz does not appear to be using cached dependencies, and downloads and compiles everything, e.g., https://github.com/astral-sh/ruff/actions/runs/11923959306/job/33233394417

Presumably this is because the crates are not actually dependencies of this project, which the rust-cache action omits by default.

@zanieb zanieb added the ci Related to internal CI tooling label Nov 20, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 20, 2024 •
edited
Loading

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

@zanieb
Copy link
Member Author

zanieb commented Nov 20, 2024 •
edited
Loading

Hm this didn't work, not sure why they're excluded from the cache.

@zanieb zanieb mentioned this pull request Nov 20, 2024
Merged
zanieb added a commit that referenced this pull request Nov 20, 2024
@zanieb
Only build the fuzz crate on main (#14478)
0dbcecc 
It's not actually doing anything per pull request and it's pretty slow?
xref #14469

It seems useful to build on `main` still to find build regressions? e.g.
#9368
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci Related to internal CI tooling
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@zanieb