From 2b4d366af41acccf112fba5383a9a929bf340db1 Mon Sep 17 00:00:00 2001 From: Nick Andreano Date: Mon, 16 Feb 2026 12:08:13 -0500 Subject: [PATCH 1/4] Removed repository value --- charts/iag4/values.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/charts/iag4/values.yaml b/charts/iag4/values.yaml index 465569f..d5df2cc 100644 --- a/charts/iag4/values.yaml +++ b/charts/iag4/values.yaml @@ -25,7 +25,8 @@ external-dns: # The image name will depend on the entitlement that has been granted. image: # -- The image repository - repository: 497639811223.dkr.ecr.us-east-2.amazonaws.com/automation-gateway + # -- Contact Itential to get the correct image repository for your entitlement. + repository: # -- The image pull policy pullPolicy: IfNotPresent # -- The image tag From a2fc46de0b638f5da51841fe24f770b5d6425517 Mon Sep 17 00:00:00 2001 From: Nick Andreano Date: Thu, 5 Mar 2026 11:16:24 -0500 Subject: [PATCH 2/4] updated LivenessProbe to check process --- charts/iag4/templates/statefulset.yaml | 9 +++++---- charts/iag4/tests/statefulset_test.yaml | 26 ++++++++++++++++++++----- charts/iag4/values.yaml | 3 +-- 3 files changed, 27 insertions(+), 11 deletions(-) diff --git a/charts/iag4/templates/statefulset.yaml b/charts/iag4/templates/statefulset.yaml index b79ff70..f10bcf1 100644 --- a/charts/iag4/templates/statefulset.yaml +++ b/charts/iag4/templates/statefulset.yaml @@ -207,10 +207,11 @@ spec: {{- if .Values.livenessProbe.enabled }} livenessProbe: failureThreshold: {{ .Values.livenessProbe.failureThreshold }} - httpGet: - path: {{ .Values.livenessProbe.path }} - port: {{ .Values.applicationPort }} - scheme: {{ ternary "HTTPS" "HTTP" .Values.useTLS }} + exec: + command: + - sh + - -c + - 'test $(pgrep -f automation-gateway | wc -l) -ge 2' periodSeconds: {{ .Values.livenessProbe.periodSeconds }} successThreshold: {{ .Values.livenessProbe.successThreshold }} timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds }} diff --git a/charts/iag4/tests/statefulset_test.yaml b/charts/iag4/tests/statefulset_test.yaml index aa10ac2..b91c2c9 100644 --- a/charts/iag4/tests/statefulset_test.yaml +++ b/charts/iag4/tests/statefulset_test.yaml @@ -285,28 +285,44 @@ tests: set: statefulset.enabled: true serviceAccount.name: "test-sa" - applicationPort: 8080 - useTLS: false livenessProbe: enabled: true periodSeconds: 20 timeoutSeconds: 10 failureThreshold: 5 successThreshold: 1 - path: "/health" applicationSettings: enabled: true env: {} asserts: - equal: - path: spec.template.spec.containers[0].livenessProbe.httpGet.scheme - value: "HTTP" + path: spec.template.spec.containers[0].livenessProbe.exec.command + value: + - sh + - -c + - 'test $(pgrep -f automation-gateway | wc -l) -ge 2' - equal: path: spec.template.spec.containers[0].livenessProbe.periodSeconds value: 20 - equal: path: spec.template.spec.containers[0].livenessProbe.failureThreshold value: 5 + - notExists: + path: spec.template.spec.containers[0].livenessProbe.httpGet + + # Test 12b: Test liveness probe is not created when disabled + - it: should not configure liveness probe when disabled + set: + statefulset.enabled: true + serviceAccount.name: "test-sa" + livenessProbe: + enabled: false + applicationSettings: + enabled: true + env: {} + asserts: + - notExists: + path: spec.template.spec.containers[0].livenessProbe # Test 13: Test readiness probe - it: should configure readiness probe when enabled diff --git a/charts/iag4/values.yaml b/charts/iag4/values.yaml index d5df2cc..89f6f5e 100644 --- a/charts/iag4/values.yaml +++ b/charts/iag4/values.yaml @@ -160,7 +160,6 @@ livenessProbe: timeoutSeconds: 5 failureThreshold: 3 successThreshold: 1 - path: "/api/v2.0/poll" # The readiness probe used to determine if the container is ready to accept requests. If the # readiness probe fails, the endpoints controller removes the pod’s IP address from the @@ -168,7 +167,7 @@ livenessProbe: # initial delay is failure. If a container does not provide a readiness probe, the default # state is success. These are suggested values. readinessProbe: - readinessProbe: false + enabled: false periodSeconds: 10 timeoutSeconds: 5 failureThreshold: 3 From 3c58814f0da25f052fede1b43e1ca47ec8d73bd1 Mon Sep 17 00:00:00 2001 From: Nick Andreano Date: Thu, 5 Mar 2026 13:16:55 -0500 Subject: [PATCH 3/4] updated helm test to be compatable with platformhooks. --- .github/workflows/helm-ci.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/helm-ci.yaml b/.github/workflows/helm-ci.yaml index fd4d107..7017804 100644 --- a/.github/workflows/helm-ci.yaml +++ b/.github/workflows/helm-ci.yaml @@ -17,9 +17,9 @@ jobs: uses: actions/checkout@v4 - name: Set up Helm - uses: azure/setup-helm@v3 + uses: azure/setup-helm@v4 with: - version: v3.13.3 + version: v3.17.0 - name: Install helm-unittest plugin run: helm plugin install https://github.com/helm-unittest/helm-unittest From 6ea0aec2336934bccf5866793ef6205359530db5 Mon Sep 17 00:00:00 2001 From: Nick Andreano Date: Thu, 5 Mar 2026 13:19:39 -0500 Subject: [PATCH 4/4] updated helm test to be compatable with platformhooks. --- .github/workflows/helm-ci.yaml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/helm-ci.yaml b/.github/workflows/helm-ci.yaml index 7017804..5aacfa7 100644 --- a/.github/workflows/helm-ci.yaml +++ b/.github/workflows/helm-ci.yaml @@ -17,12 +17,12 @@ jobs: uses: actions/checkout@v4 - name: Set up Helm - uses: azure/setup-helm@v4 + uses: azure/setup-helm@v3 with: - version: v3.17.0 + version: v3.16.4 - name: Install helm-unittest plugin - run: helm plugin install https://github.com/helm-unittest/helm-unittest + run: helm plugin install https://github.com/helm-unittest/helm-unittest --version v0.6.0 - name: Lint Helm chart run: helm lint charts/iag4