-
Notifications
You must be signed in to change notification settings - Fork 108
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 installer customizations to blueprints and cloud API (HMS-1161) #4005
Conversation
vsphere failure is a known issue (server down). |
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.
Looks good to me. Schutzbot needs a vacation though :)
e82335b
to
3b49dbe
Compare
@mcattamoredhat pointed out an issue with the kickstart injection in the ostree-ng test. Fixed now. |
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.
Looks good. Schutzbot seems to have had a good rest, previously failing tests are passing, just waiting for it to finish up.
3b49dbe
to
79c297a
Compare
79c297a
to
74d81b4
Compare
@ondrejbudai Can I get a quick re-approval and get this merged? The only change since the previous approvals was the test fix (and a rebase). |
Use the new installer customizations to build and boot an unattended installation and add the users to the sudoers with NOPASSWD enabled. This is added both to he base installer test (installers.sh) and the ostree test (ostree-ng.sh). For the ostree test, we keep the mkksiso step to retain the remote configuration change. The modksiso() function is changed to explicitly read the osbuild.ks file since we don't want to modify the osbuild-base.ks file and we want to ensure that the kernel command line argument points to the osbuild.ks file.
Adding both as a traditional compose request customization and in the new blueprint customizations.
Nightly pipelines test against older osbuild-composer versions so we need to still modify the kickstart file until the feature reaches the nightly composes.
57b8d8e
to
fc04a6c
Compare
Again! |
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 👍
Describe the new installer customizations introduced into osbuild-composer in osbuild/osbuild-composer#4005.
Describe the new installer customizations introduced into osbuild-composer in osbuild/osbuild-composer#4005.
Describe the new installer customizations introduced into osbuild-composer in osbuild/osbuild-composer#4005.
This PR updates the osbuild/images dependency and exposes the new installer customizations in the blueprint and the cloud API.
Relevant osbuild/images PRs:
The kickstart injection in the installer test had been removed and we now use the new options to create an unattended ISO. In the edge installer test (
ostree-ng.sh
), we keep the mkksiso call to add the ostree remote fix, but remove all the parts covered by our new customizations.This pull request includes: