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

Adds GitHub Action to Build Raspbian Lite #786

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

theGOTOguy
Copy link

This action uses entirely default settings, and disables stage3-stage5 builds. I was unable to get any of these builds to complete within the space constraints of the GitHub runners, but if they add disk space to their runners it may be possible to build these in the future.

Tested and confirmed working on a Raspberry Pi Zero W.

Completes #785

Copy link

@usimd usimd left a comment

Choose a reason for hiding this comment

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

Cool idea to increase the disk space for building. Thinking about adding this to pi-gen-action as well.

echo "Free space:"
df -h

- name: Install Dependencies
Copy link

Choose a reason for hiding this comment

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

You could reuse the declared build dependencies instead of listing them manually.

Copy link
Author

Choose a reason for hiding this comment

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

Good call. Mostly using depends now, but there was one additional needed dependency, qemu-user-static. Not sure whether the right thing is to put it into depends, but for now it's installed separately as part of the build step.

Copy link

Choose a reason for hiding this comment

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

Maybe @XECDesign can elaborate on whether this could be added to depends as well.

.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
@theGOTOguy
Copy link
Author

Updated dependency install and upload-artifact and checkout versions.

Build confirmed working: https://github.com/theGOTOguy/pi-gen/actions/runs/10175875697

@usimd
Copy link

usimd commented Nov 7, 2024

FWIW, I've released a new version of pi-gen-action that includes a similar but more aggressive mechanism as in easimon/maximize-build-space.

With this option turned on (increase-runner-disk-size: true) I was able to build a full image up to stage5 on a free GH runner. If you're curious and maybe want to extend your change, have a look at the additional resources to free:

https://github.com/usimd/pi-gen-action/blob/afb6cdd932ef6e17cc3850ace040df9430325065/src/increase-runner-disk-size.ts#L13-L102

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