1
38%
main: 38%

Ran 08 Apr 2026 08:51AM UTC

Files 26

Run time 1s

Badge

Embed ▾

Committed 08 Apr 2026 08:48AM UTC coverage: 37.939% (+0.07%) from 37.871%

Job # 24126594272.1

Build Type

Pull #382

github

Committed by

qinqon

Commit Message

fix: clear FailureReason/FailureMessage when VM recovers from terminal state

When a KubeVirt VM is stopped (runStrategy changed to Halted), the VMI
reaches a finalized state and IsTerminal() correctly sets FailureReason
on the KubevirtMachine. However, when the VM is started back
(runStrategy changed to Always), IsTerminal() returns false but the
FailureReason/FailureMessage fields are never cleared.

This causes the CAPI Machine phase to stay stuck at "Failed" even though
all Machine conditions (Ready, InfrastructureReady, NodeHealthy) are
True. Downstream consumers that rely on the Machine phase (such as
HyperShift's endpointslice reconciler) then permanently mark endpoints
as not ready, breaking ingress into hosted clusters.

The fix adds an else clause to nil out FailureReason and FailureMessage
when IsTerminal() returns false, allowing VMs to properly recover.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>

Pull Request Pull Request #382: fix: clear FailureReason/FailureMessage when VM recovers from terminal state

Coverage Stats

1049 of 2765 relevant lines covered (37.94%)

2.14 hits per line

kubernetes-sigs / cluster-api-provider-kubevirt / 24126594272 / 1
38%
main: 38%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24126594272.1

kubernetes-sigs / cluster-api-provider-kubevirt / 24126594272 / 1 38% main: 38%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24126594272.1

kubernetes-sigs / cluster-api-provider-kubevirt / 24126594272 / 1
38%
main: 38%

README BADGES
x