Fix wrong conditional check (#451)

jeesmon · web-flow · commit 7a2ac7304ec9 · 2024-12-05T08:53:14.000Z
Condition should be `!=` instead of `==` as logging is needed when `template.Render` fails.

Signed-off-by: Jeesmon Jacob &lt;jjeesmon@vmware.com&gt;
diff --git a/config-reloader/datasource/kube_informer.go b/config-reloader/datasource/kube_informer.go
@@ -375,7 +375,7 @@ func (d *kubeInformerConnection) handlePodChange(ctx context.Context, obj interf
 	buf := new(strings.Builder)
 	if err := template.Render(buf, configdata, map[string]string{
 		"Namespace": mObj.GetNamespace(),
-	}); err == nil {
+	}); err != nil {
 		logrus.Errorf("failed to render config in namespace: %v", mObj.GetNamespace())
 	}
 	configdata = buf.String()

Original file line number	Diff line number	Diff line change
`@@ -375,7 +375,7 @@ func (d *kubeInformerConnection) handlePodChange(ctx context.Context, obj interf`
`375`	`375`	`buf := new(strings.Builder)`
`376`	`376`	`if err := template.Render(buf, configdata, map[string]string{`
`377`	`377`	`"Namespace": mObj.GetNamespace(),`
`378`		`- }); err == nil {`
	`378`	`+ }); err != nil {`
`379`	`379`	`logrus.Errorf("failed to render config in namespace: %v", mObj.GetNamespace())`
`380`	`380`	`}`
`381`	`381`	`configdata = buf.String()`