Skip to content

fix(helm): propagate supervisor image overrides#2216

Merged
TaylorMutch merged 1 commit into
mainfrom
2068-helm-supervisor-image-defaults/tm
Jul 10, 2026
Merged

fix(helm): propagate supervisor image overrides#2216
TaylorMutch merged 1 commit into
mainfrom
2068-helm-supervisor-image-defaults/tm

Conversation

@TaylorMutch

Copy link
Copy Markdown
Collaborator

Summary

Let the gateway use its version-pinned built-in supervisor image for the default Helm configuration while still propagating Helm image overrides into gateway.toml.

Related Issue

Follow-up to #2068 and #2070.

Changes

  • Omit supervisor_image from the rendered gateway configuration when the standard repository and empty tag are unchanged.
  • Propagate custom supervisor repositories and explicit tags to the Kubernetes driver settings.
  • Resolve repository-only overrides with the effective gateway image tag and tag-only overrides with the official repository.
  • Add Helm unit coverage for default, repository-only, tag-only, and complete overrides.
  • Update chart, deployment, and architecture documentation.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (not applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Signed-off-by: Taylor Mutch <taylormutch@gmail.com>
@copy-pr-bot

copy-pr-bot Bot commented Jul 10, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions

Copy link
Copy Markdown

@TaylorMutch TaylorMutch added the test:e2e Requires end-to-end coverage label Jul 10, 2026
@TaylorMutch TaylorMutch marked this pull request as ready for review July 10, 2026 18:41
@github-actions

Copy link
Copy Markdown

Label test:e2e applied for ecf67ce. Open Branch E2E Checks, find the run for commit ecf67ce, and click Re-run all jobs to execute with the label set. The run will execute the standard E2E suite after building the required gateway and supervisor images once. The matching required CI gate status on this PR will flip green automatically once the run finishes.

@TaylorMutch

Copy link
Copy Markdown
Collaborator Author

CC @benoitf as a followup to your PR

@benoitf

benoitf commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Ok thanks @TaylorMutch

@TaylorMutch TaylorMutch merged commit bebf440 into main Jul 10, 2026
77 of 80 checks passed
@TaylorMutch TaylorMutch deleted the 2068-helm-supervisor-image-defaults/tm branch July 10, 2026 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:e2e Requires end-to-end coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants