Improve logging levels for config map creation and updates

Author: chenzhiguo

pkg/mutation/mutation_deploy.go

@@ -58,7 +58,7 @@ func injectionDeploy(request *admissionv1.AdmissionRequest) (*admissionv1.Admiss
 		err := jsoniter.Unmarshal(request.Object.Raw, &deploy)
 		if err != nil {
 			errMsg := fmt.Sprintf("[mutation] /injection-deploy: failed to unmarshal object: %v", err)
-			log.Error(errMsg)
+			log.Errorf(errMsg)
 			return &admissionv1.AdmissionResponse{
 				Allowed: false,
 				Result: &metav1.Status{
@@ -67,6 +67,7 @@ func injectionDeploy(request *admissionv1.AdmissionRequest) (*admissionv1.Admiss
 				},
 			}, nil
 		}
+		log.Infof("[mutation] /injection-deploy: create config map for this deployment: %s, namespace: %s", deploy.Name, deploy.Namespace)
 		err = createConfigMap(&deploy)
 		if err != nil {
 			errMsg := fmt.Sprintf("[mutation] /injection-deploy: failed to create configmap: %v", err)
@@ -112,7 +113,6 @@ func createConfigMap(deploy *appsv1.Deployment) error {
 		},
 		Data: configMapData,
 	}
-	//logger.Debugf("create configmap: %v", configMap)
 	err := rs.CreateOrUpdateConfigMap(context.Background(), deploy.Namespace, configMap)
 	if err != nil {
 		log.Errorf("create configmap %s in %s error: %v", deploy.Name, deploy.Namespace, err)

pkg/resource/resource.go

@@ -73,18 +73,18 @@ func GetResource() *Resource {
 
 func (r *Resource) CreateOrUpdateConfigMap(ctx context.Context, namespace string, configMap *corev1.ConfigMap) error {
 	cm, err := r.ClientSet.CoreV1().ConfigMaps(namespace).Get(ctx, configMap.Name, metav1.GetOptions{})
-	if !errors2.IsNotFound(err) {
+	if err != nil && !errors2.IsNotFound(err) {
 		return err
 	}
 	if cm == nil || errors2.IsNotFound(err) {
 		// create
-		log.Info("create configMap", zap.String("name", configMap.Name), zap.String("namespace", namespace))
+		log.Debug("create configMap", zap.String("name", configMap.Name), zap.String("namespace", namespace))
 		_, err = r.ClientSet.CoreV1().ConfigMaps(namespace).Create(context.TODO(), configMap, metav1.CreateOptions{})
 		if err != nil {
 			return err
 		}
 	} else {
-		log.Info("update configMap", zap.String("name", configMap.Name), zap.String("namespace", namespace), zap.Any("data", configMap.Data))
+		log.Debug("update configMap", zap.String("name", configMap.Name), zap.String("namespace", namespace), zap.Any("data", configMap.Data))
 		cm.Data = configMap.Data
 		_, err = r.ClientSet.CoreV1().ConfigMaps(namespace).Update(context.TODO(), cm, metav1.UpdateOptions{})
 		if err != nil {