use latest tag to fetch controller image (#79)

resolves Issue: #74 

instead of pinning to a fix version, use latest tag to fetch the correct
image and that should also update in workflow

---------

Signed-off-by: ankittk <ankit.kumar@getweave.com>

Author: ankittk

.github/workflows/tag.yaml

@@ -51,6 +51,10 @@ jobs:
         fi
         echo "Building Docker image with version: ${VERSION}"
         make docker-build VERSION=${VERSION}
+        
+        # Also tag and push as 'latest'
+        echo "Tagging and pushing as 'latest'..."
+        make docker-tag-latest VERSION=${VERSION}
   push-helm-chart:
     needs:
     - push-images

Makefile

@@ -85,15 +85,12 @@ helm-package: manifests
 	mkdir -p $(DIST_FOLDER)
 	@echo "Packaging Helm chart with version $(VERSION)..."
 	@cp helm/kmcp/Chart.yaml helm/kmcp/Chart.yaml.bak
-	@sed "s/^version: .*/version: $(VERSION)/" helm/kmcp/Chart.yaml.bak > helm/kmcp/Chart.yaml
+	@awk '/^version:/ { print; print "appVersion: \"$(VERSION)\""; next } { print }' helm/kmcp/Chart.yaml.bak > helm/kmcp/Chart.yaml
 	@helm package helm/kmcp --version $(VERSION) -d $(DIST_FOLDER)
 	@mv helm/kmcp/Chart.yaml.bak helm/kmcp/Chart.yaml
 	@echo "Helm package created: $(DIST_FOLDER)/kmcp-$(VERSION).tgz"
 	@cp config/crd/bases/kagent.dev_mcpservers.yaml helm/kmcp-crds/templates/mcpserver-crd.yaml
-	@cp helm/kmcp-crds/Chart.yaml helm/kmcp-crds/Chart.yaml.bak
-	@sed "s/^version: .*/version: $(VERSION)/" helm/kmcp-crds/Chart.yaml.bak > helm/kmcp-crds/Chart.yaml
 	@helm package helm/kmcp-crds --version $(VERSION) -d $(DIST_FOLDER)
-	@mv helm/kmcp-crds/Chart.yaml.bak helm/kmcp-crds/Chart.yaml
 	@echo "Helm package created: $(DIST_FOLDER)/kmcp-crds-$(VERSION).tgz"
 
 .PHONY: helm-cleanup
@@ -240,6 +237,14 @@ docker-build: ## Build docker image with the manager.
 	$(DOCKER_BUILDER) build $(DOCKER_BUILD_ARGS) -t ${CONTROLLER_IMG} .
 	- $(DOCKER_BUILDER) rm $(BUILDX_BUILDER_NAME)
 
+.PHONY: docker-tag-latest
+docker-tag-latest: ## Tag the built image as 'latest' and push it
+	@echo "Tagging image as 'latest'..."
+	@docker tag ${CONTROLLER_IMG} $(DOCKER_REGISTRY)/$(DOCKER_REPO)/$(CONTROLLER_IMAGE_NAME):latest
+	@echo "Pushing 'latest' tag..."
+	@docker push $(DOCKER_REGISTRY)/$(DOCKER_REPO)/$(CONTROLLER_IMAGE_NAME):latest
+	@echo "Latest tag pushed successfully"
+
 .PHONY: docker-push
 docker-push: ## Push docker image with the manager.
 	$(DOCKER_BUILDER) push ${CONTROLLER_IMG}

helm/kmcp-crds/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: kmcp-crds
 description: CRDs for KMCP (Kubernetes MCP Server Controller)
 type: application
-version: 0.1.0
+version: 1.0.0
 home: https://github.com/kagent-dev/kmcp
 sources:
   - https://github.com/kagent-dev/kmcp

helm/kmcp/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: kmcp
 description: A Helm chart for KMCP (Kubernetes MCP Server Controller)
 type: application
-version: 0.1.0
+version: 1.0.0
 home: https://github.com/kagent-dev/kmcp
 sources:
   - https://github.com/kagent-dev/kmcp

helm/kmcp/templates/_helpers.tpl

@@ -65,7 +65,7 @@ Allows overriding it for multi-namespace deployments in combined charts.
 Create the image reference
 */}}
 {{- define "kmcp.image" -}}
-{{- $tag := .Values.image.tag | default .Chart.Version }}
+{{- $tag := .Values.image.tag | default .Chart.AppVersion | default "latest" }}
 {{- printf "%s:%s" .Values.image.repository $tag }}
 {{- end }}
 

helm/kmcp/values.yaml

@@ -13,8 +13,9 @@ namespaceOverride: ""
 image:
   repository: ghcr.io/kagent-dev/kmcp/controller
   pullPolicy: IfNotPresent
-  # Overrides the image tag whose default is the chart appVersion.
-  tag: "v0.1.0"
+  # Overrides the image tag whose default is the chart's appVersion.
+  # Set to a specific version (e.g., "v0.1.7") to pin to that version.
+  # tag: ""
 
 # Image pull secrets for private registries
 imagePullSecrets: []