diff --git a/Chart.yaml b/Chart.yaml index 6b6c65b..70ca781 100644 --- a/Chart.yaml +++ b/Chart.yaml @@ -12,7 +12,7 @@ description: A Helm chart for deploying HeLx to Kubernetes. type: application # 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. -version: 4.5.5 +version: 4.5.6 # This is the version number of the application being deployed. This version number should be # incremented each time you make changes to the application. diff --git a/README.md b/README.md index 90dd2e7..5ca5e98 100644 --- a/README.md +++ b/README.md @@ -2,7 +2,7 @@ A Helm chart for deploying HeLx to Kubernetes. -![Version: 4.5.5](https://img.shields.io/badge/Version-4.5.5-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 3.6.5](https://img.shields.io/badge/AppVersion-3.6.5-informational?style=flat-square) +![Version: 4.5.6](https://img.shields.io/badge/Version-4.5.6-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 3.6.5](https://img.shields.io/badge/AppVersion-3.6.5-informational?style=flat-square) HeLx puts the most advanced analytical scientific models at investigator’s finger tips using equally advanced cloud native, container orchestrated, distributed computing systems. HeLx can be applied in many domains. Its ability to empower researchers to leverage advanced analytical tools without installation or other infrastructure concerns has broad reaching benefits. diff --git a/charts/resty/README.md b/charts/resty/README.md index 0dad613..937ce4d 100644 --- a/charts/resty/README.md +++ b/charts/resty/README.md @@ -19,13 +19,13 @@ A Helm chart for Kubernetes | global.airflow_service_name | string | `"airflow-webserver"` | | | global.ambassador_service_name | string | `"ambassador"` | | | global.apps_namespace | string | `""` | Namespace where Tycho launches app pods (for /private DNS names). Defaults to the release namespace when empty. | -| global.appstore_service_name | string | `"appstore"` | PROTOTYPE (ambassador removal): direct backend service names that were previously reached only through the "ambassador" service. Set these to match your appstore / ui / sockets Service names. | +| global.appstore_service_name | string | `nil` | PROTOTYPE (ambassador removal): direct backend service names that were previously reached only through the "ambassador" service. appstore_service_name / ui_service_name: leave null to auto-derive — the bare chart name ("appstore"/"ui") when resty is deployed on its own, or "-appstore"/"-ui" when deployed as a subchart of the umbrella. Set explicitly only if your Service names differ. | | global.appstore_sockets_service_name | string | `"appstore-sockets-service"` | | | global.cluster_dns_suffix | string | `"svc.cluster.local"` | | | global.dug_search_client_service_name | string | `"dug-search-client"` | | | global.dug_web_service_name | string | `"dug-web"` | | | global.restartr_api_service_name | string | `"restartr-api-service"` | | -| global.ui_service_name | string | `"helx-ui"` | | +| global.ui_service_name | string | `nil` | | | image.pullPolicy | string | `"IfNotPresent"` | | | image.repository | string | `"bitnami/openresty"` | | | image.tag | float | `1.21` | Overrides the image tag whose default is the chart appVersion. | diff --git a/charts/resty/templates/nginx-default-configmap.yaml b/charts/resty/templates/nginx-default-configmap.yaml index d74af6d..603ff11 100644 --- a/charts/resty/templates/nginx-default-configmap.yaml +++ b/charts/resty/templates/nginx-default-configmap.yaml @@ -31,8 +31,14 @@ data: resolves via the `resolver` directive, which does not apply the pod's DNS search domains -- bare service names fail. Platform services are assumed to live in the same namespace as launched apps. */}} - {{- $appstore := printf "%s.%s.%s:8000" (.Values.global.appstore_service_name | default "appstore") $appsNamespace $dnsSuffix }} - {{- $ui := printf "%s.%s.%s:80" (.Values.global.ui_service_name | default "helx-ui") $appsNamespace $dnsSuffix }} + {{- /* Default the appstore/ui Service names to the bare chart name when + resty is the root chart (deployed on its own), or "-" + when it is a subchart of the umbrella (Helm prefixes subchart Service + names with the release name). Override via global.*_service_name. */}} + {{- $appstoreDefault := ternary "appstore" (printf "%s-appstore" .Release.Name) .Chart.IsRoot }} + {{- $uiDefault := ternary "ui" (printf "%s-ui" .Release.Name) .Chart.IsRoot }} + {{- $appstore := printf "%s.%s.%s:8000" (.Values.global.appstore_service_name | default $appstoreDefault) $appsNamespace $dnsSuffix }} + {{- $ui := printf "%s.%s.%s:80" (.Values.global.ui_service_name | default $uiDefault) $appsNamespace $dnsSuffix }} {{- $sockets := printf "%s.%s.%s:5555" (.Values.global.appstore_sockets_service_name | default "appstore-sockets-service") $appsNamespace $dnsSuffix }} {{- $airflow := printf "%s.%s.%s:8080" (.Values.global.airflow_service_name | default "airflow-webserver") $appsNamespace $dnsSuffix }} diff --git a/charts/resty/values.yaml b/charts/resty/values.yaml index 6887041..e8967dd 100644 --- a/charts/resty/values.yaml +++ b/charts/resty/values.yaml @@ -102,10 +102,13 @@ global: dug_search_client_service_name: dug-search-client restartr_api_service_name: restartr-api-service # -- PROTOTYPE (ambassador removal): direct backend service names that - # were previously reached only through the "ambassador" service. Set - # these to match your appstore / ui / sockets Service names. - appstore_service_name: appstore - ui_service_name: helx-ui + # were previously reached only through the "ambassador" service. + # appstore_service_name / ui_service_name: leave null to auto-derive — the + # bare chart name ("appstore"/"ui") when resty is deployed on its own, or + # "-appstore"/"-ui" when deployed as a subchart of the + # umbrella. Set explicitly only if your Service names differ. + appstore_service_name: + ui_service_name: appstore_sockets_service_name: appstore-sockets-service airflow_service_name: airflow-webserver # -- Namespace where Tycho launches app pods (for /private DNS names).