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

CI: replace cachepot with sccache #6499

Closed
wants to merge 10 commits into from

Conversation

bayandin
Copy link
Member

@bayandin bayandin commented Jan 26, 2024

Problem

Cachepot development has been suspended, from their README:

If you're still using cachepot, at this point, all of the relevant changes should be backported to sccache or are in progress. Feel free to submit a porting request at sccache referencing a PR in cachepot.

Results with sccache:

job no cache1 sccache2
build-neon (debug)* 2m 2m 4s
build-neon (release)* 3m 12s 1m 42s
neon-arch-image (x64) 7m 6s 3m 55s
neon-arch-image (arm64) 13m 8s 5m 59s
compute-node-image-arch (v14, x64) 4m 41s 2m 39s
compute-node-image-arch (v14, arm64) 10m 2s 6m 8s
compute-node-image-arch (v15, x64) 4m 45s 2m 35s
compute-node-image-arch (v15, arm64) 9m 34s 5m 53s
compute-node-image-arch (v16, x64) 4m 48s 2m 48s
compute-node-image-arch (v16, arm64) 10m 2s 6m 9s
  • cargo build-like steps only
  1. no cache: https://github.com/neondatabase/neon/actions/runs/9717370650/job/26828975170
  2. sccache: https://github.com/neondatabase/neon/actions/runs/9732037662/job/26857211223

Summary of changes

  • Replace cachepot with sccache
  • Set correct env vars to make sccache work

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
  • If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

  • Do not forget to reformat commit message to not include the above checklist

Copy link

github-actions bot commented Jan 26, 2024

2940 tests run: 2823 passed, 0 failed, 117 skipped (full report)


Code coverage* (full report)

  • functions: 32.7% (6902 of 21123 functions)
  • lines: 50.1% (54129 of 108144 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
6fca885 at 2024-06-30T13:25:46.358Z :recycle:

@bayandin bayandin force-pushed the bayandin/replace-cachepot-with-sccache branch 2 times, most recently from 75b8b10 to c44fd23 Compare May 24, 2024 12:44
@bayandin bayandin force-pushed the bayandin/replace-cachepot-with-sccache branch 2 times, most recently from b7c9b86 to d906139 Compare June 29, 2024 21:13
@bayandin bayandin force-pushed the bayandin/replace-cachepot-with-sccache branch from d906139 to b7e72e5 Compare June 29, 2024 22:08
@bayandin bayandin force-pushed the bayandin/replace-cachepot-with-sccache branch from 84bbde3 to 8a60644 Compare June 30, 2024 12:20
@bayandin bayandin force-pushed the bayandin/replace-cachepot-with-sccache branch from 8a60644 to 6fca885 Compare June 30, 2024 12:36
@bayandin bayandin changed the title Try to replace cachepot with sccache CI: replace cachepot with sccache Jul 1, 2024
@bayandin bayandin marked this pull request as ready for review July 1, 2024 14:15
@bayandin bayandin closed this Jul 4, 2024
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.

1 participant