diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index d31dc212..205f0156 100644 --- a/.github/workflows/release.yml +++ b/.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 diff --git a/helm-chart/eoapi/CHANGELOG.md b/helm-chart/eoapi/CHANGELOG.md index a1cffc41..219d160a 100644 --- a/helm-chart/eoapi/CHANGELOG.md +++ b/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 diff --git a/helm-chart/eoapi/Chart.yaml b/helm-chart/eoapi/Chart.yaml index 6b17a024..17a54423 100644 --- a/helm-chart/eoapi/Chart.yaml +++ b/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" diff --git a/helm-chart/eoapi/templates/_helpers.tpl b/helm-chart/eoapi/templates/_helpers.tpl index 8d4dd98e..e22b55be 100644 --- a/helm-chart/eoapi/templates/_helpers.tpl +++ b/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 -}} - diff --git a/helm-chart/eoapi/tests/hpa_tests.yaml b/helm-chart/eoapi/tests/hpa_tests.yaml index 3fa342d0..1f0ca6d1 100644 --- a/helm-chart/eoapi/tests/hpa_tests.yaml +++ b/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 +