Skip to content

fix(helm): remove hardcoded kmcp tag image #1322

Merged
EItanya merged 2 commits intokagent-dev:mainfrom
jmhbh:fix/remove-kmcp-hardcoded-tag
Feb 17, 2026
Merged

fix(helm): remove hardcoded kmcp tag image #1322
EItanya merged 2 commits intokagent-dev:mainfrom
jmhbh:fix/remove-kmcp-hardcoded-tag

Conversation

@jmhbh
Copy link
Contributor

@jmhbh jmhbh commented Feb 17, 2026

Remove the hardcoded kmcp.image.tag value from the default helm values. The issue in kmcp has been resolved and we now use the kmcp image tag set by KMCP_VERSION.

Tested locally by running helm install with the updated values.yaml.

❯ k get deploy -n kagent kagent-kmcp-controller-manager -oyaml | grep image
        image: ghcr.io/kagent-dev/kmcp/controller:0.2.6
        imagePullPolicy: IfNotPresent

Signed-off-by: JM Huibonhoa <jm.huibonhoa@solo.io>
Copilot AI review requested due to automatic review settings February 17, 2026 22:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the hardcoded kmcp image tag from the helm chart values, allowing the kmcp dependency to use its default versioning mechanism. The change eliminates a temporary workaround that was needed until a kmcp issue was resolved (referenced as kagent-dev/kmcp#74).

Changes:

  • Removed hardcoded kmcp.image.tag: "0.2.2" from helm/kagent/values.yaml
  • Removed associated comment about the workaround and chart values reference

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jmhbh jmhbh changed the title fix(helm): remove hardcoded tag image now defaults to latest image tag fix(helm): remove hardcoded kmcp tag image Feb 17, 2026
Signed-off-by: JM Huibonhoa <jm.huibonhoa@solo.io>
@EItanya EItanya merged commit 0969344 into kagent-dev:main Feb 17, 2026
28 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants