charts icon indicating copy to clipboard operation
charts copied to clipboard

Enabling ndmExporter results in duplicate "name" keys

Open NxDs opened this issue 1 year ago • 1 comments

With default values simply enabling the exporter

    ndmExporter:
      enabled: false

Results in the following (in openebs-ndm-cluster-exporter deployment):

        chart: openebs-3.5.0
        heritage: Helm
        openebs.io/version: "3.5.0"
        app: openebs-ndm-cluster-exporter
        release: openebs
        component: openebs-ndm-cluster-exporter
        name: openebs-ndm-node-exporter
        openebs.io/component-name: openebs-ndm-cluster-exporter
        name: openebs-ndm-cluster-exporter

which includes the "name" field twice causing issues such as flux helm-controller failing to install the chart

NxDs avatar Mar 30 '23 18:03 NxDs

Encountered this one too. I think it's because in charts/openebs/templates/ndm/cluster-exporter.yaml, the include and with blocks are duplicate:

  template:
    metadata:
      labels:
        {{- include "openebs.ndm-cluster-exporter.labels" . | nindent 8 }}
        {{- with .Values.ndmExporter.clusterExporter.podLabels }}
        {{ toYaml . }}
        {{- end }}

DArtagan avatar Jun 25 '23 04:06 DArtagan