Skip to content

Conversation

@rpardini
Copy link
Member

build: gha: actions/checkout: don't clean working copy & fetch full history from git

  • instead of cleaning the whole working copy, clean out just the out and dist dirs
  • this allows the cache/ dir to survive on-disk on self-hosted runners
  • bonus: using fetch-depth: 0 allows for much faster git fetch times on self-hosted runners over time
  • this is done only for the jobs that can run in self-hosted runners (not prep/release jobs)

Signed-off-by: Ricardo Pardini ricardo@pardini.net

@rpardini
Copy link
Member Author

Altenative to #230

@jacobweinstock
Copy link
Member

Thanks for this @rpardini , both this a #230 are very reasonable. I think i'm more in favor of #230. Let's use that one and see how it goes. Thanks for looking into this!

…istory from git

- instead of cleaning the whole working copy, clean out just the `out` and `dist` dirs
- this allows the `cache/` dir to survive on-disk on self-hosted runners
- bonus: using `fetch-depth: 0` allows for much faster `git fetch` times on self-hosted runners over time
- this is done only for the jobs that can run in self-hosted runners (not prep/release jobs)

Signed-off-by: Ricardo Pardini <ricardo@pardini.net>
@rpardini rpardini force-pushed the preserve-cache-dir-across-CI-runs branch from 08f2e8e to dbf4c63 Compare June 27, 2024 12:17
@rpardini
Copy link
Member Author

Hey @jacobweinstock I haven't followed up, but it looks to me #230 was good enough? If so I'll close this....

@jacobweinstock
Copy link
Member

Hey @jacobweinstock I haven't followed up, but it looks to me #230 was good enough? If so I'll close this....

Hey @rpardini. Yeah, we should be good for now, thanks!

@rpardini rpardini closed this Jun 28, 2024
@rpardini rpardini deleted the preserve-cache-dir-across-CI-runs branch March 5, 2025 10:05
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.

2 participants