From 023dae3dd94e84c8ca6ea273e579f5bf821d289c Mon Sep 17 00:00:00 2001 From: Myasnikov Daniil Date: Wed, 20 May 2026 15:48:56 +0500 Subject: [PATCH] fix(kubernetes): add spec.timeout to tenant CSI HelmRelease MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Aligns kubevirt-csi-node HR with every other tenant addon HR in packages/apps/kubernetes/templates/helmreleases/ (cilium, coredns, ingress-nginx, etc.) — all set timeout: 10m, csi.yaml was the only exception. 9f9d8f85 ("Allign timeouts for HelmReleases") missed it. Symptom: e2e NFS test (hack/e2e-apps/run-kubernetes.sh) times out waiting for nfs-test-pod to reach Succeeded. Cozyreport events show the tenant csi HR running Install -> Uninstall -> Install remediation cycles. With install.remediation.retries: -1 and the default 5m Helm timeout, kstatus marks the kubevirt-csi-node DaemonSet 'InProgress' past 5m on a slow tenant control-plane bringup (Kamaji APIs flapping, VM image pulls), helm-controller times out, the remediation uninstalls the DaemonSet, then reinstalls. Any volume mount in flight at that moment is wedged. Observed on PR #2619 CI run 26142986969: csi HR remediation at ~07:07 happened ~10 min into the nfs-test-pod wait, killing the mount and timing out the pod-Succeeded condition. 10m matches the surrounding HRs and gives the DaemonSet headroom without changing remediation semantics on truly-broken installs. Signed-off-by: Myasnikov Daniil (cherry picked from commit 866588ff613a0c452a8b939695d596fb5034568d) (cherry picked from commit d8ba954e9081f83f771b7819c8ed220949396fe7) --- packages/apps/kubernetes/templates/helmreleases/csi.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/packages/apps/kubernetes/templates/helmreleases/csi.yaml b/packages/apps/kubernetes/templates/helmreleases/csi.yaml index 109d78e13..690a05f34 100644 --- a/packages/apps/kubernetes/templates/helmreleases/csi.yaml +++ b/packages/apps/kubernetes/templates/helmreleases/csi.yaml @@ -9,6 +9,7 @@ metadata: sharding.fluxcd.io/key: tenants spec: interval: 5m + timeout: 10m releaseName: csi chartRef: kind: ExternalArtifact