Merge pull request #277 from jgehrcke/jp/v25.3.0-rc.1

Set version to 25.3.0-rc.1

Author: jgehrcke

deployments/container/Makefile

@@ -27,6 +27,10 @@ ifeq ($(IMAGE_NAME),)
 IMAGE_NAME := $(REGISTRY)/$(DRIVER_NAME)
 endif
 
+# Note: this is sometimes a v-prefixed version string
+# (in case of proper releases) or sometimes a commit hash
+# (in case of regular CI builds) in which case VERSION
+# is not v-prefixed.
 IMAGE_VERSION := $(VERSION)
 
 IMAGE_TAG ?= $(IMAGE_VERSION)-$(DIST)

deployments/helm/nvidia-dra-driver-gpu/Chart.yaml

@@ -15,10 +15,15 @@ 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: v25.3.0
+
+# Note(JP): no v prefix allowed
+version: 25.3.0-rc.1
 
 # 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: "v25.3.0"
+
+# Note(JP): this currently defines the default `tag` value in a k8s
+# image specification, and therefore must currently be v-prefixed.
+appVersion: "v25.3.0-rc.1"

deployments/helm/nvidia-dra-driver-gpu/values.yaml

@@ -39,7 +39,9 @@ imagePullSecrets: []
 image:
   repository: nvcr.io/nvidia/cloud-native/k8s-dra-driver-gpu
   pullPolicy: IfNotPresent
-  # Overrides the image tag whose default is the chart appVersion.
+  # Note: an empty string is translated to the `appVersion` string from
+  # the Helm chart YAML (effectively implementing the default value to be
+  # the current version).
   tag: ""
 
 serviceAccount:

hack/package-helm-charts.sh

@@ -19,10 +19,19 @@ set -o pipefail
 # if arg1 is set, it will be used as the version number
 if [ -z "$1" ]; then
   VERSION=$(awk -F= '/^VERSION/ { print $2 }' versions.mk | tr -d '[:space:]')
+  # Remove any v prefix, if exists.
+  VERSION="${VERSION#v}"
 else
   VERSION=$1
 fi
 VERSION=${VERSION}
 
+
+# Note(JP): the goal below is for VERSION to always be
+# strictly semver-compliant (parseable with a semver
+# parser). That enables best compatibility with the Helm
+# ecosystem. For example, that means that no `v` prefix
+# should be used here.
+
 # Create release assets to be uploaded
 helm package deployments/helm/nvidia-dra-driver-gpu/ --version $VERSION --app-version $VERSION

versions.mk

@@ -18,7 +18,7 @@ MODULE := github.com/NVIDIA/$(DRIVER_NAME)
 
 REGISTRY ?= nvcr.io/nvidia/cloud-native
 
-VERSION  ?= v25.3.0
+VERSION  ?= v25.3.0-rc.1
 
 # vVERSION represents the version with a guaranteed v-prefix
 vVERSION := v$(VERSION:v%=%)