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

Add example script to help build airgapped package repo #43

Closed
wants to merge 0 commits into from
Closed

Add example script to help build airgapped package repo #43

wants to merge 0 commits into from

Conversation

rdoxenham
Copy link
Contributor

This script provides the ability to specify a list of local packages, along with any packages that you want to pull from SLE Micro (or additionally specified package repos) to generate a local package repository that contains all of the RPM's (and any identified dependencies) based on a base image provided by the user as the source, so it can be used within Combustion. It generates the file /tmp/eib/repo/package-list.txt with all of the packages that can be passed into zypper in at the Combustion stage.

Copy link
Contributor

@jdob jdob left a comment

Choose a reason for hiding this comment

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

I appreciate the resource effort that went into this, but I don't see the value in checking this into the repo as an executable script. It's not something an EIB user or dev would ever need to run, especially once the important bits have been extracted out of this into EIB functionality. Arguably, putting it here as a script puts us on the hook to maintain it and keep feature parity with this script v. what EIB does internally.

If you want to commit this as research, I'd prefer to see it under /docs as a design doc with some context around it (the comments do a really good job of that, so mostly a note that says this was a POC to show how to do it).

@rdoxenham rdoxenham closed this Dec 11, 2023
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