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

Remove SCA prerequisite #3570

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

Conversation

AkshayGadhaveRH
Copy link
Contributor

Migrating to SCA is a one-time thing and has been a part of the earlier version. Removing it as it is not required in later versions.

JIRA:
https://issues.redhat.com/browse/SAT-27580
Earlier PR for reference:
#3228

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.

Copy link

github-actions bot commented Jan 15, 2025

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.

There's no option for SCA in Foreman/Katello nightly anymore. LGTM. To my understanding, this could/should get cherry-picked to 3.10+: https://docs.theforeman.org/3.10/Release_Notes/index-katello.html#_entitlement_mode_removed_in_favor_of_simple_content_access

Needs technical ACK on the cherry-picking please.

@maximiliankolb maximiliankolb added style review done No issues from docs style/grammar perspective Needs tech review Requires a review from the technical perspective labels Jan 15, 2025
Akshay Gadhave added 2 commits January 16, 2025 11:02
Migrating to SCA is a one-time thing and has been a part of the earlier version.
Removing it as it is not required in later versions.

JIRA:
https://issues.redhat.com/browse/SAT-27580
Earlier PR for reference:
theforeman#3228
@AkshayGadhaveRH AkshayGadhaveRH force-pushed the remove_SCA_prerequisites branch from c4599d7 to a121de6 Compare January 16, 2025 05:35
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.

3 participants