github/open-cluster-management
1
0
Fork 0
mirror of https://github.com/open-cluster-management-io/ocm.git synced 2026-05-17 06:37:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

clear the comments for deleting unmanaged appliedmanifestworks (#174)

Browse Source 
Signed-off-by: Wei Liu <liuweixa@redhat.com>

Signed-off-by: Wei Liu <liuweixa@redhat.com>
This commit is contained in:
Wei Liu
2023-01-06 10:56:02 +08:00
committed by GitHub
parent 5417cc44c9
commit e0a940cab0
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
pkg/spoke/controllers/finalizercontroller/unmanaged_appliedmanifestwork_controller.go
View File

@@ -83,10 +83,9 @@ func (m *UnManagedAppliedWorkController) sync(ctx context.Context, controllerCon
return err
}
// We delete the old applied work until the work of this applied work is applied.
// This will avoid to delete the old applied work prematurely, if an applied work is an
// owner of its applied resource, if we delete the old applied work prematurely, this will
// casue to delete the applied resource.
// We delete the old AppliedManifestWork only when the related ManifestWork is applied with the new
// AppliedManifestWork as the new owner. This can avoid deleting the old AppliedManifestWork prematurely
// before the new AppliedManifestWork takes the ownership of the applied resources.
manifestWork, err := m.manifestWorkLister.Get(appliedManifestWork.Spec.ManifestWorkName)
if errors.IsNotFound(err) {
// work not found, could have been deleted, do nothing.