Add redis as an option in the chart

Author: whatnick

.github/actions/setup-helm-repos/action.yml

@@ -9,4 +9,5 @@ runs:
       run: |
         helm repo add elasticsearch https://helm.elastic.co
         helm repo add opensearch https://opensearch-project.github.io/helm-charts/
+        helm repo add bitnami https://charts.bitnami.com/bitnami
         helm repo update

helm-chart/stac-fastapi/Chart.yaml

@@ -30,3 +30,7 @@ dependencies:
     version: 3.2.1
     repository: https://opensearch-project.github.io/helm-charts/
     condition: opensearch.enabled
+  - name: redis
+    version: 19.6.2
+    repository: https://charts.bitnami.com/bitnami
+    condition: redis.enabled

helm-chart/stac-fastapi/README.md

@@ -20,10 +20,19 @@ The chart provides a flexible deployment solution for STAC FastAPI with the foll
 
 ## Quick Start
 
+### Add Helm repositories
+
+```bash
+helm repo add elasticsearch https://helm.elastic.co
+helm repo add opensearch https://opensearch-project.github.io/helm-charts/
+helm repo add bitnami https://charts.bitnami.com/bitnami
+helm repo update
+```
+
 ### Deploy with Elasticsearch
 
 ```bash
-# Add dependencies
+# Download chart dependencies
 helm dependency update ./helm-chart/stac-fastapi
 
 # Install with Elasticsearch backend
@@ -36,7 +45,7 @@ helm install my-stac-api ./helm-chart/stac-fastapi \
 ### Deploy with OpenSearch
 
 ```bash
-# Add dependencies
+# Download chart dependencies
 helm dependency update ./helm-chart/stac-fastapi
 
 # Install with OpenSearch backend

helm-chart/stac-fastapi/templates/_helpers.tpl

@@ -165,6 +165,48 @@ Create the database port based on backend selection
 {{- end }}
 {{- end }}
 
+{{/*
+Determine the Redis service name when the bundled chart is enabled
+*/}}
+{{- define "stac-fastapi.redisServiceName" -}}
+{{- if .Values.redis.fullnameOverride }}
+{{- printf "%s-master" .Values.redis.fullnameOverride }}
+{{- else }}
+{{- printf "%s-redis-master" .Release.Name }}
+{{- end }}
+{{- end }}
+
+{{/*
+Compute the Redis host, preferring the bundled deployment when enabled
+*/}}
+{{- define "stac-fastapi.redisHost" -}}
+{{- if and (.Values.redis.enabled) (not (and .Values.redis.external .Values.redis.external.enabled)) -}}
+  {{- $service := include "stac-fastapi.redisServiceName" . | trim -}}
+  {{- $namespace := .Release.Namespace | default "default" -}}
+  {{- $clusterDomain := default "cluster.local" .Values.global.clusterDomain -}}
+  {{- printf "%s.%s.svc.%s" $service $namespace $clusterDomain -}}
+{{- else if and .Values.redis.external .Values.redis.external.enabled -}}
+  {{- .Values.redis.external.host | default "" -}}
+{{- else -}}
+  {{- "" -}}
+{{- end }}
+{{- end }}
+
+{{/*
+Resolve the Redis port based on the active configuration
+*/}}
+{{- define "stac-fastapi.redisPort" -}}
+{{- if and (.Values.redis.enabled) (not (and .Values.redis.external .Values.redis.external.enabled)) -}}
+  {{- $ports := default (dict) (dig "master" "service" "ports" (dict) .Values.redis) -}}
+  {{- $port := default 6379 (index $ports "redis") -}}
+  {{- $port -}}
+{{- else if and .Values.redis.external .Values.redis.external.enabled -}}
+  {{- .Values.redis.external.port | default 6379 -}}
+{{- else -}}
+  {{- 6379 -}}
+{{- end }}
+{{- end }}
+
 {{/*
 Create the image repository with tag
 */}}
@@ -186,6 +228,9 @@ Create environment variables for the application
 {{- $envFromSecret := default (dict) .Values.app.envFromSecret -}}
 {{- $dbAuth := deepCopy (default (dict) .Values.app.databaseAuth) -}}
 {{- $opensearchSecurity := default (dict) .Values.opensearchSecurity -}}
+{{- $redisConfig := default (dict) .Values.redis -}}
+{{- $redisExternal := default (dict) (get $redisConfig "external") -}}
+{{- $redisEnabled := or ($redisConfig.enabled) (and $redisExternal ($redisExternal.enabled)) -}}
 {{- if and (eq .Values.backend "opensearch") (get $opensearchSecurity "generateAdminPassword") }}
   {{- $secretName := include "stac-fastapi.opensearchAdminSecretName" . -}}
   {{- if or (not (hasKey $dbAuth "existingSecret")) (not $dbAuth.existingSecret) }}

helm-chart/stac-fastapi/values-elasticsearch.yaml

@@ -64,7 +64,7 @@ app:
     STAC_FASTAPI_DESCRIPTION: "High-performance STAC API for geospatial data discovery"
     ENVIRONMENT: "production"
     WEB_CONCURRENCY: "4"
-    ENABLE_DIRECT_RESPONSE: "true"  # Enable for maximum performance
+    ENABLE_DIRECT_RESPONSE: "false"
     DATABASE_REFRESH: "false"  # Better performance for bulk operations
     ENABLE_DATETIME_INDEX_FILTERING: "true"  # Enable for large datasets
     DATETIME_INDEX_MAX_SIZE_GB: "50"
@@ -130,6 +130,25 @@ opensearch:
 externalDatabase:
   enabled: false
 
+# Redis cache configuration for API response caching
+redis:
+  enabled: true
+  architecture: standalone
+  database: 0
+  master:
+    persistence:
+      enabled: true
+      size: 8Gi
+    resources:
+      requests:
+        cpu: 100m
+        memory: 256Mi
+      limits:
+        cpu: 500m
+        memory: 512Mi
+  auth:
+    enabled: false
+
 # Monitoring configuration
 monitoring:
   enabled: true

helm-chart/stac-fastapi/values-opensearch.yaml

@@ -64,7 +64,7 @@ app:
     STAC_FASTAPI_DESCRIPTION: "High-performance STAC API for geospatial data discovery with OpenSearch"
     ENVIRONMENT: "production"
     WEB_CONCURRENCY: "4"
-    ENABLE_DIRECT_RESPONSE: "true"  # Enable for maximum performance
+    ENABLE_DIRECT_RESPONSE: "false"
     DATABASE_REFRESH: "false"  # Better performance for bulk operations
     ENABLE_DATETIME_INDEX_FILTERING: "true"  # Enable for large datasets
     DATETIME_INDEX_MAX_SIZE_GB: "50"
@@ -75,7 +75,7 @@ app:
   # Optional: pull connection credentials from a secret managed by the
   # OpenSearch chart (or provide plain values via username/password).
   databaseAuth:
-    existingSecret: ""
+    existingSecret: "stac-opensearch-admin"
     usernameKey: "username"
     passwordKey: "password"
 
@@ -165,6 +165,25 @@ opensearch:
 externalDatabase:
   enabled: false
 
+# Redis cache configuration for API response caching
+redis:
+  enabled: true
+  architecture: standalone
+  database: 0
+  master:
+    persistence:
+      enabled: true
+      size: 8Gi
+    resources:
+      requests:
+        cpu: 100m
+        memory: 256Mi
+      limits:
+        cpu: 500m
+        memory: 512Mi
+  auth:
+    enabled: false
+
 # Monitoring configuration
 monitoring:
   enabled: true

helm-chart/stac-fastapi/values.yaml

@@ -358,6 +358,33 @@ externalDatabase:
   # API key field in the secret
   apiKeySecretKey: "api-key"
 
+# Redis cache configuration
+redis:
+  # Deploy an in-cluster Redis when enabled
+  enabled: false
+  # Optional override for the generated Redis fullname
+  fullnameOverride: ""
+  # Default Redis database index used by the application
+  database: 0
+  architecture: standalone
+  master:
+    service:
+      ports:
+        redis: 6379
+  auth:
+    enabled: false
+    # Provide an existing secret containing the Redis password when auth is enabled
+    existingSecret: ""
+    existingSecretPasswordKey: "redis-password"
+  # Configure connection details for an externally managed Redis instance
+  external:
+    enabled: false
+    host: ""
+    port: 6379
+    # Optional secret that stores the Redis password for the external instance
+    passwordSecret: ""
+    passwordKey: "redis-password"
+
 # Configuration for monitoring and observability
 monitoring:
   # Enable monitoring

helm-chart/test-chart.sh

@@ -95,6 +95,32 @@ maybe_collect_backend_logs() {
     fi
 }
 
+ensure_helm_repo() {
+    local name="$1"
+    local url="$2"
+
+    local existing_repos
+    existing_repos=$(helm repo list 2>/dev/null | awk 'NR>1 {print $1}' || true)
+
+    if ! grep -qx "$name" <<<"$existing_repos"; then
+        log_info "Adding Helm repository: $name ($url)"
+        helm repo add "$name" "$url"
+    fi
+}
+
+ensure_chart_dependencies() {
+    log_info "Ensuring required Helm repositories are configured..."
+    ensure_helm_repo elasticsearch https://helm.elastic.co
+    ensure_helm_repo opensearch https://opensearch-project.github.io/helm-charts/
+    ensure_helm_repo bitnami https://charts.bitnami.com/bitnami
+
+    log_info "Updating Helm repositories..."
+    helm repo update
+
+    log_info "Updating chart dependencies..."
+    helm dependency update "$CHART_PATH"
+}
+
 get_desired_app_replicas() {
     local values_path="$1"
     local default_count="${APP_REPLICA_COUNT:-}"
@@ -287,9 +313,7 @@ lint_chart() {
         exit 1
     fi
 
-    # Update dependencies
-    log_info "Updating chart dependencies..."
-    helm dependency update "$CHART_PATH"
+    ensure_chart_dependencies
 
     # Lint the chart
     helm lint "$CHART_PATH"
@@ -310,6 +334,8 @@ lint_chart() {
 # Run Helm chart tests
 test_chart() {
     log_info "Running Helm chart tests..."
+
+    ensure_chart_dependencies
 
     # Dry run installation
     log_info "Testing chart installation (dry run)..."
@@ -331,8 +357,7 @@ install_chart() {
     # Create namespace if it doesn't exist
     kubectl create namespace "$NAMESPACE" --dry-run=client -o yaml | kubectl apply -f -
 
-    # Update dependencies
-    helm dependency update "$CHART_PATH"
+    ensure_chart_dependencies
 
     # Select appropriate values file for backend
     local values_file=""
@@ -412,7 +437,7 @@ install_chart() {
 upgrade_chart() {
     log_info "Upgrading STAC FastAPI chart..."
 
-    helm dependency update "$CHART_PATH"
+    ensure_chart_dependencies
 
     # Select appropriate values file for backend
     local values_file=""