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

Dapr dashboard does not respect the global registry value #14

Open
aytekinar opened this issue Oct 18, 2021 · 0 comments
Open

Dapr dashboard does not respect the global registry value #14

aytekinar opened this issue Oct 18, 2021 · 0 comments

Comments

@aytekinar
Copy link

Current Behavior

I am using version 1.4.3 of dapr/dapr, and I have noticed that the Dapr dashboard char does not respect the global registry value in values.yaml.

Desired Behavior

Similar to, e.g., dapr-sentry, dapr-dashboard should also pick the global.registry value from values.yaml.

Possible Fix

The current deployment template (for dapr-dashboard) reads as below:

      - name: dapr-dashboard
        image: "{{ .Values.image.registry }}/{{ .Values.image.name }}:{{ .Values.image.tag }}"
        imagePullPolicy: {{ .Values.global.imagePullPolicy }}

Perhaps, it should read instead (cheating from dapr-sentry's deployment template):

{{- if contains "/" .Values.image.name }}
        image: "{{ .Values.image.name }}"
{{- else }}
        image: "{{ .Values.global.registry }}/{{ .Values.image.name }}:{{ .Values.image.tag }}"
{{- end }}

Cc: @baboune

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

No branches or pull requests

1 participant