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

Fixes #37230 - Remove SCA hammer references #930

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

chris1984
Copy link
Member

@chris1984 chris1984 commented Mar 6, 2024

  • What are the changes introduced in this pull request?

    • Removing the SCA command in hammer to disable/enable/check SCA status
    • Removing SCA references out of organization commands
    • Removed SCA command tests
    • Updated organization tests to remove SCA unit test
  • Considerations taken when implementing this change?

    • To make sure nothing breaks
  • What are the testing steps for this pull request?

    • Make sure you have the latest Katello master changes pulled in with the API updates around SCA
    • Check out PR
    • Try to view info on an org and make sure you don't see any SCA items
    • Verify the simple-content-access sub command is missing

@chris1984
Copy link
Member Author

@jeremylenz here is the hammer one

Copy link
Member

@ianballou ianballou left a comment

Choose a reason for hiding this comment

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

Works for me!

@chris1984 chris1984 merged commit 269635c into Katello:master Mar 8, 2024
7 checks passed
@chris1984 chris1984 deleted the remove-sca branch March 8, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants