feat(fluentd-aggregator): Updated image to v2.10.1

Signed-off-by: Steve Hipwell <[email protected]>

charts/fluentd-aggregator/CHANGELOG.md

@@ -20,6 +20,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [UNRELEASED]
 
+## [v4.1.1] - 2023-12-07
+
+### Changed
+
+- Updated _Fluentd Aggregator_ OCI image to [v2.10.1](https://github.com/stevehipwell/fluentd-aggregator/releases/tag/v2.10.1) (_Fluentd_ [v1.16.3](https://github.com/fluent/fluentd/releases/tag/v1.16.3)).
+
+### Fixed
+
+- Fixed legacy migration by adding `service.legacy` value to support keeping the legacy headless service name.
+- Fixed incorrect truncation of names when full name is longer than `53` characters.
+
 ## [v4.1.0] - 2023-11-23
 
 ### Changed

charts/fluentd-aggregator/Chart.yaml

@@ -2,8 +2,8 @@ apiVersion: v2
 name: fluentd-aggregator
 description: Helm chart for Fluentd running as an aggregation StatefulSet and using the fluent-plugin-route router.
 type: application
-version: 4.1.0
-appVersion: 2.10.0
+version: 4.1.1
+appVersion: 2.10.1
 keywords:
   - fluent
   - fluentd
@@ -22,4 +22,8 @@ annotations:
   artifacthub.io/alternativeName: fluentd
   artifacthub.io/changes: |
     - kind: changed
-      description: "Updated _Fluentd Aggregator_ OCI image to [v2.10.0](https://github.com/stevehipwell/fluentd-aggregator/releases/tag/v2.10.0) (_Fluentd_ [v1.16.3](https://github.com/fluent/fluentd/releases/tag/v1.16.3))."
+      description: "Updated _Fluentd Aggregator_ OCI image to [v2.10.1](https://github.com/stevehipwell/fluentd-aggregator/releases/tag/v2.10.1) (_Fluentd_ [v1.16.3](https://github.com/fluent/fluentd/releases/tag/v1.16.3))."
+    - kind: fixed
+      description: "Fixed legacy migration by adding `service.legacy` value to support keeping the legacy headless service name."
+    - kind: fixed
+      description: "Fixed incorrect truncation of names when full name is longer than `53` characters."

charts/fluentd-aggregator/templates/_helpers.tpl

@@ -68,28 +68,36 @@ Create the name of the service account to use
 Define the service name
 */}}
 {{- define "fluentd-aggregator.serviceName" -}}
+{{- if .Values.service.legacy }}
+{{- (printf "%s-default" (include "fluentd-aggregator.fullname" .) | trunc 55 | trimSuffix "-") }}
+{{- else }}
 {{- include "fluentd-aggregator.fullname" . }}
 {{- end }}
+{{- end }}
 
 {{/*
 Define the headless service name
 */}}
 {{- define "fluentd-aggregator.headlessServiceName" -}}
-{{- (printf "%s-headless" (include "fluentd-aggregator.serviceName" .)) | trunc 63 | trimSuffix "-" }}
+{{- if .Values.service.legacy }}
+{{- include "fluentd-aggregator.fullname" . }}
+{{- else }}
+{{- (printf "%s-headless" (include "fluentd-aggregator.serviceName" .) | trunc 54 | trimSuffix "-") }}
+{{- end }}
 {{- end }}
 
 {{/*
 Define the config configmap name
 */}}
 {{- define "fluentd-aggregator.configConfigMapName" -}}
-{{- (printf "%s-config" (include "fluentd-aggregator.fullname" .)) | trunc 63 | trimSuffix "-" }}
+{{- (printf "%s-config" (include "fluentd-aggregator.fullname" .) | trunc 56 | trimSuffix "-") }}
 {{- end }}
 
 {{/*
 Define the dashboard configmap name
 */}}
 {{- define "fluentd-aggregator.dashboardConfigMapName" -}}
-{{- (printf "%s-dashboard" (include "fluentd-aggregator.fullname" .)) | trunc 63 | trimSuffix "-" }}
+{{- (printf "%s-dashboard" (include "fluentd-aggregator.fullname" .) | trunc 53 | trimSuffix "-") }}
 {{- end }}
 
 {{/*

charts/fluentd-aggregator/values.yaml

@@ -54,6 +54,8 @@ service:
       port: 24224
       containerPort: 24224
       protocol: TCP
+  # -- If `true`, use legacy naming so the headless service doesn't change.
+  legacy: false
 
 # -- Ingresses, each input plugin will need it's own.
 ingresses: []