Skip to content

Conversation

@siddarthkay
Copy link
Contributor

When building nim compiler we pass $CI_CACHE to scripts/build_nim.sh as the 4th argument but its not set anywhere.

https://github.com/status-im/nimbus-build-system/blob/e6c2c9da39c2d368d9cf420ac22692e99715d22c/makefiles/targets.mk#L80-L88

This could be the reason for weird caching we have been seeing on runner-node-02-linux-01-eu-hel1

Explicitly setting $CI_CACHE to {{runner.temp}} ensure we use a fresh directory always so that we do not get a cached version of nim commit to build.

https://github.com/status-im/nimbus-build-system/blob/e6c2c9da39c2d368d9cf420ac22692e99715d22c/scripts/build_nim.sh#L214

When building nim compiler we pass $CI_CACHE to scripts/build_nim.sh as the 4th argument but its not set anywhere.

https://github.com/status-im/nimbus-build-system/blob/e6c2c9da39c2d368d9cf420ac22692e99715d22c/makefiles/targets.mk#L80-L88

This could be the reason for weird caching we have been seeing on runner-node-02-linux-01-eu-hel1

Explicitly setting $CI_CACHE to {{runner.temp}} ensure we use a fresh directory always so that we do not get a cached version of nim commit to build.

https://github.com/status-im/nimbus-build-system/blob/e6c2c9da39c2d368d9cf420ac22692e99715d22c/scripts/build_nim.sh#L214
@siddarthkay siddarthkay self-assigned this Sep 11, 2025
@github-actions
Copy link

Unit Test Results

       15 files  +         3    2 990 suites  +598   1h 13m 32s ⏱️ + 9m 54s
11 260 tests ±         0  10 698 ✔️ ±         0  562 💤 ±  0  0 ±0 
72 121 runs  +14 425  71 329 ✔️ +14 353  792 💤 +72  0 ±0 

Results for commit d6845cf. ± Comparison against base commit c3d2270.

Copy link
Member

@jakubgs jakubgs left a comment

Choose a reason for hiding this comment

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

@siddarthkay siddarthkay merged commit 3d74066 into unstable Sep 11, 2025
20 checks passed
@siddarthkay siddarthkay deleted the set-ci-cache-to-tmp branch September 11, 2025 10:02
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