Skip to content

[deckhouse-cli] Platform-aware image pulls from RPP#390

Merged
ldmonster merged 1 commit into
mainfrom
feat/multiplatform-rpp-request
Jun 25, 2026
Merged

[deckhouse-cli] Platform-aware image pulls from RPP#390
ldmonster merged 1 commit into
mainfrom
feat/multiplatform-rpp-request

Conversation

@Glitchy-Sheep

Copy link
Copy Markdown
Contributor

Important

🧩 Stacked PR · d8 self-update & plugins · 2 / 3 · Multiplatform support
Merge order:

  1. Update - [deckhouse-cli] Plugins / d8 self-update with requirements check #386 ✅ merged
  2. Multiplatform support - this PR (replaces [deckhouse-cli] Platform-aware image pulls from RPP #387, which was accidentally merged into the feat/cli-and-plugins-autoupdate feature branch instead of main)
  3. Direct manifest download - [deckhouse-cli] Fetch plugin contracts from manifest annotation #388

Proxy side: deckhouse/deckhouse#20795 · deckhouse/deckhouse#20800

Summary

d8 now pulls the binary for the current running machine architecture and os.
So multi-platform images handled correctly (d8 and plugins).

How it works

  • Binary downloads use GET /v1/images/<image>/images/<version>?platform=<os>-<arch>. The value is the running GOOS-GOARCH joined with a dash, for example linux-amd64.
  • The proxy splits it back into OS and arch.
  • Proxy resolves the platform to the specific image from a multi-platform image index.

Before / After

Before: a pull addressed the image by tag and got the registry's default platform.
The CLI did not resolve a multi-platform image per arch.

After: a pull names its platform and gets that build, or a 404 when the image has no such platform.

Notes

- Image downloads move to `/<image>/images/<version>`; `PullImage` sends
  `?platform=<os>-<arch>`, so the proxy picks the matching child manifest of a
  multi-platform index. The dash keeps the value a single unescaped URL token.
- Releases ship as one multi-platform index per version tag, so `ListTags` and
  `ExtractBinary` work with plain version tags.
- A proxy too old to honor the query serves its default platform (linux/amd64).

Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
@Glitchy-Sheep Glitchy-Sheep requested a review from ldmonster as a code owner June 25, 2026 10:40
@ldmonster ldmonster merged commit 2522ccb into main Jun 25, 2026
5 checks passed
@ldmonster ldmonster deleted the feat/multiplatform-rpp-request branch June 25, 2026 10:53
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