feat: add option to use standard helm naming conventions

Author: alexstojda

charts/pdp/templates/_helpers.tpl

@@ -1,3 +1,28 @@
+{{/*
+Expand the name of the chart.
+*/}}
+{{- define "..name" -}}
+{{- default .Chart.Name .Values.nameOverride | trunc 63 | trimSuffix "-" }}
+{{- end }}
+
+{{/*
+Create a default fully qualified app name.
+We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec).
+If release name contains chart name it will be used as a full name.
+*/}}
+{{- define "..fullname" -}}
+{{- if .Values.fullnameOverride }}
+{{- .Values.fullnameOverride | trunc 63 | trimSuffix "-" }}
+{{- else }}
+{{- $name := default .Chart.Name .Values.nameOverride }}
+{{- if contains $name .Release.Name }}
+{{- .Release.Name | trunc 63 | trimSuffix "-" }}
+{{- else }}
+{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" }}
+{{- end }}
+{{- end }}
+{{- end }}
+
 {{/*
 Selector labels
 */}}
@@ -22,8 +47,12 @@ Get the secret name for the API key
 {{- if .Values.pdp.existingApiKeySecret -}}
 {{- .Values.pdp.existingApiKeySecret.name -}}
 {{- else -}}
+{{- if .Values.useStandardHelmNamingConventions }}
+{{- include "..fullname" . }}
+{{- else -}}
 permitio-pdp-secret
 {{- end -}}
+{{- end -}}
 {{- end }}
 
 {{/*

charts/pdp/templates/deployment.yaml

@@ -1,7 +1,11 @@
 apiVersion: apps/v1
 kind: Deployment
 metadata:
+  {{- if .Values.useStandardHelmNamingConventions }}
+  name: {{ include "..fullname" . }}
+  {{- else }}
   name: permitio-pdp
+  {{- end }}
   labels:
     {{- include "pdp.labels" . | nindent 4 }}
   {{- with .Values.annotations }}

charts/pdp/templates/logs-forwarder-cm.yaml

@@ -2,7 +2,11 @@
 apiVersion: v1
 kind: ConfigMap
 metadata:
+  {{- if .Values.useStandardHelmNamingConventions }}
+  name: {{ include "..fullname" . }}-fluentbit-config
+  {{- else }}
   name: fluentbit-config
+  {{- end }}
   {{- with .Values.labels }}
   labels:
     {{- toYaml . | nindent 4 }}

charts/pdp/templates/poddisruptionbudget.yaml

@@ -2,7 +2,11 @@
 apiVersion: policy/v1
 kind: PodDisruptionBudget
 metadata:
+  {{- if .Values.useStandardHelmNamingConventions }}
+  name: {{ include "..fullname" . }}
+  {{- else }}
   name: permitio-pdp-pdb
+  {{- end }}
   labels:
     {{- include "pdp.labels" . | nindent 4 }}
   {{- with .Values.annotations }}

charts/pdp/templates/service.yaml

@@ -1,7 +1,11 @@
 apiVersion: v1
 kind: Service
 metadata:
+  {{- if .Values.useStandardHelmNamingConventions }}
+  name: {{ include "..fullname" . }}
+  {{- else }}
   name: permitio-pdp
+  {{- end }}
   {{- with .Values.labels }}
   labels:
     {{- toYaml . | nindent 4 }}

charts/pdp/values.yaml

@@ -1,6 +1,25 @@
+# Feature flag added to preserve backward compatibility with the old naming conventions
+# When useStandardHelmNamingConventions is false (default):
+# - Service name: permitio-pdp
+# - Secret name: permitio-pdp-secret
+# - Deployment name: permitio-pdp
+# - PodDisruptionBudget name: permitio-pdp-pdb
+#
+# When useStandardHelmNamingConventions is true:
+# - Service name: {release-name}-{chart-name} (e.g., my-release-pdp)
+# - Secret name: {release-name}-{chart-name} (e.g., my-release-pdp)
+# - Deployment name: {release-name}-{chart-name} (e.g., my-release-pdp)
+# - PodDisruptionBudget name: {release-name}-{chart-name} (e.g., my-release-pdp)
+#
+# The standard Helm naming convention follows the pattern {release-name}-{chart-name}
+# and automatically truncates names to 63 characters to comply with Kubernetes DNS naming spec.
+useStandardHelmNamingConventions: false
+
 labels: {}
 annotations: {}
 
+nameOverride: ""
+
 pdp:
   pdpEnvs:
     []