Skip to content

Conversation

@a-gave
Copy link

@a-gave a-gave commented Apr 5, 2025

document an alpine setup to self build smaller imagebuilder containers

@a-gave a-gave marked this pull request as ready for review June 14, 2025 14:32
@a-gave
Copy link
Author

a-gave commented Jun 14, 2025

I saw there was a little interest on this: #8

It seems convenient to have also alpine based images to reduce the size of imagebuilders and sdks, eg:

ghcr.io/openwrt/imagebuilder-alpine  ath79-generic-v24.10.1        460321c3b32e  7 weeks ago        1.56 GB
ghcr.io/openwrt/imagebuilder     ath79-generic-v24.10.1   6b78c2158b99   8 weeks ago     2.07GB

and for faster pulls from registries.

Maybe the list of requirements installed by apk could be reviewed
Smoke tests from the ci seem to work correctly.

ping @aparcar who reviewed #8

@a-gave a-gave changed the title Dockerfile: add or document an alpine setup (imagebuilder only) Dockerfile: add or document an alpine setup Jun 14, 2025
@a-gave
Copy link
Author

a-gave commented Jun 15, 2025

fix: move docker args and envs after the installation of system requirements.

@aparcar
Copy link
Member

aparcar commented Jun 18, 2025

@ynezz what do you think?

@a-gave
Copy link
Author

a-gave commented Jun 18, 2025

fix: set default distros when no input from workflow dispatch is provided
fix: run rootfs builds once since they are then based on scratch

Copy link
Member

@aparcar aparcar left a comment

Choose a reason for hiding this comment

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

Thanks for your work, some minor changes please

@a-gave
Copy link
Author

a-gave commented Aug 15, 2025

Thanks for reviewing. Submitted the requested changes.

Signed-off-by: Samuele Longhi <agave@dracaena.it>
@a-gave
Copy link
Author

a-gave commented Dec 5, 2025

fd93873 adds the missing path to Dockerfile also to the step Push.

I loaded some images based on alpine at https://hub.docker.com/r/agave0/openwrt-imagebuilder
There are still some minor things to clean, like it is missing /builder/tmp and ib is using as fallback /tmp

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