tests and releases

.github/workflows/release.yml

@@ -7,7 +7,7 @@ on:
 
 jobs:
   release:
-    if: github.actor == 'ranchodeluxe' || github.actor == 'gcorradini'
+    if: github.actor == 'ranchodeluxe' || github.actor == 'gcorradini' || github.actor == 'sunu'
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3

helm-chart/eoapi/CHANGELOG.md

@@ -1,5 +1,10 @@
 version numbers below correspond to helm chart `appVersion`: see `./helm-chart/eoapi/Chart.yaml`
 ---
+### 0.2.10 (2023-11-22)
+
+* give `ingress.className == "nginx"` the ability pass hosts and tls information
+* more tests to give user feedback about the above functionality if the wrong `ingress.className` was given
+
 ### 0.2.9 (2023-10-25)
 
 * removed `providerContext` and any support for minikube from the templates so this is a breaking change

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.11"
+version: "0.1.12"
 
 # 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.2.9"
+appVersion: "0.2.10"

helm-chart/eoapi/templates/_helpers.tpl

@@ -78,8 +78,14 @@ values.schema.json doesn't play nice combined value checks
 so we use this helper function to check autoscaling rules
 */}}
 {{- define "eoapi.validateAutoscaleRules" -}}
-{{- if and .Values.ingress.enabled (ne .Values.ingress.className "nginx") (or (.Values.raster.autoscaling.enabled) (.Values.stac.autoscaling.enabled) (.Values.vector.autoscaling.enabled)) }}
-{{- fail "When using an 'ingress.className' other than 'nginx' you cannot enable autoscaling by 'requestRate' at this time b/c it's soley an nginx metric" }}
+{{- if and .Values.ingress.enabled (ne .Values.ingress.className "nginx") }}
+{{/* "requestRate" cannot be enabled for any service if not "nginx" so give feedback and fail */}}
+{{- if (or (and .Values.raster.autoscaling.enabled (eq .Values.raster.autoscaling.type "requestRate")) (and .Values.stac.autoscaling.enabled (eq .Values.stac.autoscaling.type "requestRate")) (and .Values.vector.autoscaling.enabled (eq .Values.vector.autoscaling.type "requestRate")) ) }}
+{{- fail "When using an 'ingress.className' other than 'nginx' you cannot enable autoscaling by 'requestRate' at this time b/c it's solely an nginx metric" }}
+{{- end }}
+{{/* "both" cannot be enabled for any service if not "nginx" so give feedback and fail */}}
+{{- if (or (and .Values.raster.autoscaling.enabled (eq .Values.raster.autoscaling.type "both")) (and .Values.stac.autoscaling.enabled (eq .Values.stac.autoscaling.type "both")) (and .Values.vector.autoscaling.enabled (eq .Values.vector.autoscaling.type "both")) ) }}
+{{- fail "When using an 'ingress.className' other than 'nginx' you cannot enable autoscaling by 'both' at this time b/c 'requestRate' is solely an nginx metric" }}
+{{- end }}
 {{- end }}
 {{- end -}}
-

helm-chart/eoapi/tests/hpa_tests.yaml

@@ -1,8 +1,8 @@
-suite: service defaults deployment
+suite: autoscaling feedback when className is not nginx
 templates:
   - templates/services/hpa.yaml
 tests:
-  - it: "vector hpa fail state"
+  - it: "vector hpa fail for requestRate"
     set:
       raster.enabled: false
       stac.enabled: false
@@ -11,4 +11,5 @@ tests:
       vector.autoscaling.enabled: true
     asserts:
       - failedTemplate:
-          errorMessage: When using an 'ingress.className' other than 'nginx' you cannot enable autoscaling by 'requestRate' at this time b/c it's soley an nginx metric
+          errorMessage: When using an 'ingress.className' other than 'nginx' you cannot enable autoscaling by 'requestRate' at this time b/c it's solely an nginx metric
+