Skip to content

chore(ci): make sccache fail gracefully#4208

Merged
dignifiedquire merged 5 commits intomainfrom
arqu/sccache_check
Apr 30, 2026
Merged

chore(ci): make sccache fail gracefully#4208
dignifiedquire merged 5 commits intomainfrom
arqu/sccache_check

Conversation

@Arqu
Copy link
Copy Markdown
Collaborator

@Arqu Arqu commented Apr 29, 2026

Description

Wraps the sccache config into a check so if infra fails for some reason it retries and if that fails too it moves on to complete the job without it.

Breaking Changes

Notes & open questions

Change checklist

  • Self-review.
  • Documentation updates following the style guide, if relevant.
  • Tests if relevant.
  • All breaking changes documented.
    • List all breaking changes in the above "Breaking Changes" section.
    • Open an issue or PR on any number0 repos that are affected by this breaking change. Give guidance on how the updates should be handled or do the actual updates themselves. The major ones are:

@Arqu Arqu requested review from Frando and dignifiedquire April 29, 2026 09:22
@Arqu Arqu self-assigned this Apr 29, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh/pr/4208/docs/iroh/

Last updated: 2026-04-29T12:57:45Z

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026

Netsim report & logs for this PR have been generated and is available at: LOGS
This report will remain available for 3 days.

Last updated for commit: cdf356b

@n0bot n0bot Bot added this to iroh Apr 29, 2026
@github-project-automation github-project-automation Bot moved this to 🚑 Needs Triage in iroh Apr 29, 2026
@Arqu Arqu moved this from 🚑 Needs Triage to 👀 In review in iroh Apr 29, 2026
done
echo "::warning::sccache backend unreachable; running this job without sccache"
# mozilla-actions/sccache-action exports these via $GITHUB_ENV — override.
echo "RUSTC_WRAPPER=" >> "$GITHUB_ENV"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this the right way to clear an env variable? :O

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

well 2 things happening here:
a) if it was set before you kinda need to unset it so this would be bash wise wrong
b) this is github actions and they have their funny github_env thing which I'd rather not touch more than needed and it doesn't have proper unset semantics

@dignifiedquire dignifiedquire merged commit a187130 into main Apr 30, 2026
35 checks passed
@dignifiedquire dignifiedquire deleted the arqu/sccache_check branch April 30, 2026 07:55
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in iroh Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants