feat: add eg.fullname support for resources names (#8156)

Signed-off-by: Dmitrii Golubtsov <evarome10@gmail.com>

Author: Miracle05

charts/gateway-helm/README.md

@@ -33,7 +33,7 @@ Please refer to Helm's [documentation](https://helm.sh/docs) to get started.
 Once Helm has been set up correctly, install the chart from dockerhub:
 
 ``` shell
-helm install eg oci://docker.io/envoyproxy/gateway-helm --version v0.0.0-latest -n envoy-gateway-system --create-namespace
+helm install envoy-gateway oci://docker.io/envoyproxy/gateway-helm --version v0.0.0-latest -n envoy-gateway-system --create-namespace
 ```
 You can find all helm chart release in [Dockerhub](https://hub.docker.com/r/envoyproxy/gateway-helm/tags)
 
@@ -52,7 +52,7 @@ make kube-deploy TAG=latest
 You can install the eg chart along without Gateway API CRDs and Envoy Gateway CRDs, make sure CRDs exist in Cluster first if you want to skip to install them, otherwise EG may fail to start:
 
 ``` shell
-helm install eg --create-namespace oci://docker.io/envoyproxy/gateway-helm --version v0.0.0-latest -n envoy-gateway-system --skip-crds
+helm install envoy-gateway --create-namespace oci://docker.io/envoyproxy/gateway-helm --version v0.0.0-latest -n envoy-gateway-system --skip-crds
 ```
 
 To uninstall the chart:

charts/gateway-helm/templates/_helpers.tpl

@@ -9,7 +9,7 @@ Allow the release namespace to be overridden.
 Expand the name of the chart.
 */}}
 {{- define "eg.name" -}}
-{{- default .Chart.Name .Values.nameOverride | trunc 63 | trimSuffix "-" }}
+{{- default .Values.nameOverride "envoy-gateway" | trunc 63 | trimSuffix "-" }}
 {{- end }}
 
 {{/*
@@ -21,7 +21,7 @@ If release name contains chart name it will be used as a full name.
 {{- if .Values.fullnameOverride }}
 {{- .Values.fullnameOverride | trunc 63 | trimSuffix "-" }}
 {{- else }}
-{{- $name := default .Chart.Name .Values.nameOverride }}
+{{- $name := default .Values.nameOverride "envoy-gateway" }}
 {{- if contains $name .Release.Name }}
 {{- .Release.Name | trunc 63 | trimSuffix "-" }}
 {{- else }}

charts/gateway-helm/templates/certgen-rbac.yaml

@@ -96,7 +96,7 @@ rules:
     resources:
       - mutatingwebhookconfigurations
     resourceNames:
-      - 'envoy-gateway-topology-injector.{{ include "eg.namespace" . }}'
+      - '{{ include "eg.fullname" . }}-topology-injector.{{ include "eg.namespace" . }}'
     verbs:
       - update
       - patch

charts/gateway-helm/templates/certgen.yaml

@@ -52,7 +52,7 @@ spec:
           value: {{ .Values.kubernetesClusterDomain }}
         image: {{ include "eg.image" . }}
         imagePullPolicy: {{ include "eg.image.pullPolicy" . }}
-        name: envoy-gateway-certgen
+        name: {{ include "eg.fullname" . }}-certgen
         {{- with .Values.certgen.job.resources }}
         resources:
           {{- toYaml . | nindent 10 }}

charts/gateway-helm/templates/envoy-gateway-config.yaml

@@ -1,7 +1,7 @@
 apiVersion: v1
 kind: ConfigMap
 metadata:
-  name: envoy-gateway-config
+  name: {{ include "eg.fullname" . }}-config
   namespace: {{ include "eg.namespace" . }}
   labels:
   {{- include "eg.labels" . | nindent 4 }}

charts/gateway-helm/templates/envoy-gateway-deployment.yaml

@@ -1,22 +1,22 @@
 apiVersion: apps/v1
 kind: Deployment
 metadata:
-  name: envoy-gateway
+  name: {{ include "eg.fullname" . }}
   namespace: {{ include "eg.namespace" . }}
   {{- with .Values.deployment.annotations }}
   annotations:
     {{- toYaml . | nindent 4 }}
   {{- end }}
   labels:
-    control-plane: envoy-gateway
+    control-plane: {{ include "eg.fullname" . }}
   {{- include "eg.labels" . | nindent 4 }}
 spec:
 {{- if not .Values.hpa.enabled }}
   replicas: {{ .Values.deployment.replicas }}
 {{- end }}
   selector:
     matchLabels:
-      control-plane: envoy-gateway
+      control-plane: {{ include "eg.fullname" . }}
     {{- include "eg.selectorLabels" . | nindent 6 }}
   template:
     metadata:
@@ -25,7 +25,7 @@ spec:
         {{- toYaml . | nindent 8 }}
       {{- end }}
       labels:
-        control-plane: envoy-gateway
+        control-plane: {{ include "eg.fullname" . }}
       {{- include "eg.selectorLabels" . | nindent 8 }}
       {{- with .Values.deployment.pod.labels }}
         {{- toYaml . | nindent 8 }}
@@ -67,7 +67,7 @@ spec:
             port: 8081
           initialDelaySeconds: 15
           periodSeconds: 20
-        name: envoy-gateway
+        name: {{ include "eg.fullname" . }}
         ports:
         {{- range .Values.deployment.ports }}
         - containerPort: {{ .port }}
@@ -98,13 +98,13 @@ spec:
       {{- with .Values.deployment.priorityClassName }}
       priorityClassName: {{ . | quote }}
       {{- end }}
-      serviceAccountName: envoy-gateway
+      serviceAccountName: {{ include "eg.fullname" . }}
       terminationGracePeriodSeconds: 10
       volumes:
       - configMap:
           defaultMode: 420
-          name: envoy-gateway-config
+          name: {{ include "eg.fullname" . }}-config
         name: envoy-gateway-config
       - name: certs
         secret:
-          secretName: envoy-gateway
+          secretName: {{ include "eg.fullname" . }}

charts/gateway-helm/templates/envoy-gateway-hpa.yaml

@@ -2,13 +2,13 @@
 apiVersion: autoscaling/v2
 kind: HorizontalPodAutoscaler
 metadata:
-  name: envoy-gateway
+  name: {{ include "eg.fullname" . }}
   namespace: {{ include "eg.namespace" $ }}
 spec:
   scaleTargetRef:
     apiVersion: apps/v1
     kind: Deployment
-    name: envoy-gateway
+    name: {{ include "eg.fullname" . }}
   {{- if .Values.hpa.minReplicas }}
   minReplicas: {{ .Values.hpa.minReplicas }}
   {{- end }}

charts/gateway-helm/templates/envoy-gateway-poddisruptionbudget.yaml

@@ -2,7 +2,7 @@
 apiVersion: policy/v1
 kind: PodDisruptionBudget
 metadata:
-  name: envoy-gateway
+  name: {{ include "eg.fullname" . }}
   namespace: {{ include "eg.namespace" . }}
 spec:
   {{- if and .Values.podDisruptionBudget.minAvailable }}
@@ -13,6 +13,6 @@ spec:
   {{- end }}
   selector:
     matchLabels:
-      control-plane: envoy-gateway
+      control-plane: {{ include "eg.fullname" . }}
     {{- include "eg.selectorLabels" . | nindent 6 }}
 {{- end }}

charts/gateway-helm/templates/envoy-gateway-rbac.yaml

@@ -32,7 +32,7 @@ roleRef:
   name: {{ include "eg.fullname" $ }}-envoy-gateway-role
 subjects:
 - kind: ServiceAccount
-  name: 'envoy-gateway'
+  name: {{ include "eg.fullname" $ }}
   namespace: {{ include "eg.namespace" $ }}
 {{ end }}
 ---
@@ -54,7 +54,7 @@ roleRef:
   name: {{ include "eg.fullname" . }}-envoy-gateway-role
 subjects:
 - kind: ServiceAccount
-  name: 'envoy-gateway'
+  name: '{{ include "eg.fullname" . }}'
   namespace: {{ include "eg.namespace" . }}
 {{ else }}
 ---
@@ -77,6 +77,6 @@ roleRef:
   name: {{ include "eg.fullname" . }}-envoy-gateway-role
 subjects:
 - kind: ServiceAccount
-  name: 'envoy-gateway'
+  name: '{{ include "eg.fullname" . }}'
   namespace: {{ include "eg.namespace" . }}
 {{ end }}

charts/gateway-helm/templates/envoy-gateway-service.yaml

@@ -1,22 +1,22 @@
 apiVersion: v1
 kind: Service
 metadata:
-  name: envoy-gateway
+  name: {{ include "eg.fullname" . }}
   namespace: {{ include "eg.namespace" . }}
   {{- with .Values.service.annotations }}
   annotations:
     {{- toYaml . | nindent 4 }}
   {{- end }}
   labels:
-    control-plane: envoy-gateway
+    control-plane: {{ include "eg.fullname" . }}
   {{- include "eg.labels" . | nindent 4 }}
 spec:
   type: {{ .Values.service.type }}
   {{- if and (eq .Values.service.type "LoadBalancer") .Values.service.loadBalancerIP }}
   loadBalancerIP: {{ .Values.service.loadBalancerIP }}
   {{- end }}
   selector:
-    control-plane: envoy-gateway
+    control-plane: {{ include "eg.fullname" . }}
   {{- include "eg.selectorLabels" . | nindent 4 }}
   ports:
   {{- .Values.deployment.ports | toYaml | nindent 2 -}}