Fixed a bug in the nifi-cluster helm chart where the cluster.logbackConfig.replaceConfigMap couldn't be set - #452

Thanks for the info, I didn't realise you could set both!

I think the helm-chart is still incorrectly applying a default logback in it's current configuration though. If I use the following helm values:

cluster:
  logbackConfig:
    replaceConfigMap:
      data: logback.xml
      name: cluster-config
      namespace: nifi
    replaceSecretConfig: {}

because cluster.logbackConfig.replaceSecretConfig is empty, the chart will apply the default template provided by the helm chart: https://github.com/konpyutaika/nifikop/blob/master/helm/nifi-cluster/templates/nifi-cluster.yaml#L91-L102, meaning it'll ignore the one I provided in my configmap?

Thanks for the info, I didn't realise you could set both!

I think the helm-chart is still incorrectly applying a default logback in it's current configuration though. If I use the following helm values:

cluster:
  logbackConfig:
    replaceConfigMap:
      data: logback.xml
      name: cluster-config
      namespace: nifi
    replaceSecretConfig: {}

because cluster.logbackConfig.replaceSecretConfig is empty, the chart will apply the default template provided by the helm chart: https://github.com/konpyutaika/nifikop/blob/master/helm/nifi-cluster/templates/nifi-cluster.yaml#L91-L102, meaning it'll ignore the one I provided in my configmap?

There is no "default" logback configuration.

    logbackConfig:
    {{- if .Values.cluster.logbackConfig.replaceSecretConfig }}
      replaceSecretConfig:
        data: {{ .Values.cluster.logbackConfig.replaceSecretConfig.data }}
        name: {{ .Values.cluster.logbackConfig.replaceSecretConfig.name }}
        namespace: {{ .Values.cluster.logbackConfig.replaceSecretConfig.namespace }}
    {{- else }}
      # the default
      replaceSecretConfig:
        data: logback.xml
        name: {{ include "nifi-cluster.fullname" . }}
        namespace: {{ .Release.Namespace }}
    {{- end }}
    {{- if .Values.cluster.logbackConfig.replaceConfigMap }}
      replaceConfigMap:
        data: {{ .Values.cluster.logbackConfig.replaceConfigMap.data }}
        name: {{ .Values.cluster.logbackConfig.replaceConfigMap.name }}
        namespace: {{ .Values.cluster.logbackConfig.replaceConfigMap.namespace }}
    {{- end }}

If you don't set replaceSecretConfig or replaceConfigMap. It will just fallback on the default Secret created here but there isn't any logback.xml in it. So NiFiKOp will use its default template present here without any addition.

Thanks for the info, I didn't realise you could set both!
I think the helm-chart is still incorrectly applying a default logback in it's current configuration though. If I use the following helm values:

cluster:
  logbackConfig:
    replaceConfigMap:
      data: logback.xml
      name: cluster-config
      namespace: nifi
    replaceSecretConfig: {}

because cluster.logbackConfig.replaceSecretConfig is empty, the chart will apply the default template provided by the helm chart: https://github.com/konpyutaika/nifikop/blob/master/helm/nifi-cluster/templates/nifi-cluster.yaml#L91-L102, meaning it'll ignore the one I provided in my configmap?

There is no "default" logback configuration.

    logbackConfig:
    {{- if .Values.cluster.logbackConfig.replaceSecretConfig }}
      replaceSecretConfig:
        data: {{ .Values.cluster.logbackConfig.replaceSecretConfig.data }}
        name: {{ .Values.cluster.logbackConfig.replaceSecretConfig.name }}
        namespace: {{ .Values.cluster.logbackConfig.replaceSecretConfig.namespace }}
    {{- else }}
      # the default
      replaceSecretConfig:
        data: logback.xml
        name: {{ include "nifi-cluster.fullname" . }}
        namespace: {{ .Release.Namespace }}
    {{- end }}
    {{- if .Values.cluster.logbackConfig.replaceConfigMap }}
      replaceConfigMap:
        data: {{ .Values.cluster.logbackConfig.replaceConfigMap.data }}
        name: {{ .Values.cluster.logbackConfig.replaceConfigMap.name }}
        namespace: {{ .Values.cluster.logbackConfig.replaceConfigMap.namespace }}
    {{- end }}

If you don't set replaceSecretConfig or replaceConfigMap. It will just fallback on the default Secret created here but there isn't any logback.xml in it. So NiFiKOp will use its default template present here without any addition.

The nifi-cluster helm chart will use this configuration if there's not one provided as an override: https://github.com/konpyutaika/nifikop/blob/master/helm/nifi-cluster/config/logback.xml

But there's actually a bug here where it's not possible to override the logback.xml using a ConfigMap due to the logic here: https://github.com/konpyutaika/nifikop/blob/master/helm/nifi-cluster/templates/nifi-cluster.yaml#L105-L122

So that logic needs updating and perhaps the helm chart should not even provide a default and let the operator generate it from the template you've linked. But because there's a default in the helm chart, the operator's template won't be used as currently written.