Skip to content

[release/0.14]: skip SandboxPlatform validation when platform is not explicitly set#2665

Merged
anmaxvl merged 2 commits intomicrosoft:release/0.14from
anmaxvl:backports-to-release-0.14
Apr 8, 2026
Merged

[release/0.14]: skip SandboxPlatform validation when platform is not explicitly set#2665
anmaxvl merged 2 commits intomicrosoft:release/0.14from
anmaxvl:backports-to-release-0.14

Conversation

@anmaxvl
Copy link
Copy Markdown
Contributor

@anmaxvl anmaxvl commented Apr 7, 2026

When runtime options are non-empty (e.g., SandboxIsolation is set) but SandboxPlatform is empty, skip the platform validation rather than failing. The validation only needs to check that the spec and shim options match when SandboxPlatform is explicitly configured.

containerd's default config (config_windows.go) sets SandboxIsolation=1 for the runhcs-wcow-hypervisor runtime handler but omits SandboxPlatform, making options non-empty with an empty platform string. This causes platforms.Parse("") to fail with 'invalid runtime sandbox platform'.

(cherry picked from commit 88c818f)

… set (microsoft#2620)

When runtime options are non-empty (e.g., SandboxIsolation is set) but
SandboxPlatform is empty, skip the platform validation rather than
failing. The validation only needs to check that the spec and shim
options match when SandboxPlatform is explicitly configured.

containerd's default config (config_windows.go) sets SandboxIsolation=1
for the runhcs-wcow-hypervisor runtime handler but omits SandboxPlatform,
making options non-empty with an empty platform string. This causes
platforms.Parse("") to fail with 'invalid runtime sandbox platform'.

Signed-off-by: Dawei Wei <wei.dawei.cn@gmail.com>
(cherry picked from commit 88c818f)
@anmaxvl anmaxvl requested a review from a team as a code owner April 7, 2026 19:35
@anmaxvl anmaxvl force-pushed the backports-to-release-0.14 branch from c213645 to d6b1549 Compare April 7, 2026 21:53
Signed-off-by: Maksim An <maksiman@microsoft.com>
@anmaxvl anmaxvl force-pushed the backports-to-release-0.14 branch from d6b1549 to 66504bd Compare April 7, 2026 21:53
@anmaxvl anmaxvl merged commit fb5aa2e into microsoft:release/0.14 Apr 8, 2026
93 of 101 checks passed
@anmaxvl anmaxvl deleted the backports-to-release-0.14 branch April 8, 2026 16:33
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.

3 participants