avoid immediate scaleup if autoscale is enabled

.github/workflows/helm-tests.yml

@@ -142,6 +142,8 @@ jobs:
 #        uses: lhotari/action-upterm@v1
 
       - name: run the tests
+        id: testrunner
+        continue-on-error: true
         run: |
           kubectl config set-context --current --namespace=eoapitest
           PUBLICIP='http://'$(kubectl -n ingress-nginx get svc/ingress-nginx-controller -o jsonpath='{.spec.loadBalancerIP}')
@@ -165,6 +167,14 @@ jobs:
           head -n 5 .github/workflows/tests/test_raster.py
           pytest .github/workflows/tests/test_raster.py
 
+      - name: cleanup if tests faile
+        if: steps.testrunner.outcome == 'failure'
+        run: |
+          echo "The previous step failed or timed out. Running cleanup logic..."
+          helm uninstall $RELEASE_NAME
+          # force GH action to show failed result
+          exit 128
+
       - name: helm uinstall eoapi templates
         run: |
           helm uninstall $RELEASE_NAME

helm-chart/eoapi/CHANGELOG.md

@@ -1,6 +1,10 @@
 ##### version numbers below correspond to helm chart `appVersion`: see ./helm-chart/eoapi/Chart.yaml
 ---
-### 0.1.7 (2023-10-01)
+### 0.1.8 (2023-10-02)
+
+* adjust cpu limits so if autoscaling is enabled it doesn't immediately scaleup
+
+### 0.1.7 (2023-10-02)
 
 * adds `autoscaling` options to each service for HPA
 

helm-chart/eoapi/Chart.yaml

@@ -17,10 +17,10 @@ kubeVersion: ">=1.23.0-0"
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: "0.1.9"
+version: "0.1.10"
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to
 # follow Semantic Versioning. They should reflect the version the application is using.
 # It is recommended to use it with quotes.
-appVersion: "0.1.7"
+appVersion: "0.1.8"

helm-chart/eoapi/tests/deploy_tests.yaml

@@ -18,7 +18,7 @@ tests:
           value: "RollingUpdate"
       - equal:
           path: spec.template.spec.containers[0].resources.limits.cpu
-          value: "512m"
+          value: "768m"
       - equal:
           path: spec.template.spec.containers[0].resources.requests.cpu
           value: "256m"
@@ -47,7 +47,7 @@ tests:
           value: "RollingUpdate"
       - equal:
           path: spec.template.spec.containers[0].resources.limits.cpu
-          value: "512m"
+          value: "768m"
       - equal:
           path: spec.template.spec.containers[0].resources.requests.cpu
           value: "256m"
@@ -76,7 +76,7 @@ tests:
           value: "RollingUpdate"
       - equal:
           path: spec.template.spec.containers[0].resources.limits.cpu
-          value: "512m"
+          value: "768m"
       - equal:
           path: spec.template.spec.containers[0].resources.requests.cpu
           value: "256m"

helm-chart/eoapi/values.yaml

@@ -108,7 +108,7 @@ raster:
     # https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/
     resources:
       limits:
-        cpu: "512m"
+        cpu: "768m"
         memory: "4096Mi"
       requests:
         cpu: "256m"
@@ -156,7 +156,7 @@ stac:
     # https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/
     resources:
       limits:
-        cpu: "512m"
+        cpu: "768m"
         memory: "1024Mi"
       requests:
         cpu: "256m"
@@ -191,7 +191,7 @@ vector:
     # https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/
     resources:
       limits:
-        cpu: "512m"
+        cpu: "768m"
         memory: "1024Mi"
       requests:
         cpu: "256m"