silence member config logs when there's no error

Member-operator was spamming this message in its logs, but there was no `err` field set. Check that an error actually exists before logging it. Signed-off-by: Andy Sadler <[email protected]>
codeready-toolchain · Aug 1, 2024 · 4aa1235 · 4aa1235
1 parent c538d1b
commit 4aa1235
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/cmd/webhook/main.go b/cmd/webhook/main.go
@@ -148,8 +148,9 @@ func main() {
 func reloadMemberConfig(cl client.Client) context.Context {
 	ctx := ctrl.SetupSignalHandler()
 	go wait.Until(func() {
-		_, _, err := configuration.LoadLatest(cl, &toolchainv1alpha1.MemberOperatorConfig{})
-		setupLog.Error(err, "unable to load latest member config")
+		if _, _, err := configuration.LoadLatest(cl, &toolchainv1alpha1.MemberOperatorConfig{}); err != nil {
+			setupLog.Error(err, "unable to load latest member config")
+		}
 	}, 5*time.Second, ctx.Done())
 	return ctx
 }