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

Recommendation for connected and disconnected user #3021

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

Conversation

bangelic
Copy link
Contributor

After the Prerequisistes in the Importing a Red Hat subscription
manifest into Satellite Server section, we need to recommend users to
create a manifest from access.redhat.com or via
console.redhat.com.

Please cherry-pick my commits into:

  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (Satellite 6.15; orcharhino 6.8)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • Foreman 3.6/Katello 4.8
  • Foreman 3.5/Katello 4.7 (Satellite 6.13; orcharhino 6.6/6.7)
  • Foreman 3.4/Katello 4.6 (EL8 only)
  • Foreman 3.3/Katello 4.5 on EL7 & EL8 (Satellite 6.12 on EL8 only; orcharhino 6.4/6.5 on EL8 only)
  • We do not accept PRs for Foreman older than 3.3.

Copy link
Contributor

@AkshayGadhaveRH AkshayGadhaveRH left a comment

Choose a reason for hiding this comment

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

Maybe we can also add a link to the procedure to import manifest in the UI?

@Lennonka Lennonka added Waiting on contributor Requires an action from the author and removed Not yet reviewed labels May 15, 2024
After the Prerequisistes in the Importing a Red Hat
subscription
manifest into Satellite Server section, we need to
recommend users to
create a manifest from access.redhat.com or via
console.redhat.com.
@bangelic bangelic force-pushed the bangelic-SAT-24859-Need-clarification-whether-manifest-should-be-created-from-Customer-Portal-or-consoleredhatdotcom branch from b873416 to c6f8776 Compare May 17, 2024 17:27
@pr-processor pr-processor bot added Needs re-review and removed Waiting on contributor Requires an action from the author labels May 17, 2024
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.

@bangelic Let's try again. Hint:
Above your edit, there are Prerequisites.
The first prerequisite talks about subscription manifests.
Do you think you could utilize the prerequisite somehow?

@pr-processor pr-processor bot added Waiting on contributor Requires an action from the author and removed Needs re-review labels May 20, 2024
@bangelic
Copy link
Contributor Author

@bangelic Let's try again. Hint: Above your edit, there are Prerequisites. The first prerequisite talks about subscription manifests. Do you think you could utilize the prerequisite somehow?

@Lennonka Are you referring to the link in the first prerequisite?

@Lennonka
Copy link
Contributor

Lennonka commented May 22, 2024

@bangelic Not only the link, but the whole first prerequisite. We could do here something similar to #3010, except that in here we can use a condition for connected/disconnected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Waiting on contributor Requires an action from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants