Some more debugging in skipUnhealthy.

pkg/gameservers/health.go

@@ -246,17 +246,23 @@ func (hc *HealthController) skipUnhealthy(gs *agonesv1.GameServer) (bool, error)
 	for _, cs := range pod.Status.ContainerStatuses {
 		if cs.Name == gs.Spec.Container {
 			if cs.State.Terminated != nil {
+				hc.baseLogger.WithField("gs", gs.ObjectMeta.Name).WithField("podStatus", pod.Status).Debug("skipUnhealthy: Container is terminated, returning false")
 				return false, nil
 			}
 			if cs.LastTerminationState.Terminated != nil {
 				// if the current container is running, and is the ready container, then we know this is some
 				// other pod update, and we previously had a restart before we got to being Ready, and therefore
 				// shouldn't move to Unhealthy.
-				return cs.ContainerID == gs.Annotations[agonesv1.GameServerReadyContainerIDAnnotation], nil
+				check := cs.ContainerID == gs.Annotations[agonesv1.GameServerReadyContainerIDAnnotation]
+				if !check {
+					hc.baseLogger.WithField("gs", gs.ObjectMeta.Name).WithField("gsMeta", gs.ObjectMeta).WithField("podStatus", pod.Status).Debug("skipUnhealthy: Container crashed after Ready, returning false")
+				}
+				return check, nil
 			}
 			break
 		}
 	}
 
+	hc.baseLogger.WithField("gs", gs.ObjectMeta.Name).WithField("gsMeta", gs.ObjectMeta).WithField("podStatus", pod.Status).Debug("skipUnhealthy: Should not reach here")
 	return false, nil
 }