[bitnami/fluentd] When an HPA is enabled, it is recommended that the …

…value of spec.replicas of the StatefulSet be removed so hpa can manage the minReplicas & maxReplicas (bitnami#15688) Signed-off-by: Vinod <vinod.dampuru@thoughtworks.com>
rbudiharso · Mar 23, 2023 · ff8d8ce · ff8d8ce
1 parent 45349ae
commit ff8d8ce
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/bitnami/fluentd/Chart.yaml b/bitnami/fluentd/Chart.yaml
@@ -25,4 +25,4 @@ name: fluentd
 sources:
   - https://github.com/bitnami/containers/tree/main/bitnami/fluentd
   - https://www.fluentd.org/
-version: 5.6.0
+version: 5.6.1
diff --git a/bitnami/fluentd/README.md b/bitnami/fluentd/README.md
@@ -80,7 +80,7 @@ The command removes all the Kubernetes components associated with the chart and
 ### Fluentd parameters
 
 | Name                                                           | Description                                                                                                                                                        | Value                                                      |
-| -------------------------------------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------ | ---------------------------------------------------------- |
+| -------------------------------------------------------------- |--------------------------------------------------------------------------------------------------------------------------------------------------------------------| ---------------------------------------------------------- |
 | `image.registry`                                               | Fluentd image registry                                                                                                                                             | `docker.io`                                                |
 | `image.repository`                                             | Fluentd image repository                                                                                                                                           | `bitnami/fluentd`                                          |
 | `image.tag`                                                    | Fluentd image tag (immutable tags are recommended)                                                                                                                 | `1.15.3-debian-11-r46`                                     |
@@ -188,7 +188,7 @@ The command removes all the Kubernetes components associated with the chart and
 | `aggregator.image.registry`                                    | Fluentd aggregator image registry override                                                                                                                         | `""`                                                       |
 | `aggregator.image.repository`                                  | Fluentd aggregator image repository override                                                                                                                       | `""`                                                       |
 | `aggregator.image.tag`                                         | Fluentd aggregator image tag override (immutable tags are recommended)                                                                                             | `""`                                                       |
-| `aggregator.replicaCount`                                      | Number of aggregator pods to deploy in the Stateful Set                                                                                                            | `1`                                                        |
+| `aggregator.replicaCount`                                      | Number of aggregator pods to deploy in the Stateful Set. Will be ignored if autoscaling is enabled                                                                 | `1`                                                        |
 | `aggregator.podSecurityContext.enabled`                        | Enable security context for aggregator pods                                                                                                                        | `true`                                                     |
 | `aggregator.podSecurityContext.runAsUser`                      | User ID for aggregator's containers                                                                                                                                | `1001`                                                     |
 | `aggregator.podSecurityContext.runAsGroup`                     | Group ID for aggregator's containers                                                                                                                               | `1001`                                                     |

diff --git a/bitnami/fluentd/templates/aggregator-statefulset.yaml b/bitnami/fluentd/templates/aggregator-statefulset.yaml
@@ -24,7 +24,9 @@ spec:
       app.kubernetes.io/component: aggregator
   serviceName: {{ include "common.names.fullname" . }}-headless
   podManagementPolicy: {{ .Values.aggregator.podManagementPolicy }}
+  {{- if not .Values.aggregator.autoscaling.enabled }}
   replicas: {{ .Values.aggregator.replicaCount }}
+  {{- end }}
   updateStrategy: {{- toYaml .Values.aggregator.updateStrategy | nindent 4 }}
   template:
     metadata: