Skip to content

Conversation

@simbit18
Copy link
Contributor

@simbit18 simbit18 commented Nov 22, 2024

Summary

You need to disable the msys2 cache from the GitHub action because we moved the Scheduled Merge Jobs to a new NuttX Mirror Repo and approaching total cache storage limit !!!

Nuttx caches

msys2 GitHub Action cache

Impact

Impact on user: No changes to user-facing functionality
Impact on build: Build process remains the same

Testing

CI
Github

You need to disable the msys2 cache from the GitHub action because we moved the Scheduled Merge Jobs to a new NuttX Mirror Repo and approaching total cache storage limit !!!

https://github.com/apache/nuttx/actions/caches

https://github.com/msys2/setup-msys2?tab=readme-ov-file#cache
@github-actions github-actions bot added Area: CI Size: XS The size of the change in this PR is very small labels Nov 22, 2024
@simbit18
Copy link
Contributor Author

simbit18 commented Nov 22, 2024

@lupyuen will be necessary to remove cache: false from new NuttX Mirror Repo

@xiaoxiang781216 xiaoxiang781216 merged commit a1cbed0 into apache:master Nov 22, 2024
@lupyuen
Copy link
Member

lupyuen commented Nov 23, 2024

Thanks @simbit18! I have given you admin access to the NuttX Mirror Repo, in case you need to do any cache maintenance.

@simbit18
Copy link
Contributor Author

Thanks @lupyuen. I feel honoured !

@simbit18
Copy link
Contributor Author

@lupyuen in the NuttX Mirror Repo I enabled the cache on msys2/setup-msys2 GitHub Action.

@lupyuen
Copy link
Member

lupyuen commented Nov 25, 2024

Thanks for helping out @simbit18! Shall I update my script that enables the builds on the Mirror Repo? enable-macos-windows.sh

Because at every sync, your changes shall be discarded and overwritten like this: NuttX@5091f5d

@simbit18
Copy link
Contributor Author

@lupyuen Yes, then it is necessary

@simbit18 simbit18 deleted the simbit18-cache-msys2 branch November 25, 2024 08:57
@lupyuen
Copy link
Member

lupyuen commented Nov 25, 2024

Yep I have updated the script, we'll see the updates at the next sync. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: CI Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants