Skip to content

c8d: review uses of platforms.Format vs platforms.FormatAll #48659

@thaJeztah

Description

@thaJeztah

It occurred to me that we're using platforms.Format() here, and not platforms.FormatAll(). The latter also includes "os-version" if it's set. While we don't use that / document that (yet) in the CLI documentation, it may become more relevant with Windows images.

I don't think it's problematic to have it here, but I'll open a tracking ticket to make a pass at looking for places where we could replace Format with FormatAll.

Originally posted by @thaJeztah in #48631 (comment)

Metadata

Metadata

Assignees

Labels

area/uxcontainerd-integrationIssues and PRs related to containerd integrationkind/enhancementEnhancements are not bugs or new features but can improve usability or performance.

Type

Projects

Status

Done

Status

Required for default containerd

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions