fix: service names should be dynamic (#1958)

Service names should be unique per Helm release by default. Otherwise,
the user needs to know to override the hostname values.

Author: keegoid-nr

charts/synthetics-job-manager/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: synthetics-job-manager
 description: New Relic Synthetics Containerized Job Manager
 type: application
-version: 3.0.37
+version: 3.0.38
 appVersion: release-471
 home: https://github.com/orgs/newrelic/teams/proactive-monitoring
 maintainers:

charts/synthetics-job-manager/templates/_helpers.tpl

@@ -41,14 +41,16 @@ Allows to override the appVersion to use.
 Allows overriding of the synthetics-job-manager Service hostname
 */}}
 {{- define "synthetics-job-manager.hostname" -}}
-{{- default "synthetics-job-manager" .Values.global.hostnameOverride | trunc 63 | trimSuffix "-" }}
+{{- $name := default "synthetics-job-manager" (index .Values "global" "hostnameOverride") -}}
+{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}}
 {{- end }}
 
 {{/*
-Allows overriding of the ping-runtime Service hostname
+Allows overriding of the ping-runtime Service hostname, making it unique per release
 */}}
 {{- define "ping-runtime.hostname" -}}
-{{- default "ping" (index .Values "global" "ping-runtime" "hostnameOverride") | trunc 63 | trimSuffix "-" }}
+{{- $name := default "ping" (index .Values "global" "ping-runtime" "hostnameOverride") -}}
+{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}}
 {{- end }}
 
 {{/*