-
Notifications
You must be signed in to change notification settings - Fork 38
Add list of additional package installs #499
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
Conversation
…rm-appliance into feat/extra-packages
sjpb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comments on docs only, will need an image rebuild when rebasing onto main.
Co-authored-by: Steve Brasier <[email protected]>
sjpb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry ... variable name changes AND corresponding docs changes please!
Co-authored-by: Steve Brasier <[email protected]>
sjpb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
bertiethorpe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Packages list in
appliances_default_extra_packagesare now installed during build by default. Can optionally re-run install commands during site by settingappliances_packages_during_configure