feat(hpa): update behavior configuration to support null values for memory utilization and ensure proper referencing in HPA settings

Author: abhi1693

charts/netbox/templates/hpa.yaml

@@ -16,9 +16,9 @@ spec:
     name: {{ include "common.names.fullname" . }}
   minReplicas: {{ .Values.autoscaling.minReplicas }}
   maxReplicas: {{ .Values.autoscaling.maxReplicas }}
-  {{- with .Values.autoscaling.behavior }}
+  {{- if .Values.autoscaling.behavior }}
   behavior:
-    {{- with .scaleUp }}
+    {{- with .Values.autoscaling.behavior.scaleUp }}
     scaleUp:
       {{- if .stabilizationWindowSeconds }}
       stabilizationWindowSeconds: {{ .stabilizationWindowSeconds }}
@@ -35,7 +35,7 @@ spec:
         {{- end }}
       {{- end }}
     {{- end }}
-    {{- with .scaleDown }}
+    {{- with .Values.autoscaling.behavior.scaleDown }}
     scaleDown:
       {{- if .stabilizationWindowSeconds }}
       stabilizationWindowSeconds: {{ .stabilizationWindowSeconds }}

charts/netbox/templates/worker/hpa.yaml

@@ -16,9 +16,9 @@ spec:
     name: {{ include "common.names.fullname" . }}-worker
   minReplicas: {{ .Values.worker.autoscaling.minReplicas }}
   maxReplicas: {{ .Values.worker.autoscaling.maxReplicas }}
-  {{- with .Values.worker.autoscaling.behavior }}
+  {{- if .Values.worker.autoscaling.behavior }}
   behavior:
-    {{- with .scaleUp }}
+    {{- with .Values.worker.autoscaling.behavior.scaleUp }}
     scaleUp:
       {{- if .stabilizationWindowSeconds }}
       stabilizationWindowSeconds: {{ .stabilizationWindowSeconds }}
@@ -35,7 +35,7 @@ spec:
         {{- end }}
       {{- end }}
     {{- end }}
-    {{- with .scaleDown }}
+    {{- with .Values.worker.autoscaling.behavior.scaleDown }}
     scaleDown:
       {{- if .stabilizationWindowSeconds }}
       stabilizationWindowSeconds: {{ .stabilizationWindowSeconds }}

charts/netbox/values.schema.json

@@ -175,10 +175,10 @@
           "type": "integer"
         },
         "targetCPUUtilizationPercentage": {
-          "type": "integer"
+          "type": ["integer", "null"]
         },
         "targetMemoryUtilizationPercentage": {
-          "type": "integer"
+          "type": ["integer", "null"]
         },
         "behavior": {
           "type": "object",
@@ -1530,10 +1530,10 @@
               "type": "integer"
             },
             "targetCPUUtilizationPercentage": {
-              "type": "integer"
+              "type": ["integer", "null"]
             },
             "targetMemoryUtilizationPercentage": {
-              "type": "integer"
+              "type": ["integer", "null"]
             },
             "behavior": {
               "type": "object",

charts/netbox/values.yaml

@@ -1165,8 +1165,8 @@ autoscaling:
   minReplicas: 1
   maxReplicas: 100
   targetCPUUtilizationPercentage: 80
-  # targetMemoryUtilizationPercentage: 80
-  # behavior:
+  targetMemoryUtilizationPercentage: null
+  behavior: {}
   #   scaleUp:
   #     stabilizationWindowSeconds: 0
   #     selectPolicy: Max
@@ -1663,8 +1663,8 @@ worker:
     minReplicas: 1
     maxReplicas: 100
     targetCPUUtilizationPercentage: 80
-    # targetMemoryUtilizationPercentage: 80
-    # behavior:
+    targetMemoryUtilizationPercentage: null
+    behavior: {}
     #   scaleUp:
     #     stabilizationWindowSeconds: 0
     #     selectPolicy: Max