From 554d150df992d58342d4857881404d5140d1ff86 Mon Sep 17 00:00:00 2001 From: Sergey Shaykhullin <46970457+sergeyshaykhullin@users.noreply.github.com> Date: Wed, 12 Jan 2022 01:35:29 +0300 Subject: [PATCH] fix(argo-rollouts): Fix service name for ingress rollouts dashboard (#1084) * Fix service name for ingress rollouts dashboard Signed-off-by: Sergey Shaykhullin * Update Chart.yaml Signed-off-by: Sergey Shaykhullin * Fix it centrally Signed-off-by: Marco Kilchhofer Co-authored-by: Marco Kilchhofer --- charts/argo-rollouts/Chart.yaml | 4 ++-- charts/argo-rollouts/templates/dashboard/ingress.yaml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/charts/argo-rollouts/Chart.yaml b/charts/argo-rollouts/Chart.yaml index 653ef314..9fe46cc3 100644 --- a/charts/argo-rollouts/Chart.yaml +++ b/charts/argo-rollouts/Chart.yaml @@ -2,7 +2,7 @@ apiVersion: v2 appVersion: "v1.1.1" description: A Helm chart for Argo Rollouts name: argo-rollouts -version: 2.9.0 +version: 2.9.1 icon: https://argoproj.github.io/argo-rollouts/assets/logo.png home: https://github.com/argoproj/argo-helm maintainers: @@ -11,4 +11,4 @@ maintainers: - name: jessesuen annotations: artifacthub.io/changes: | - - "[Added]: Dashboard ingress support" + - "[Fixed]: Dashboard ingress service name" diff --git a/charts/argo-rollouts/templates/dashboard/ingress.yaml b/charts/argo-rollouts/templates/dashboard/ingress.yaml index 86dde720..0555d1fc 100644 --- a/charts/argo-rollouts/templates/dashboard/ingress.yaml +++ b/charts/argo-rollouts/templates/dashboard/ingress.yaml @@ -1,5 +1,5 @@ {{- if and .Values.dashboard.enabled .Values.dashboard.ingress.enabled -}} -{{- $serviceName := include "argo-rollouts.fullname" . -}} +{{- $serviceName := printf "%s-dashboard" (include "argo-rollouts.fullname" .) -}} {{- $servicePort := .Values.dashboard.service.port -}} {{- $paths := .Values.dashboard.ingress.paths -}} {{- $extraPaths := .Values.dashboard.ingress.extraPaths -}}