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 link about containers for RHEL 10 #3622

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

bangelic
Copy link
Contributor

What changes are you introducing?

Adding link for containers for RHEL 10.
Add heading for Additional resources.
Make list of links in bullet form.

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

In reference to https://issues.redhat.com/browse/SAT-28100.

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

The link being added is still in beta but will be changed in another PR when beta goes alpha.

Checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.13/Katello 4.15 (EL9 only)
  • Foreman 3.12/Katello 4.14 (Satellite 6.16)
  • Foreman 3.11/Katello 4.13 (orcharhino 6.11 on EL8 only; orcharhino 7.0 on EL8+EL9)
  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (Satellite 6.15; orcharhino 6.8/6.9/6.10)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • We do not accept PRs for Foreman older than 3.7.

Copy link

github-actions bot commented Jan 28, 2025

The PR preview for 831087e is available at theforeman-foreman-documentation-preview-pr-3622.surge.sh

The following output files are affected by this PR:

show diff

show diff as HTML

Copy link
Contributor

@Lennonka Lennonka left a comment

Choose a reason for hiding this comment

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

Thank you for formatting the links into a list!

However, I think we should wait until RHEL 10 GA with this one. WDYT?

@bangelic
Copy link
Contributor Author

Thank you for formatting the links into a list!

However, I think we should wait until RHEL 10 GA with this one. WDYT?

I was thinking we probably have to.

@bangelic bangelic force-pushed the bangelic-SAT-28100-Link-to-containers-for-RHEL10 branch from 49dfca9 to 276afe3 Compare January 31, 2025 21:02
@bangelic bangelic force-pushed the bangelic-SAT-28100-Link-to-containers-for-RHEL10 branch from 276afe3 to edb06cc Compare February 27, 2025 19:05
@maximiliankolb
Copy link
Contributor

triage: will merge and cherry-pick today.

Copy link
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

one more minor comment, rest LGTM.

From yesterdays triage: we think that it's OK to merge this to nightly as there are no backports to released Satellite versions. Is that OK for you Brian?

@bangelic bangelic force-pushed the bangelic-SAT-28100-Link-to-containers-for-RHEL10 branch from edb06cc to 831087e Compare March 25, 2025 12:48
@bangelic
Copy link
Contributor Author

one more minor comment, rest LGTM.

From yesterdays triage: we think that it's OK to merge this to nightly as there are no backports to released Satellite versions. Is that OK for you Brian?

I'll need to check and get back to you.

@bangelic
Copy link
Contributor Author

bangelic commented Mar 26, 2025

@maximiliankolb DPMs discussed in the program call to wait before merging.

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