Skip to content

Commit 7889b7d

Browse files
authored
Fix HPA utilization calc by removing labels from test pod (#526)
1 parent 0fb9d1d commit 7889b7d

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

charts/whatsapp-proxy-chart/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ type: application
1717
# This is the chart version. This version number should be incremented each time you make changes
1818
# to the chart and its templates, including the app version.
1919
# Versions are expected to follow Semantic Versioning (https://semver.org/)
20-
version: 1.3.4
20+
version: 1.3.5
2121

2222
# This is the version number of the application being deployed. This version number should be
2323
# incremented each time you make changes to the application. Versions are not expected to

charts/whatsapp-proxy-chart/templates/tests/test-connection.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,12 @@ apiVersion: v1
66
kind: Pod
77
metadata:
88
name: "{{ include "whatsapp-proxy-chart.fullname" . }}-test-connection"
9-
labels:
10-
{{- include "whatsapp-proxy-chart.labels" . | nindent 4 }}
9+
# Intentionally omit common/selector labels so this pod is not matched by the
10+
# HPA's selector. This test pod has no resource requests, and including it
11+
# would break the HPA's utilization calculation.
1112
annotations:
1213
"helm.sh/hook": test
14+
"helm.sh/hook-delete-policy": hook-succeeded
1315
spec:
1416
containers:
1517
- name: wget

0 commit comments

Comments
 (0)