Skip to content

Remove OS suffix from connect-content-init image tag#882

Open
bschwedler wants to merge 1 commit into
mainfrom
content-init-os-agnostic
Open

Remove OS suffix from connect-content-init image tag#882
bschwedler wants to merge 1 commit into
mainfrom
content-init-os-agnostic

Conversation

@bschwedler
Copy link
Copy Markdown
Contributor

The connect-content-init image copies an architecture-specific runtime binary and is not OS-specific. Simplify the tag to use the chart appVersion directly (e.g. 2026.05.1) rather than 2026.05.1-ubuntu-24.04.

  • Remove os field from backends.kubernetes.defaultInitContainer and launcher.defaultInitContainer
  • Update tag construction in _helpers.tpl and configmap.yaml
  • Remove duplicate test that asserted the old OS-suffixed tag format

Any os: entries in existing values.yaml files can be removed, but are harmlessly ignored.

@bschwedler bschwedler requested a review from a team June 4, 2026 17:27
bschwedler added a commit to posit-dev/images-connect that referenced this pull request Jun 4, 2026
The init container copies an architecture-specific runtime binary;
the OS variant has no effect on what gets downloaded. Remove the
Ubuntu 22.04 secondary entries from the versions where Ubuntu 24.04
is already primary (2025.12.1+), and drop the ubuntu2204 template
and rendered Containerfiles for those versions.

Older versions (2025.11 and earlier) that only have Ubuntu 22.04
are unchanged.

Related: rstudio/helm#882
@bschwedler bschwedler force-pushed the content-init-os-agnostic branch 2 times, most recently from 12ee0d0 to d2f2843 Compare June 4, 2026 18:38
@bschwedler bschwedler force-pushed the content-init-os-agnostic branch from d2f2843 to 70be48b Compare June 4, 2026 19:05
The connect-content-init image copies an architecture-specific binary
and is not OS-specific. Remove the `os` field from
`backends.kubernetes.defaultInitContainer` and
`launcher.defaultInitContainer`, and update tag construction to use
the appVersion directly (e.g. `2026.05.1` rather than
`2026.05.1-ubuntu-24.04`).

Any `os:` entries in existing values.yaml files can be removed.
@bschwedler bschwedler force-pushed the content-init-os-agnostic branch from 70be48b to 6fd1047 Compare June 4, 2026 19:40
@bschwedler bschwedler requested a review from jforest June 5, 2026 16:26
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.

1 participant