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

Include information about N-2 Smart Proxy support #3573

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

Conversation

AkshayGadhaveRH
Copy link
Contributor

Project now supports Smart Proxies of verions N-1 and N-2. Added a new attribute for N-2 Smart Proxy : {ProjectVersionPrevious-Previous}. Added information about N-2 Smart Proxy support for Upgrade guide modules.

What changes are you introducing?

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

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

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.

@AkshayGadhaveRH AkshayGadhaveRH added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective labels Jan 16, 2025
Akshay Gadhave added 2 commits January 17, 2025 11:41
Project	now supports Smart Proxies of verions N-1 and N-2.
Added a	new attribute for N-2 Smart Proxy : {ProjectVersionPrevious-Previous}.
Added information about N-2 Smart Proxy support for Upgrade guide modules.
@AkshayGadhaveRH AkshayGadhaveRH force-pushed the support_n-2_smartproxies branch from 9475b30 to 04d5ba2 Compare January 17, 2025 06:19
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.

diff looks sane to me; needs technical ACK. cc @sbernhard

@maximiliankolb maximiliankolb added style review done No issues from docs style/grammar perspective and removed Needs style review Requires a review from docs style/grammar perspective labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs tech review Requires a review from the technical perspective style review done No issues from docs style/grammar perspective
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants