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

Changing OS Image build via kiwi to running via podman and adding eib pre-processing #9507

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

Conversation

Martin-Weiss
Copy link

What does this PR change?

We worked on this as part of https://hackweek.opensuse.org/projects/build-edge-image-builder-iso-with-suse-manager

In this PR we have several changes:

  1. switch from local installed kiwi to running kiwi via image
  2. run eib pre-processing in case eib/eib.yaml exists

We hope someone else can take this forward as this is just an initial "PoC" even though it is working well for us.
Limitations that might need to be addressed as well:

  • did not remove the installation of kiwi-ng
  • did not test on SLES 12 build host
  • did not add "air-gapped" - so the build-hosts need to be able to pull these images from the internet:
docker.io/dgiebert/edge-image-builder:1.2.7
registry.suse.com/bci/kiwi:10.1.10

We have been testing with https://github.com/Martin-Weiss/kiwi-image-micro-gpu-60

GUI diff

No difference.

Documentation

  • We need to explain the git repo eib/eib.yaml and point to the official EIB docs...
  • We need to explain Podman requirements

Test coverage

ℹ️ If a major new functionality is added, it is strongly recommended that tests for the new functionality are added to the Cucumber test suite

  • We did not add test-cases so far - but build on x86, aarch64 and with repos having eib/eib.yaml and without need to be added.

@admd admd assigned aaannz, mcalmer and rjmateus and unassigned mcalmer, aaannz and rjmateus Dec 2, 2024
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.

4 participants