[stable/fairwinds-insights] add option for hostname max length (#1364)

* Truncate URLs to 17 chars * Update Chart.yaml * Update CHANGELOG.md * Update _helpers.tpl * Update CHANGELOG.md * Update values.yaml * helm-docs * Update _helpers.tpl * Update _helpers.tpl
FairwindsOps · Nov 8, 2023 · 6e6b3d3 · 6e6b3d3
1 parent 87f2863
commit 6e6b3d3
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 2 deletions.
diff --git a/stable/fairwinds-insights/CHANGELOG.md b/stable/fairwinds-insights/CHANGELOG.md
@@ -1,5 +1,8 @@
 # Changelog
 
+## 0.21.8
+* Allow additional chars in the URL prefix
+
 ## 0.21.7
 * Update application version to 14.7. [See the release notes for more details](https://insights.docs.fairwinds.com/release-notes)
 

diff --git a/stable/fairwinds-insights/Chart.yaml b/stable/fairwinds-insights/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 appVersion: "14.7"
 description: A Helm chart to run the Fairwinds Insights server
 name: fairwinds-insights
-version: 0.21.7
+version: 0.21.8
 kubeVersion: ">= 1.22.0-0"
 maintainers:
   - name: rbren

diff --git a/stable/fairwinds-insights/README.md b/stable/fairwinds-insights/README.md
@@ -127,6 +127,7 @@ See [insights.docs.fairwinds.com](https://insights.docs.fairwinds.com/technical-
 | service.type | string | `"ClusterIP"` | Service type for the API and Dashboard services |
 | service.annotations | string | `nil` | Annotations for the services |
 | sanitizedBranch | string | `nil` | Prefix to use on hostname. Generally not needed. |
+| sanitizedPrefixMaxLength | int | `12` | Maximum length for hostname prefix. |
 | ingress.enabled | bool | `false` | Enable Ingress |
 | ingress.tls | bool | `true` | Enable TLS |
 | ingress.hostedZones | list | `[]` | Hostnames to use for Ingress |

diff --git a/stable/fairwinds-insights/templates/_helpers.tpl b/stable/fairwinds-insights/templates/_helpers.tpl
@@ -33,7 +33,7 @@ Create chart name and version as used by the chart label.
 
 {{- define "fairwinds-insights.sanitizedPrefix" -}}
 {{- if .Values.sanitizedBranch -}}
-{{- printf "%s." (.Values.sanitizedBranch | trunc 12 | trimSuffix "-") -}}
+{{- printf "%s." (.Values.sanitizedBranch | trunc (int .Values.sanitizedPrefixMaxLength | default 12) | trimSuffix "-") -}}
 {{- end -}}
 {{- end -}}
 

diff --git a/stable/fairwinds-insights/values.yaml b/stable/fairwinds-insights/values.yaml
@@ -433,6 +433,9 @@ service:
 # -- Prefix to use on hostname. Generally not needed.
 sanitizedBranch:
 
+# -- Maximum length for hostname prefix.
+sanitizedPrefixMaxLength: 12
+
 ingress:
   # -- Enable Ingress
   enabled: false