Skip to content

helm: bump bundled oauth2-proxy subchart (7.0.0 → 10.x) — image tag override is mangled #2239

Description

@elihai-risk

The kagent chart bundles oauth2-proxy chart 7.0.0 (appVersion 7.6.0). That version's deployment template hardcodes a v prefix on the image tag:

image: "{{ .Values.image.repository }}:v{{ include "oauth2-proxy.version" . }}"

so any custom image.tag that doesn't follow the vX.Y.Z convention gets mangled — e.g. a private-registry mirror tag my-repo/kagent:oauth2-proxy-v7.6.0 renders as :voauth2-proxy-v7.6.0 (ImagePullBackOff). Many orgs must mirror images into a private registry (admission policies forbid quay.io), so tag overrides are common.

Upstream oauth2-proxy/manifests already fixed this — current chart (10.7.0) renders the tag via the version helper without the hardcoded prefix:

image: "{{ tpl .Values.image.registry $ | default ... }}/{{ .Values.image.repository }}:{{ include "oauth2-proxy.version" . }}"

Bumping the bundled subchart to 10.x would fix this and pick up ~3 years of fixes. Happy to open a PR if maintainers agree on the target version / values migration.

Context: we hit this deploying kagent 0.9.11 on EKS with ECR-mirrored images and had to patch the vendored subchart. Related PRs from the same deployment: #2237 (pod labels), #2238 (UI Ingress).

🤖 Generated with Claude Code

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    Status
    Backlog

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions