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

Update index.rst with base image OS info #693

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

onefourfive
Copy link

Add base image OS recommendations to resolve #692

@onefourfive onefourfive requested a review from a team as a code owner July 17, 2024 00:43
@github-actions github-actions bot added docs Changes to documentation needs_triage New item that needs to be triaged labels Jul 17, 2024
Copy link

sonarcloud bot commented Jul 26, 2024

@@ -55,7 +55,7 @@ execution environment. You can specify these items:
Choosing a base image
---------------------

You can use any base image you choose.
You can use any base image you choose, as long as it's a RHEL-type image like CentOS, Fedora, or RHEL. These operating systems are directly supported.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this reads slightly better.

Suggested change
You can use any base image you choose, as long as it's a RHEL-type image like CentOS, Fedora, or RHEL. These operating systems are directly supported.
The base image must be a RHEL image or one of the RHEL-based variants (CentOS, Fedora, etc.). Other distributions are not supported at this time.

Copy link
Contributor

Choose a reason for hiding this comment

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

I wouldn't say "RHEL-based variants", since CentOS and Fedora are not based on RHEL (as Ubuntu is based on Debian). Maybe say something like "... or one of the similar distributions (CentOS, ..."?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Changes to documentation needs_triage New item that needs to be triaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docs: Recommend using a RHEL image
3 participants