Change clusterCredentials type to map

[nbarrientos]

Is your feature request related to a problem?

Helm does not support merging arrays so it's not possible to have for instance cluster declarations in separate "values" files (in view of encrypting them with different keys).

configs:
  clusterCredentials:
    - name: my-cluster
      server: https://example.com:6443
      config:
        bearerToken: "foo"
        tlsClientConfig:
          insecure: false
          caData: "bar"

Related helm chart

argo-cd

Describe the solution you'd like

Declare clusters with the same syntax as configs.repositories:

configs:
  clusterCredentials:
    my-cluster:
      server: https://example.com:6443
      config:
        bearerToken: "foo"
        tlsClientConfig:
          insecure: false
          caData: "bar"

as each cluster is represented by a key in the map configs.clusterCredentials it'd be possible for Helm to merge multiple clusters declared in separate values files.

Describe alternatives you've considered

No response

Additional context

No response

[nbarrientos]

Any chance before we fork locally? 🙏