Adapt HPA values and scaling base (#819)

* Adapt HPA values and scaling base * Update from review * Apply suggestions from code review Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com> --------- Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com> Co-authored-by: Tim Riffer <tim.riffer@sap.com>
kyma-project · May 21, 2024 · 0e2c735 · 0e2c735
1 parent 8628aaa
commit 0e2c735
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 10 deletions.
diff --git a/docs/release-notes/1.7.0.md b/docs/release-notes/1.7.0.md
@@ -3,4 +3,6 @@
 - Allow for opting out of the **ENABLE_EXTERNAL_NAME_ALIAS** Istio pilot environment variable in the Istio custom resource. This allows for retaining behavior that was present in Istio prior to version 1.21. See issue [#787](https://github.com/kyma-project/istio/issues/787 ).
 - Update the Istio version to 1.21.2 [#802](https://github.com/kyma-project/istio/pull/802). Read [Istio 1.21.2 Release Announcement](https://istio.io/latest/news/releases/1.21.x/announcing-1.21.2/) and [Change Notes](https://istio.io/latest/news/releases/1.21.x/announcing-1.21/change-notes/) for more details.
 - Add Request Authentication UI for Kyma dashboard [#816](https://github.com/kyma-project/istio/pull/816)
-- Now, Istio Operator does not restart Pods with Istio Sidecar, which contain custom image annotations. See the issue [#698](https://github.com/kyma-project/istio/issues/698) and [Istio Resource Annotations](https://istio.io/latest/docs/reference/config/annotations/#SidecarProxyImage) for more details.
+- Now, Istio Operator does not restart Pods with Istio Sidecar, which contain custom image annotations. See the issue [#698](https://github.com/kyma-project/istio/issues/698) and [Istio Resource Annotations](https://istio.io/latest/docs/reference/config/annotations/#SidecarProxyImage) for more details.
+- Change Istio Ingress Gateway's scaling to be based only on CPU utilization. This adjustment ensures that the scaling is more responsive to traffic changes, as the Istio Ingress Gateway memory utilization is not a good indicator of the traffic load.
+- Set the default number of Istio Ingress Gateway replicas in smaller clusters to `1`.
diff --git a/docs/user/00-50-resource-configuration.md b/docs/user/00-50-resource-configuration.md
@@ -44,7 +44,7 @@ Istio Controller installs Istio with a configuration that depends in the cluster
 | Component       | Min replicas | Max replicas |
 |-----------------|--------------|--------------|
 | Pilot           | 1            | 1            |
-| Ingress Gateway | 1            | 3            |
+| Ingress Gateway | 1            | 1            |
 
 ### CNI Autoscaling
 

diff --git a/internal/istiooperator/istio-operator-light.yaml b/internal/istiooperator/istio-operator-light.yaml
@@ -61,7 +61,7 @@ spec:
                   topologyKey: kubernetes.io/hostname
                 weight: 100
         hpaSpec:
-          maxReplicas: 5
+          maxReplicas: 1
           minReplicas: 1
         resources:
           limits:
@@ -308,7 +308,7 @@ spec:
       injectionURL: ""
     pilot:
       autoscaleEnabled: true
-      autoscaleMax: 5
+      autoscaleMax: 1
       autoscaleMin: 1
       configMap: true
       cpu:

diff --git a/internal/istiooperator/istio-operator.yaml b/internal/istiooperator/istio-operator.yaml
@@ -69,12 +69,6 @@ spec:
                 type: Utilization
                 averageUtilization: 80
             type: Resource
-          - resource:
-              name: memory
-              target:
-                type: Utilization
-                averageUtilization: 80
-            type: Resource
           minReplicas: 3
         resources:
           limits: