[BUG]fix cache leak in yurtappoverrider controller

[rambohe-ch]

What happened:

1. when a yurtappoverrider instance is deleted, the cache data in CacheOverriderMap should also be removed. so we can add cache cleanup in the following code snippet.

openyurt/pkg/yurtmanager/controller/yurtappoverrider/yurtappoverrider_controller.go

Line 129 in 3636ec0

if errors.IsNotFound(err) {

2. when entires of yurtappoverrider are changed, we should update the new instance into CacheOverriderMap.

openyurt/pkg/yurtmanager/controller/yurtappoverrider/yurtappoverrider_controller.go

Lines 162 to 168 in 3636ec0

	if cacheOverrider, ok := r.CacheOverriderMap[instance.Namespace+"/"+instance.Name]; ok {
	if reflect.DeepEqual(cacheOverrider.Entries, instance.Entries) {
	return reconcile.Result{}, nil
	}
	} else {
	r.CacheOverriderMap[instance.Namespace+"/"+instance.Name] = instance.DeepCopy()
	}

so we can improve the up codes as following:

	if cacheOverrider, ok := r.CacheOverriderMap[instance.Namespace+"/"+instance.Name]; ok {
		if reflect.DeepEqual(cacheOverrider.Entries, instance.Entries) {
			return reconcile.Result{}, nil
		}
	}
	r.CacheOverriderMap[instance.Namespace+"/"+instance.Name] = instance.DeepCopy()

Environment:

• OpenYurt version: v1.4.0
• Kubernetes version (use kubectl version):
• OS (e.g: cat /etc/os-release):
• Kernel (e.g. uname -a):
• Install tools:
• Others:

others

/kind bug

[YTGhost]

I think it can be closed.

[rambohe-ch]

/close

[openyurt-bot]

@rambohe-ch: Closing this issue.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.