-
Couldn't load subscription status.
- Fork 3
fix: rename rhtas deployment name #129
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
base: main
Are you sure you want to change the base?
Conversation
Change RHTAS operator deployment name to rhtas-controller-manager which has been caused because of cahnges in immutable sections which causes OLM upgrade to fail. Refs: securesign/secure-sign-operator#1110
Reviewer's GuideRename references to the RHTAS operator deployment from "rhtas-operator-controller-manager" to "rhtas-controller-manager" in installation task scripts to ensure OLM upgrades succeed. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @osmman - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Change RHTAS operator deployment name to rhtas-controller-manager which has been caused because of cahnges in immutable sections which causes OLM upgrade to fail.
Refs: securesign/secure-sign-operator#1110
Summary by Sourcery
Rename RHTAS operator deployment references in installation tasks to 'rhtas-controller-manager' to fix OLM upgrade failures.
Bug Fixes: