Skip to content

server: support more customization #210

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

phlogistonjohn
Copy link
Collaborator

To perform practical tests with a bunch of incomplete work I needed to combine custom dev builds of ceph, and samba, and sambacc. This series implements some work to make creating very custom image builds easier. The parts I could cleanly implement quickly at least.

These changes should have no impact on typical non-custom builds.

I start to lose track of things and find it difficult to extend the
CLI of the script when using positional-only arguments. Create
a simplistic long-options style of passing arguments to the
install-packages.sh script.

Signed-off-by: John Mulligan <[email protected]>
I tool I am experimenting with automatically creates .repo files for
you but they're all named the same. Make sure we don't overwrite repo
files with the same url when setting up a custom repo.

Signed-off-by: John Mulligan <[email protected]>
Allow passing an arbitrary list of custom repo urls (space separated).
This makes is a lot simpler to create a very custom image.
When using custom repos, it assumes you are taking control of the ceph
deps (as I am) and turns off auto-setup of the ceph repo when
you use --custom-repos.

Signed-off-by: John Mulligan <[email protected]>
The `custom` (aka `custom-repo`) package source was in our script for a
long time before now, but it was not exposed to the build-image script.
Update the script so one can "easily" create custom images.

Signed-off-by: John Mulligan <[email protected]>
Copy link
Collaborator

@synarete synarete left a comment

Choose a reason for hiding this comment

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

lgtm

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