-
Notifications
You must be signed in to change notification settings - Fork 22
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 global hub finalizer from local resources and managedcluster. #365
remove global hub finalizer from local resources and managedcluster. #365
Conversation
/retest |
/test sonarcloud |
c9fd3cf
to
33527bd
Compare
Signed-off-by: myan <myan@redhat.com>
Signed-off-by: myan <myan@redhat.com>
Signed-off-by: myan <myan@redhat.com>
Signed-off-by: myan <myan@redhat.com>
Signed-off-by: myan <myan@redhat.com>
ff85373
to
356d677
Compare
Kudos, SonarCloud Quality Gate passed! |
/cc @clyang82 @morvencao |
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.
LGTM
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: clyang82, yanmxa The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
refs: https://issues.redhat.com/secure/RapidBoard.jspa?rapidView=16808&quickFilter=105908
finalizer
to delete obj from the bundles before, and now we have to depend on thenot found error
to remove the bundle obj so that can be deleted from the database.UID
for the deleted obj, I have to add a newNamespacedname
to the bundle object for retrieval.refs: https://issues.redhat.com/browse/ACM-4597
fixed: Remove the global hub finalizer from managed cluster when deleting the MGH CR #320