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

WIP: Add Image/Package Mode feature to os_vendor #166

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

kdvalin
Copy link
Member

@kdvalin kdvalin commented Feb 10, 2025

Description

This PR adds the ability to simply specify os_vendor: rhel:mode=image to work with bootc images.

Presently all this does is enable --no_packages automatically since you cannot install packages in bootc/image mode.

Before/After Comparison

Before

User would need to manually specify --no_packages and a writable --kit_upload_directory to make Zathras work.

After

The user needs to only specify --os_mode=bootc or add it to the os_vendor in the scenario file, like os_vendor: rhel:mode=image.

Clerical Stuff

This closes #165

Relates to JIRA: RPOPC-326

Copy link

This relates to RPOPC-326

@kdvalin kdvalin requested a review from dvalinrh February 10, 2025 19:36
fi
done
if [[ $os_vendor_found -eq 0 ]]; then
cleanup_and_exit "Error: $1, is an unrecognized OS vendor. Possible values are: $gl_valid_os_vendors" 1
fi

if [ -n "$os_args" ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

Double [[, not single

fi
;;
*)
cleanup_and_exit "Error $mode is an unknown mode, valid options are: 'package' and 'image'" 1
Copy link
Contributor

Choose a reason for hiding this comment

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

You need to remove the temp file on failure.

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.

Add Bootc/RHEL Package Mode support
2 participants