fix: ensure we set the new PodIPs if they are present

lacroixthomas · lacroixthomas · commit 329bf8129655 · 2025-01-25T12:13:58.000-05:00
diff --git a/pkg/gameservers/controller.go b/pkg/gameservers/controller.go
@@ -951,7 +951,7 @@ func (c *Controller) syncGameServerRequestReadyState(ctx context.Context, gs *ag
 		}
 	}
 	addressPopulated := false
-	if gs.Status.NodeName == "" || !hasPodIPAddress {
+	if gs.Status.NodeName == "" || (!hasPodIPAddress && len(pod.Status.PodIPs) > 0) {
 		addressPopulated = true
 		if pod.Spec.NodeName == "" {
 			return gs, workerqueue.NewTraceError(errors.Errorf("node not yet populated for Pod %s", pod.ObjectMeta.Name))

Original file line number	Diff line number	Diff line change
`@@ -951,7 +951,7 @@ func (c Controller) syncGameServerRequestReadyState(ctx context.Context, gs ag`
`951`	`951`	`}`
`952`	`952`	`}`
`953`	`953`	`addressPopulated := false`
`954`		`- if gs.Status.NodeName == "" \|\| !hasPodIPAddress {`
	`954`	`+ if gs.Status.NodeName == "" \|\| (!hasPodIPAddress && len(pod.Status.PodIPs) > 0) {`
`955`	`955`	`addressPopulated = true`
`956`	`956`	`if pod.Spec.NodeName == "" {`
`957`	`957`	`return gs, workerqueue.NewTraceError(errors.Errorf("node not yet populated for Pod %s", pod.ObjectMeta.Name))`