Skip to content

[Backport release-1.4] fix(kubernetes): add spec.timeout to tenant CSI HelmRelease - #2929

Merged
Andrei Kvapil (kvaps) merged 1 commit into
release-1.4from
backport-2727-to-release-1.4
Jun 24, 2026
Merged

Andrei Kvapil (kvaps) merged 1 commit into
release-1.4from
backport-2727-to-release-1.4

Conversation

@github-actions

Copy link
Copy Markdown

Description

Backport of #2727 to release-1.4.

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. 9f9d8f8 ("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 <[email protected]>
(cherry picked from commit 866588f)
(cherry picked from commit d8ba954)
@dosubot dosubot Bot added area/kubernetes Issues or PRs related to the tenant Kubernetes app kind/backport Categorizes issue or PR as requiring a backport to the current release line kind/bug Categorizes issue or PR as related to a bug labels Jun 16, 2026
@kvaps
Andrei Kvapil (kvaps) merged commit 30edb8f into release-1.4 Jun 24, 2026
7 checks passed
@kvaps
Andrei Kvapil (kvaps) deleted the backport-2727-to-release-1.4 branch June 24, 2026 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/kubernetes Issues or PRs related to the tenant Kubernetes app kind/backport Categorizes issue or PR as requiring a backport to the current release line kind/bug Categorizes issue or PR as related to a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants