Skip to content

Revert "Revert "fix: stop overriding team-operator image when not explicitly configured""#166

Merged
amdove merged 4 commits intomainfrom
revert-157-revert-155-pin-operator-image
Mar 12, 2026
Merged

Revert "Revert "fix: stop overriding team-operator image when not explicitly configured""#166
amdove merged 4 commits intomainfrom
revert-157-revert-155-pin-operator-image

Conversation

@amdove
Copy link
Collaborator

@amdove amdove commented Mar 5, 2026

Re-applies #155: When team_operator_image is not set in ptd.yaml, PTD now passes no image override to Helm, allowing the chart to use its appVersion default. This also updates the default chart version to v1.16.2.

Previously, even when team_operator_chart_version was pinned, PTD always overrode the operator image to posit/ptd-team-operator:latest. This meant chart versions controlled Helm values but not the actual operator image running in clusters.

Changes

  1. Stop overriding team-operator image — When team_operator_image is not explicitly set, PTD no longer passes image configuration to Helm, allowing the chart's appVersion to control the image tag.

  2. Update default chart version to v1.16.2 — All clusters without an explicit team_operator_chart_version will use v1.16.2, which includes the fixes from posit-dev/team-operator#108.

Prerequisites (completed):

What changes for existing clusters

After merging this PR and running ptd ensure, clusters will:

  • Use chart version v1.16.2 by default (unless explicitly pinned to another version)
  • Pull the image version matching their chart's appVersion, instead of always pulling posit/ptd-team-operator:latest

IMPORTANT: Chart versions before v1.16.2 should not be used. Those versions have two bugs that cause ImagePullBackOff when PTD doesn't override the image. Both issues were fixed in PR #108.

@amdove amdove requested a review from a team as a code owner March 5, 2026 22:47
@amdove amdove requested a review from ian-flores March 6, 2026 17:18
ian-flores
ian-flores previously approved these changes Mar 9, 2026
@amdove amdove requested a review from ian-flores March 11, 2026 21:49
@amdove amdove added this pull request to the merge queue Mar 12, 2026
Merged via the queue into main with commit 2613ea0 Mar 12, 2026
4 checks passed
@amdove amdove deleted the revert-157-revert-155-pin-operator-image branch March 12, 2026 14:58
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.

2 participants