Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect validation of CLUSTERNAME.name in clusterCredentials #3110

Open
atgane opened this issue Jan 7, 2025 · 0 comments
Open

Incorrect validation of CLUSTERNAME.name in clusterCredentials #3110

atgane opened this issue Jan 7, 2025 · 0 comments
Labels
argo-cd enhancement New feature or request

Comments

@atgane
Copy link

atgane commented Jan 7, 2025

Is your feature request related to a problem?

name: {{ required "A valid .Values.configs.clusterCredentials.CLUSTERNAME.name entry is required!" $cluster_key }}

There is something wrong with the way the required function is used in that line. It looks like what the function means is to check CLUSTERNAME.name in clusterCredentials, but this value always contains $cluster_key, so it doesn't throw an error in required.

Related helm chart

argo-cd

Describe the solution you'd like

I would suggest using $cluster_key as the default value and writing $cluster_value.name without validation.

like this:

  name: {{ default $cluster_key $cluster_value.name }}

Describe alternatives you've considered

No response

Additional context

No response

@atgane atgane added the enhancement New feature or request label Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
argo-cd enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants