change error log level to warn

rancher · Feb 1, 2025 · 594b408 · 594b408
1 parent adf60a0
commit 594b408
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/internal/helm-locker/controllers/release/controller.go b/internal/helm-locker/controllers/release/controller.go
@@ -101,7 +101,7 @@ func (h *handler) OnObjectSetChange(setID string, obj runtime.Object) (runtime.O
 	helmReleases, err := h.helmReleaseCache.GetByIndex(HelmReleaseByReleaseKey, setID)
 	if err != nil {
 		err = fmt.Errorf("unable to find HelmReleases for objectset %s to trigger event", setID)
-		logrus.Errorf("%v", err)
+		logrus.Warnf("%v", err)
 		return nil, err
 	}
 	for _, helmRelease := range helmReleases {

diff --git a/internal/helm-project-operator/controllers/project/controller.go b/internal/helm-project-operator/controllers/project/controller.go
@@ -213,7 +213,7 @@ func (h *handler) OnChange(projectHelmChart *v1alpha2.ProjectHelmChart, projectH
 	projectHelmCharts, err := h.projectHelmChartCache.GetByIndex(ProjectHelmChartByReleaseName, releaseName)
 	if err != nil {
 		err = fmt.Errorf("unable to get ProjectHelmCharts to verify if release is already tracked: %s", err)
-		logrus.Errorf("%v", err)
+		logrus.Warnf("%v", err)
 		return nil, projectHelmChartStatus, err
 	}
 	for _, conflictingProjectHelmChart := range projectHelmCharts {
@@ -239,7 +239,7 @@ func (h *handler) OnChange(projectHelmChart *v1alpha2.ProjectHelmChart, projectH
 			conflictingProjectHelmChart.Name, conflictingProjectHelmChart.Namespace,
 			releaseName, releaseNamespace,
 		)
-		logrus.Errorf("%v", err)
+		logrus.Warnf("%v", err)
 		projectHelmChartStatus = h.getUnableToCreateHelmReleaseStatus(projectHelmChart, projectHelmChartStatus, err)
 		return nil, projectHelmChartStatus, nil
 	}
@@ -253,7 +253,7 @@ func (h *handler) OnChange(projectHelmChart *v1alpha2.ProjectHelmChart, projectH
 	targetProjectNamespaces, err := h.projectGetter.GetTargetProjectNamespaces(projectHelmChart)
 	if err != nil {
 		err = fmt.Errorf("Unable to find target project namespaces to deploy ProjectHelmChart %s/%s: %s", projectHelmChart.Name, projectHelmChart.Namespace, err)
-		logrus.Errorf("%v", err)
+		logrus.Warnf("%v", err)
 		return nil, projectHelmChartStatus, err
 	}
 	if len(targetProjectNamespaces) == 0 {
@@ -281,7 +281,7 @@ func (h *handler) OnChange(projectHelmChart *v1alpha2.ProjectHelmChart, projectH
 	valuesContentBytes, err := values.ToYAML()
 	if err != nil {
 		err = fmt.Errorf("unable to marshall spec.values: %s", err)
-		logrus.Errorf("%v", err)
+		logrus.Warnf("%v", err)
 		projectHelmChartStatus = h.getValuesParseErrorStatus(projectHelmChart, projectHelmChartStatus, err)
 		return nil, projectHelmChartStatus, nil
 	}
@@ -297,7 +297,7 @@ func (h *handler) OnChange(projectHelmChart *v1alpha2.ProjectHelmChart, projectH
 		// the newly created namespace. Without this, a deleted release namespace will always have ProjectHelmCharts stuck in
 		// WaitingForDashboardValues since the underlying helm release will never be recreated
 		err = fmt.Errorf("cannot find release namespace %s to deploy release", releaseNamespace)
-		logrus.Errorf("%v", err)
+		logrus.Warnf("%v", err)
 		projectHelmChartStatus = h.getUnableToCreateHelmReleaseStatus(projectHelmChart, projectHelmChartStatus, err)
 		return objs, projectHelmChartStatus, nil
 	} else if err != nil {
@@ -329,7 +329,7 @@ func (h *handler) OnChange(projectHelmChart *v1alpha2.ProjectHelmChart, projectH
 	dashboardValues, err := h.getDashboardValuesFromConfigmaps(projectHelmChart)
 	if err != nil {
 		err = fmt.Errorf("unable to get dashboard values from status ConfigMaps: %s", err)
-		logrus.Errorf("%v", err)
+		logrus.Warnf("%v", err)
 		return nil, projectHelmChartStatus, err
 	}
 	if len(dashboardValues) == 0 {