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 installer customizations to blueprints and cloud API (HMS-1161) #4005

Merged
merged 4 commits into from
Mar 20, 2024

Conversation

achilleas-k
Copy link
Member

@achilleas-k achilleas-k commented Mar 12, 2024

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:

  • adequate testing for the new functionality or fixed issue
  • adequate documentation informing people about the change such as
    • submit a PR for the READMEs listed here
    • submit a PR for the osbuild.org website repository if this PR changed any behavior not covered by the automatically updated READMEs

@achilleas-k
Copy link
Member Author

vsphere failure is a known issue (server down).

bcl
bcl previously approved these changes Mar 13, 2024
Copy link
Contributor

@bcl bcl left a 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 :)

ondrejbudai
ondrejbudai previously approved these changes Mar 14, 2024
@achilleas-k
Copy link
Member Author

@mcattamoredhat pointed out an issue with the kickstart injection in the ostree-ng test. Fixed now.

bcl
bcl previously approved these changes Mar 14, 2024
Copy link
Contributor

@bcl bcl left a 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.

@achilleas-k
Copy link
Member Author

achilleas-k commented Mar 18, 2024

@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).

ondrejbudai
ondrejbudai previously approved these changes Mar 18, 2024
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.
@achilleas-k
Copy link
Member Author

vsphere failure is a known issue (server down).

Again!

Copy link
Member

@thozza thozza left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@thozza thozza merged commit 3e0f8a7 into osbuild:main Mar 20, 2024
78 checks passed
@achilleas-k achilleas-k deleted the unattended-iso branch March 20, 2024 11:06
achilleas-k added a commit to osbuild/osbuild.github.io that referenced this pull request Apr 23, 2024
Describe the new installer customizations introduced into
osbuild-composer in
osbuild/osbuild-composer#4005.
ondrejbudai pushed a commit to osbuild/osbuild.github.io that referenced this pull request Apr 24, 2024
Describe the new installer customizations introduced into
osbuild-composer in
osbuild/osbuild-composer#4005.
achilleas-k added a commit to osbuild/osbuild.github.io that referenced this pull request Apr 24, 2024
Describe the new installer customizations introduced into
osbuild-composer in
osbuild/osbuild-composer#4005.
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