From 87a7d7c70a762f0b877444bf3497b7e0b821f579 Mon Sep 17 00:00:00 2001 From: Codex Date: Fri, 8 May 2026 14:12:07 -0500 Subject: [PATCH] fix(ci1): switch deprecated `running: true` -> `runStrategy: Always` Required to clear OutOfSync state after the bootOrder fix. Live VM had runStrategy: Halted (set during diagnosis to release the PVC for inspection). Manifest had running: true. KubeVirt's validating webhook rejects sync: admission webhook "virtualmachine-validator.kubevirt.io" denied the request: Running and RunStrategy are mutually exclusive. Switching to runStrategy: Always preserves the original "auto-start + auto-restart" semantics with the non-deprecated field, and gives ArgoCD a clean diff target to flip Halted -> Always. Co-Authored-By: Claude Opus 4.7 (1M context) --- apps/kubevirt-vms/ci1.yaml | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/apps/kubevirt-vms/ci1.yaml b/apps/kubevirt-vms/ci1.yaml index d3ed7e7..1a8daac 100644 --- a/apps/kubevirt-vms/ci1.yaml +++ b/apps/kubevirt-vms/ci1.yaml @@ -283,7 +283,13 @@ metadata: role: github-actions-runner flowercore.io/managed-by: bluejay-infra spec: - running: true # LIVE — ISO uploaded 2026-05-08, password in 1P + # `running: true` is deprecated in favor of `runStrategy`. They are mutually + # exclusive — KubeVirt's validating webhook rejects any VM that sets both: + # admission webhook "virtualmachine-validator.kubevirt.io" denied the request: + # Running and RunStrategy are mutually exclusive. + # `Always` keeps a VMI running and restarts it if it crashes/exits — same + # semantics as the old `running: true`. + runStrategy: Always # LIVE — ISO uploaded 2026-05-08, password in 1P template: metadata: labels: