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

Slim down the Fedora 39 builder some more #4494

Closed
wants to merge 1 commit into from
Closed

Slim down the Fedora 39 builder some more #4494

wants to merge 1 commit into from

Conversation

13r0ck
Copy link

@13r0ck 13r0ck commented Feb 9, 2024

By using buildah and podman we can build a container that doesn't have dnf nor python3-pip installed. This shrinks the fedora container down ~30% more (~300MB)

Likely needs a few tweaks to fully integrate with the CI. Host needs python3-pip, buildah, and podman installed. If that is ok?

By using buildah and podman we can build a container that
doesn't have dnf nor python3-pip installed. This shrinks the
fedora container down ~30% more (~300MB)
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Feb 9, 2024
@SeanTAllen
Copy link
Member

Thanks for this. Given that I am the only person supporting these, I want to keep it to the stock docker I have installed and not install anything in the build environment as that is my machine for these.

I'm ok with the extra image size for that tradeoff.

@13r0ck
Copy link
Author

13r0ck commented Feb 10, 2024

😞

@13r0ck 13r0ck closed this Feb 10, 2024
@ponylang-main ponylang-main added discuss during sync Should be discussed during an upcoming sync and removed discuss during sync Should be discussed during an upcoming sync labels Feb 10, 2024
@13r0ck 13r0ck deleted the shrink-fedora-container branch February 10, 2024 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss during sync Should be discussed during an upcoming sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants