BUG/MINOR: when ConfigMap is removed, corresponding annotations should be deleted

Mo3m3n · Mo3m3n · commit 3c56527ba705 · 2021-11-25T18:29:17.000+01:00
Since 60e75f6 we dont' use status for annotations, thus when ConfigMap is deleted corresponding annotations should be removed right away.
diff --git a/controller/store/events.go b/controller/store/events.go
@@ -325,6 +325,7 @@ func (k *K8s) EventConfigMap(ns *Namespace, data *ConfigMap) (updateRequired boo
 		logger.Infof("configmap '%s/%s' updated", cm.Namespace, cm.Name)
 	case DELETED:
 		cm.Loaded = false
+		cm.Annotations = map[string]string{}
 		updateRequired = true
 		logger.Debugf("configmap '%s/%s' deleted", cm.Namespace, cm.Name)
 	}

Original file line number	Diff line number	Diff line change
`@@ -325,6 +325,7 @@ func (k K8s) EventConfigMap(ns Namespace, data *ConfigMap) (updateRequired boo`
`325`	`325`	`logger.Infof("configmap '%s/%s' updated", cm.Namespace, cm.Name)`
`326`	`326`	`case DELETED:`
`327`	`327`	`cm.Loaded = false`
	`328`	`+ cm.Annotations = map[string]string{}`
`328`	`329`	`updateRequired = true`
`329`	`330`	`logger.Debugf("configmap '%s/%s' deleted", cm.Namespace, cm.Name)`
`330`	`331`	`}`