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: Add GitHub Actions #601

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ci: Add GitHub Actions #601

wants to merge 1 commit into from

Conversation

crawfxrd
Copy link
Member

@crawfxrd crawfxrd commented Feb 26, 2025

TODO:

  • Determine cache storage usage/cost
    • coreboot toolchain is using 160 MB, and apparently there's 10 GB available
  • Determine artifact storage usage/cost
    • disabled upload-artifact for now because it's going to be a lot of space
  • Replace script in private firmware repo for downloading build artifacts when staging updates
    • depends on uploading artifacts
  • Investigate reducing CI time, esp. on cloning all the submodules and installing build dependencies
  • Limit the models built by default to not hit concurrency limit?
    • But would want a way to manually trigger a build by specifying models like Jenkins currently does

@crawfxrd crawfxrd marked this pull request as ready for review February 26, 2025 19:54
@crawfxrd crawfxrd requested review from a team February 26, 2025 19:55
@crawfxrd
Copy link
Member Author

Received non-retryable error: Failed request: (409) Conflict: cache entry with the same key, version, and scope already exists

The fuck you mean?

@crawfxrd crawfxrd force-pushed the gha branch 3 times, most recently from 9c75a18 to 5e99b72 Compare February 27, 2025 16:02
Signed-off-by: Tim Crawford <tcrawford@system76.com>
@crawfxrd
Copy link
Member Author

usb.img is removed from the build artifacts as it takes up a lot of space (at least doubling the size of the output) but is not normally used. It would still be >300 MiB for the artifacts per commit, so I've disabled use of actions/upload-artifact.

UEFIPAYLOAD.fd could be removed to reduce artifacts by another 8 MiB each, as this is already included in firmware.rom.

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