Fix NPE in virtualmachine.Status.RestartCount (#819)

neondatabase · Feb 21, 2024 · 054eb6e · 054eb6e
1 parent b7243b0
commit 054eb6e
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/neonvm/controllers/virtualmachine_controller.go b/neonvm/controllers/virtualmachine_controller.go
@@ -787,7 +787,11 @@ func (r *VirtualMachineReconciler) doReconcile(ctx context.Context, virtualmachi
 			if shouldRestart {
 				log.Info("Restarting VM runner pod", "VM.Phase", virtualmachine.Status.Phase, "RestartPolicy", virtualmachine.Spec.RestartPolicy)
 				virtualmachine.Status.Phase = vmv1.VmPending // reset to trigger restart
-				*virtualmachine.Status.RestartCount += 1     // increment restart count
+				if virtualmachine.Status.RestartCount == nil {
+					var zero int32 = 0
+					virtualmachine.Status.RestartCount = &zero
+				}
+				*virtualmachine.Status.RestartCount += 1 // increment restart count
 				r.Metrics.vmRestartCounts.Inc()
 			}