adjust log level (#617)

Author: horis233

controllers/operandconfig/operandconfig_controller.go

@@ -138,7 +138,7 @@ func (r *Reconciler) updateStatus(ctx context.Context, instance *operatorv1alpha
 
 		if _, ok := sub.Labels[constant.OpreqLabel]; !ok {
 			// Subscription existing and not managed by OperandRequest controller
-			klog.V(2).Infof("Subscription %s in the namespace %s isn't created by ODLM", sub.Name, sub.Namespace)
+			klog.V(1).Infof("Subscription %s in the namespace %s isn't created by ODLM", sub.Name, sub.Namespace)
 		}
 
 		csv, err := r.GetClusterServiceVersion(ctx, sub)

controllers/operandrequest/operandrequest_controller.go

@@ -226,7 +226,7 @@ func (r *Reconciler) UpdateNamespaceScope(ctx context.Context, namespacedName ty
 	nsScopeKey := types.NamespacedName{Name: constant.NamespaceScopeCrName, Namespace: util.GetOperatorNamespace()}
 	if err := r.Client.Get(ctx, nsScopeKey, nsScope); err != nil {
 		if apierrors.IsNotFound(err) {
-			klog.V(2).Infof("Not found NamespaceScope CR %s, ignore update it.", nsScopeKey.String())
+			klog.Warningf("Not found NamespaceScope CR %s, ignore update it.", nsScopeKey.String())
 			return nil
 		}
 		return err
@@ -282,7 +282,7 @@ func (r *Reconciler) addFinalizer(ctx context.Context, cr *operatorv1alpha1.Oper
 }
 
 func (r *Reconciler) checkFinalizer(ctx context.Context, requestInstance *operatorv1alpha1.OperandRequest) error {
-	klog.V(2).Infof("Deleting OperandRequest %s in the namespace %s", requestInstance.Name, requestInstance.Namespace)
+	klog.V(1).Infof("Deleting OperandRequest %s in the namespace %s", requestInstance.Name, requestInstance.Namespace)
 	existingSub := &olmv1alpha1.SubscriptionList{}
 
 	opts := []client.ListOption{

controllers/operandrequest/reconcile_operand.go

@@ -79,7 +79,7 @@ func (r *Reconciler) reconcileOperand(ctx context.Context, requestInstance *oper
 			sub, err := r.GetSubscription(ctx, operatorName, namespace, opdRegistry.PackageName)
 
 			if apierrors.IsNotFound(err) {
-				klog.V(2).Infof("There is no Subscription %s or %s in the namespace %s", operatorName, opdRegistry.PackageName, namespace)
+				klog.Warningf("There is no Subscription %s or %s in the namespace %s", operatorName, opdRegistry.PackageName, namespace)
 				continue
 			}
 
@@ -558,7 +558,7 @@ func (r *Reconciler) deleteCustomResource(ctx context.Context, unstruct unstruct
 			if err != nil {
 				return errors.Wrapf(err, "failed to delete custom resource -- Kind: %s, NamespacedName: %s/%s", kind, namespace, name)
 			}
-			klog.V(2).Infof("Finish deleting custom resource -- Kind: %s, NamespacedName: %s/%s", kind, namespace, name)
+			klog.V(1).Infof("Finish deleting custom resource -- Kind: %s, NamespacedName: %s/%s", kind, namespace, name)
 		}
 	}
 	return nil

controllers/operandrequest/reconcile_operator.go

@@ -112,10 +112,10 @@ func (r *Reconciler) reconcileOperator(ctx context.Context, requestInstance *ope
 					}
 				} else {
 					// Subscription existing and not managed by OperandRequest controller
-					klog.V(2).Infof("Subscription %s in namespace %s isn't created by ODLM. Ignore update/delete it.", sub.Name, sub.Namespace)
+					klog.V(1).Infof("Subscription %s in namespace %s isn't created by ODLM. Ignore update/delete it.", sub.Name, sub.Namespace)
 				}
 			} else {
-				klog.V(2).Infof("Operator %s not found in the registry %s/%s", operand.Name, registryInstance.Namespace, registryInstance.Name)
+				klog.V(1).Infof("Operator %s not found in the OperandRegistry %s/%s", operand.Name, registryInstance.Namespace, registryInstance.Name)
 				requestInstance.SetNotFoundOperatorFromRegistryCondition(operand.Name, operatorv1alpha1.ResourceTypeSub, corev1.ConditionTrue)
 			}
 		}
@@ -199,7 +199,7 @@ func (r *Reconciler) updateSubscription(ctx context.Context, cr *operatorv1alpha
 func (r *Reconciler) deleteSubscription(ctx context.Context, operandName string, requestInstance *operatorv1alpha1.OperandRequest, registryInstance *operatorv1alpha1.OperandRegistry, configInstance *operatorv1alpha1.OperandConfig) error {
 	op := registryInstance.GetOperator(operandName)
 	if op == nil {
-		klog.V(2).Infof("Operand %s not found", operandName)
+		klog.Warningf("Operand %s not found", operandName)
 		return nil
 	}
 
@@ -230,14 +230,14 @@ func (r *Reconciler) deleteSubscription(ctx context.Context, operandName string,
 		}
 
 		if r.checkUninstallLabel(ctx, op.Name, namespace) {
-			klog.V(2).Infof("Operator %s has label operator.ibm.com/opreq-do-not-uninstall. Skip the uninstall", op.Name)
+			klog.V(1).Infof("Operator %s has label operator.ibm.com/opreq-do-not-uninstall. Skip the uninstall", op.Name)
 			return nil
 		}
 
 		klog.V(3).Info("Set Deleting Condition in the operandRequest")
 		requestInstance.SetDeletingCondition(csv.Name, operatorv1alpha1.ResourceTypeCsv, corev1.ConditionTrue)
 
-		klog.V(2).Infof("Deleting the ClusterServiceVersion, Namespace: %s, Name: %s", csv.Namespace, csv.Name)
+		klog.V(1).Infof("Deleting the ClusterServiceVersion, Namespace: %s, Name: %s", csv.Namespace, csv.Name)
 		if err := r.Delete(ctx, csv); err != nil {
 			requestInstance.SetDeletingCondition(csv.Name, operatorv1alpha1.ResourceTypeCsv, corev1.ConditionFalse)
 			return errors.Wrap(err, "failed to delete the ClusterServiceVersion")
@@ -255,6 +255,8 @@ func (r *Reconciler) deleteSubscription(ctx context.Context, operandName string,
 			return errors.Wrap(err, "failed to delete subscription")
 		}
 	}
+
+	klog.V(1).Infof("Subscription %s/%s is deleted", namespace, op.Name)
 	return nil
 }