fix: log template rendreing errors (#433)

Signed-off-by: Luis Davim <[email protected]>

Author: luisdavim

charts/log-router/Chart.yaml

@@ -4,7 +4,7 @@
 apiVersion: v1
 description: Distribution of Fluentd as K8S daemonset
 name: log-router
-version: 0.4.0
+version: 0.4.1
 home: https://github.com/vmware/kube-fluentd-operator
 sources:
   - https://github.com/vmware/kube-fluentd-operator

charts/log-router/templates/clusterrole.yaml

@@ -17,6 +17,9 @@ metadata:
     {{- end }}
   name: {{ template "fluentd-router.fullname" . }}
 rules:
+{{- with .Values.extraRBAC }}
+  {{- toYaml . | nindent 2 }}
+{{- end }}
   - apiGroups: [""]
     resources:
       - configmaps

charts/log-router/values.yaml

@@ -14,6 +14,11 @@ rbac:
 
 serviceAccountName: "default"
 
+extraRBAC: {}
+  # - apiGroups: [""]
+  #   resources: ["secrets"]
+  #   verbs: ["get", "list", "watch"]
+
 # Possible values: default|fake|fs|multimap|crd
 datasource: default
 

config-reloader/datasource/kube_informer.go

@@ -173,9 +173,10 @@ func (d *kubeInformerConnection) GetNamespaces(ctx context.Context) ([]*Namespac
 		buf := new(strings.Builder)
 		if err := template.Render(buf, configdata, map[string]string{
 			"Namespace": ns,
-		}); err == nil {
-			configdata = buf.String()
+		}); err != nil {
+			logrus.Errorf("failed to render config in namespace: %v", ns)
 		}
+		configdata = buf.String()
 		if configdata == "" {
 			logrus.Infof("Skipping namespace: %v because is empty", ns)
 			continue
@@ -375,8 +376,9 @@ func (d *kubeInformerConnection) handlePodChange(ctx context.Context, obj interf
 	if err := template.Render(buf, configdata, map[string]string{
 		"Namespace": mObj.GetNamespace(),
 	}); err == nil {
-		configdata = buf.String()
+		logrus.Errorf("failed to render config in namespace: %v", mObj.GetNamespace())
 	}
+	configdata = buf.String()
 	nsConfigStr := fmt.Sprintf("%#v", configdata)
 
 	if err == nil {