fix for DE620051

CAAPIM · Nov 11, 2024 · cdd2cb0 · cdd2cb0
1 parent 19ac936
commit cdd2cb0
Show file tree

Hide file tree

Showing 6 changed files with 33 additions and 2 deletions.
diff --git a/charts/portal/Chart.yaml b/charts/portal/Chart.yaml
@@ -2,14 +2,13 @@ apiVersion: v2
 appVersion: "5.3.1"
 description: CA API Developer Portal
 name: portal
-version: 2.3.10
+version: 2.3.11
 type: application
 home: https://github.com/CAAPIM/apim-charts
 maintainers:
   - name: Gazza7205
 sources:
   - https://github.com/CAAPIM/apim-charts
-engine: gotpl
 dependencies:
 - name: druid
   version: ^1.0.0

diff --git a/charts/portal/README.md b/charts/portal/README.md
@@ -3,6 +3,22 @@ The Layer7 API Developer Portal (API Portal) is part of the Layer7 API Managemen
 
 ## Introduction
 This Chart deploys the Layer7 API Developer Portal on a Kubernetes Cluster using the Helm Package Manager.
+
+## Release Notes
+
+## 2.3.11 General Updates
+- Added a preflight check (initContainer) for the core apim/ingress deployment
+  - This resolves a race condition that occurs on slower hardware where apim/ingress starts before other dependent services are ready. 
+  - This is ***enabled by default***. A Helm upgrade will restart the apim deployment.
+    - If you wish to disable this, set apim.preflightCheck.enabled to false
+    ```
+    apim:
+      ...
+      preflightCheck:
+        enabled: true
+      ...
+    ```
+
 ## 2.3.10 General Updates
 - This new version of the chart supports API Portal 5.3.1
 - Removed PSSG container
@@ -302,6 +318,7 @@ This section describes configurable parameters in **values.yaml**, there is also
 | `analytics.affinity`                 | Affinity for pod assignment                                  | `{} evaluated as a template`                                 |
 | `analytics.additionalLabels`         | A list of custom key: value labels                           | `not set`                                                    |
 | `apim.forceRedeploy`                 | Force redeployment during helm upgrade whether there is a change or not | `false`                                                      |
+| `apim.preflightCheck.enabled`        | Resolves a race condition that occurs on slower hardware where apim/ingress starts before other dependent services are ready |`true`                                                      |
 | `apim.replicaCount`                  | Number of APIM nodes                                         | `1`                                                          |
 | `apim.image.pullPolicy`              | APIM image pull policy                                       | `IfNotPresent`                                               |
 | `apim.otkDb.name`                    | APIM OTK Database name                                       | `otk_db`                                                     |

diff --git a/charts/portal/charts/druid-1.0.14.tgz b/charts/portal/charts/druid-1.0.14.tgz
diff --git a/charts/portal/templates/apim/apim-deployment.yaml b/charts/portal/templates/apim/apim-deployment.yaml
@@ -55,6 +55,17 @@ spec:
         - name: liquidbase-upgrade-rbac
           image: "{{ .Values.global.portalRepository }}{{ .Values.image.upgradeVerify }}"
           args: ["job", 'rbac-upgrade']
+        {{- if .Values.apim.preflightCheck.enabled }}
+        - name: apim-core-preflight-check
+          image: "{{ .Values.global.portalRepository }}{{ .Values.image.upgradeVerify }}"
+          command: ["/bin/sh","-c"]
+          args:
+          - sleep 120;
+            if [[ "$(curl -w '%{http_code}' -s -o /dev/null rabbitmq:15672)" != "200" ]]; then echo "broker not ready" && exit 1;fi;
+            if [[ "$(curl -w '%{http_code}' -s -o /dev/null portal-data:8080/portal-data/portalHealth)" != "200" ]]; then echo "portal-data not ready" && exit 1;fi;
+            if [[ "$(curl -w '%{http_code}' -s -o /dev/null http://authenticator:9081/v1/echo;)" != "200" ]]; then echo "authenticator not ready" && exit 1;fi;
+            if [[ "$(curl -w '%{http_code}' -s -o /dev/null  http://tenant-provisioner:9001/actuator/health)" != "200" ]]; then echo "tenant provisioning service not ready" && exit 1;fi;
+        {{- end }}
       containers:
         - name: apim
           image: "{{ .Values.global.portalRepository }}{{ .Values.image.apim }}"

diff --git a/charts/portal/values-production.yaml b/charts/portal/values-production.yaml
@@ -217,6 +217,8 @@ analytics:
 
 apim:
   forceRedeploy: false
+  preflightCheck:
+    enabled: true
   replicaCount: 2
   image:
     pullPolicy: IfNotPresent

diff --git a/charts/portal/values.yaml b/charts/portal/values.yaml
@@ -210,6 +210,8 @@ analytics:
 
 apim:
   forceRedeploy: false
+  preflightCheck:
+    enabled: true
   replicaCount: 1
   image:
     pullPolicy: IfNotPresent