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

build: allow validating image-size of resulting images #3097

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

blocktrron
Copy link
Member

This Pull-Request adds the ability to check the resulting images from a Gluon build to be checked in terms of available image-size.

Gluon already fails in case an image is discarded due to exceeding the available size. However, this ignores the requirement of additional space for the jffs2 overlay.

Include the IMAGE_SIZE defined in OpenWrt into the profile.json. Also start copying this file to the Gluon output directory.
Add a script for validating the image-size with a customizable overhead.
Include this script into the CI action of Gluon.

@Djfe
Copy link
Contributor

Djfe commented Jul 28, 2024

Interesting PR that I missed, thanks for showing it to me :)

why is overhead set to 256 in particular?
it assume it represents erase-size, so it's either:

  • 4KiB *5 = 20KiB (we likely want to pad that since it's not enough for the gluon config)
  • 64KiB *5 = 320KiB
  • 128KiB *5 = 640 KiB
  • 256KiB *5 = 1280KiB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants