fix: release (#196)

* fix: add checkout

* add retry/backoff

* fix: version

* fix: version

* fix: versions

Author: mishraomp

.github/scripts/get-metabase-version.sh

@@ -1,10 +1,37 @@
 #!/usr/bin/env bash
 set -euo pipefail
 
-app_version="$(yq '.metabase.metabaseImage.tag' charts/nr-metabase/values.yaml)"
+# Get Metabase version from values.yaml (the actual image tag)
+values_version="$(yq '.metabase.metabaseImage.tag' charts/nr-metabase/values.yaml)"
 
-if [ "$app_version" = "null" ]; then
-  app_version=""
+if [ "$values_version" = "null" ] || [ -z "$values_version" ]; then
+  echo "❌ ERROR: Failed to extract metabase.metabaseImage.tag from values.yaml" >&2
+  exit 1
 fi
 
-printf '%s\n' "$app_version"
+# Get appVersion from Chart.yaml
+chart_app_version="$(yq '.appVersion' charts/nr-metabase/Chart.yaml)"
+
+if [ "$chart_app_version" = "null" ] || [ -z "$chart_app_version" ]; then
+  echo "❌ ERROR: Failed to extract appVersion from Chart.yaml" >&2
+  exit 1
+fi
+
+# Normalize versions for comparison (both should have 'v' prefix)
+normalized_values="${values_version#v}"  # Remove leading 'v' if present
+normalized_chart="${chart_app_version#v}"  # Remove leading 'v' if present
+
+# Check for version mismatch
+if [ "$normalized_values" != "$normalized_chart" ]; then
+  echo "❌ VERSION MISMATCH DETECTED:" >&2
+  echo "   values.yaml (metabase.metabaseImage.tag): $values_version" >&2
+  echo "   Chart.yaml (appVersion): $chart_app_version" >&2
+  echo "" >&2
+  echo "🔧 ACTION REQUIRED:" >&2
+  echo "   Update Chart.yaml to match values.yaml:" >&2
+  echo "   1. Set 'version' to: $normalized_values" >&2
+  echo "   2. Set 'appVersion' to: $values_version" >&2
+  exit 1
+fi
+
+printf '%s\n' "$values_version"

.github/workflows/merge-main.yml

@@ -77,6 +77,8 @@ jobs:
         run: |
           APP_VERSION="$(bash .github/scripts/get-metabase-version.sh)"
           VERSION="${APP_VERSION#v}"
+          echo "APP_VERSION=$APP_VERSION"
+          echo "VERSION=$VERSION"
           helm package -u --destination=.cr-release-packages --app-version="$APP_VERSION" --version=$VERSION charts/${{ github.event.repository.name }}
 
       - name: Run chart-releaser

README.md

@@ -50,7 +50,7 @@ The stock Metabase image cannot talk to Oracle, and it certainly cannot talk to
                                                  │
                  ┌─────────────────────────────────────────────────────────┐
                  │  Metabase Pod (custom image)                            │
-                 │   • metabase.jar (v0.61.1) on Temurin 25                │
+                 │   • metabase.jar (vx.x.x) on Temurin 25                │
                  │   • ojdbc8-full Oracle driver in /plugins               │
                  │   • run_app.sh imports Oracle TLS certs → JVM cacerts   │
                  │   • log4j2 config mounted from ConfigMap                │
@@ -133,7 +133,7 @@ All values live in `charts/nr-metabase/values.yaml` and are validated against `v
 |---|---|---|
 | `metabase.enabled` | `true` | Toggle the Metabase component. |
 | `metabase.replicaCount` | `1` | Pod replicas (Metabase is not horizontally scalable by default). |
-| `metabase.metabaseImage.tag` | `v0.61.1` | Metabase version deployed by the pod. |
+| `metabase.metabaseImage.tag` | `vx.x.x` | Metabase version deployed by the pod. |
 | `metabase.dbHostPortEnv` | `~` | Comma-separated `host:port` list of Oracle endpoints whose TLS certs are imported at startup. |
 | `metabase.service.port` / `targetPort` | `80` / `3000` | Service port mapping. |
 | `metabase.resources.requests` | `250m` CPU / `1200Mi` | Resource requests. |

charts/nr-metabase/Chart.yaml

@@ -13,12 +13,12 @@ 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.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.48.7
+version: 0.62.4
 # 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: "v0.48.7" # aligned with metabase version. ALso update the global values appversion, there is no way to pass to
+appVersion: "v0.62.4" # aligned with metabase version in values.yaml
 dependencies:
   - name: database
     condition: database.enabled

charts/nr-metabase/values.yaml

@@ -25,7 +25,7 @@ metabase:
   # Track the upstream Metabase image tag used by the deployment template.
   metabaseImage:
     repository: metabase/metabase
-    tag: v0.62.4.3
+    tag: v0.62.4
   containerPort: 3000
   environment: production
   service: