Skip to content
This repository has been archived by the owner on May 8, 2024. It is now read-only.

Enabling ndmExporter results in duplicate "name" keys #324

Open
NxDs opened this issue Mar 30, 2023 · 1 comment
Open

Enabling ndmExporter results in duplicate "name" keys #324

NxDs opened this issue Mar 30, 2023 · 1 comment

Comments

@NxDs
Copy link

NxDs commented Mar 30, 2023

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

@DArtagan
Copy link

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 }}

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants