fix: panic on shutdown

Fixes #9017 Don't assume the config is there before trying to access it. Signed-off-by: Andrey Smirnov <[email protected]>
siderolabs · Jul 16, 2024 · d983e44 · d983e44
1 parent 01404ed
commit d983e44
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/internal/app/machined/pkg/runtime/v1alpha1/v1alpha1_sequencer.go b/internal/app/machined/pkg/runtime/v1alpha1/v1alpha1_sequencer.go
@@ -377,11 +377,13 @@ func (*Sequencer) Reset(r runtime.Runtime, in runtime.ResetOptions) []runtime.Ph
 
 // Shutdown is the shutdown sequence.
 func (*Sequencer) Shutdown(r runtime.Runtime, in *machineapi.ShutdownRequest) []runtime.Phase {
+	skipNodeRegistration := r.Config() != nil && r.Config().Machine() != nil && r.Config().Machine().Kubelet().SkipNodeRegistration()
+
 	phases := PhaseList{}.Append(
 		"storeShudown",
 		StoreShutdownEmergency,
 	).AppendWhen(
-		!in.GetForce() && !r.Config().Machine().Kubelet().SkipNodeRegistration(),
+		!in.GetForce() && !skipNodeRegistration,
 		"drain",
 		CordonAndDrainNode,
 	).Append(