Implmemnted Feature Windows Gameserver Move Alpha to beta by Sivasankaran25 · Pull Request #4560 · agones-dev/agones

Sivasankaran25 · 2026-05-11T09:01:05Z

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug
/kind cleanup
/kind documentation

/kind feature

/kind hotfix
/kind release

What this PR does / Why we need it:

Which issue(s) this PR fixes:

Closes #4523

Special notes for your reviewer:

agones-bot · 2026-05-11T09:18:18Z

Build Failed 😭

Build Id: 1d89798a-9fcf-422e-a2a2-c65b60892f56

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

agones-bot · 2026-05-11T09:53:58Z

Build Failed 😭

Build Id: e35ec88a-3848-4a70-8f87-3fe1b11a06e5

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

Sivasankaran25 · 2026-05-12T06:30:05Z

/gcbrun

agones-bot · 2026-05-12T06:45:29Z

Build Failed 😭

Build Id: 0894d8f3-6aa3-484e-8c51-b7d089f35370

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

agones-bot · 2026-05-12T08:47:50Z

Build Failed 😭

Build Id: f680848d-d334-4731-a973-ddd02c1184e4

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

agones-bot · 2026-05-24T19:00:23Z

Build Failed 😭

Build Id: 441eecfa-987c-407a-9c93-4dfaa26a41b6

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

agones-bot · 2026-05-24T20:57:25Z

Build Failed 😭

Build Id: c486f2ce-f194-47ec-a6cf-7a9112176a76

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

Sivasankaran25 · 2026-05-26T05:00:00Z

/gcbrun

agones-bot · 2026-05-26T05:14:34Z

Build Failed 😭

Build Id: 03abe090-4b08-4114-85dc-109f562c5d16

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

igooch

GKE Autopilot does not support Windows Nodes https://docs.cloud.google.com/kubernetes-engine/docs/concepts/windows-server-gke#limitations. Try running tests on Standard only.

agones-bot · 2026-05-30T06:28:24Z

Build Failed 😭

Build Id: d9561616-5ef7-4585-9398-81a4d7bce436

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

agones-bot · 2026-05-30T06:55:42Z

Build Failed 😭

Build Id: d899272a-a35a-4aed-86fa-6eb15696a675

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

agones-bot · 2026-05-30T07:36:30Z

Build Failed 😭

Build Id: c4b35f9c-0c5d-421a-a377-e514d5730e33

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

markmandel · 2026-06-12T03:57:56Z

Curious to hear how the work here is going? I can't see the clusters, but what is failing here?

Sivasankaran25 · 2026-06-12T04:13:42Z

Curious to hear how the work here is going? I can't see the clusters, but what is failing here?

The e2e-windows-smoke test step failed because GKE Autopilot does not support Windows nodes. The test must be configured to run only on GKE Standard clusters. Please suggest possible approaches to fix this build error.

markmandel · 2026-06-12T04:24:28Z

Could we use something like what we did here to skip autopilot?

agones/test/e2e/gameserver_test.go

Line 263 in 00f9151

    
           framework.SkipOnCloudProduct(t, "gke-autopilot", "does not support Static PortPolicy")

Sivasankaran25 · 2026-06-12T04:51:31Z

Could we use something like what we did here to skip autopilot?

agones/test/e2e/gameserver_test.go

Line 263 in 00f9151

framework.SkipOnCloudProduct(t, "gke-autopilot", "does not support Static PortPolicy")

Yes Sure I will try

agones-bot · 2026-06-12T06:16:28Z

Build Failed 😭

Build Id: 3ab22cd7-7fe0-41b2-b442-25b7e0366410

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

agones-bot · 2026-06-17T21:16:49Z

Build Failed 😭

Build Id: d9c984fe-6414-419b-97de-45d737e5cc2c

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

markmandel · 2026-06-19T03:46:40Z

      containers:
        - name: simple-game-server
-          image: us-docker.pkg.dev/agones-images/examples/simple-game-server:0.43
+          image: us-docker.pkg.dev/agones-images/examples/simple-game-server:0.42-windows_amd64-ltsc2019


This shouldn't be necessary -- the manifest should load the windows image.

markmandel · 2026-06-19T03:47:43Z

-{{% alert title="Warning" color="warning" %}}
-Running `GameServers` on Windows nodes is currently Alpha, and any feedback
-would be appreciated.
+{{% alert title="Note" color="info" %}}


All doc changes need to be feature shortcoded - https://agones.dev/site/docs/contribute/documentation-editing-contribution/

markmandel · 2026-06-19T03:47:56Z

 | linux     | `amd64`      | **Stable** |
 | linux     | `arm64`      | Alpha      |
-| [windows] | `amd64`      | Alpha      |
+| [windows] | `amd64`      | Beta       |


Same here - feature shortcoded.

markmandel · 2026-06-19T03:50:48Z

+
+// windowsGameServerImage is the published windows/amd64 simple-game-server.
+// Uses ltsc2019 — the currently available tag in agones-images registry.
+const windowsGameServerImage = "us-docker.pkg.dev/agones-images/examples/simple-game-server:0.42-windows_amd64-ltsc2019"


I'd double check the manifest for this image - but pretty sure it's a multi-arch and should select the correct image by detecting the host arch

markmandel · 2026-06-19T03:57:05Z

+  # The test itself calls t.Skip() when CloudProduct == "gke-autopilot".
+  # Runs parallel with e2e-feature-gates to avoid adding wall-clock time.
+  #
+  - name: e2e-runner


🤔 if the test windows_test.go is in the e2e folder - why do we need this extra entry in this config? Shouldn't it just run in the standard e2e test suite?

I'll reiterate this - we have tests that only run on autopilot, why do we need this extra build step? This should just be a single extra test that runs on standard GKE clusters, that tests the windows nodes.

agones-bot · 2026-07-07T04:11:54Z

Build Failed 😭

Build Id: e02d0b91-aa8d-4994-80d2-0e2797f14165

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

…rtcodes to docs

agones-bot · 2026-07-08T07:53:58Z

Build Failed 😭

Build Id: 67fb4e1b-8d4d-4cae-b37c-d4abd7c00ea8

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

agones-bot · 2026-07-08T08:30:24Z

Build Failed 😭

Build Id: 197cd300-465a-45f5-a92b-182bb44f0c56

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

markmandel · 2026-07-08T20:57:53Z

+  # The test itself calls t.Skip() when CloudProduct == "gke-autopilot".
+  # Runs parallel with e2e-feature-gates to avoid adding wall-clock time.
+  #
+  - name: e2e-runner


I'll reiterate this - we have tests that only run on autopilot, why do we need this extra build step? This should just be a single extra test that runs on standard GKE clusters, that tests the windows nodes.

markmandel · 2026-07-08T20:58:23Z

+
+	gs := framework.DefaultGameServer(framework.Namespace)
+	gs.Spec.Template.Spec.NodeSelector = map[string]string{
+		"kubernetes.io/os": "windows",


Looks good.

markmandel · 2026-07-08T20:59:25Z

  - started-by-${_PARENT_BUILD_ID}
 timeout: 5400s  # 1.5h
-queueTtl: 7200s  # 2h // only one set of e2es should be running at once
-logsBucket: gs://agones-build-logs


Don't remove the log bucket - that's why I can't see the logs.

markmandel · 2026-07-08T21:00:05Z

+  #
+  # Run Windows GameServer smoke test.
+  # Windows nodes are NOT supported on GKE Autopilot:
+  # https://cloud.google.com/kubernetes-engine/docs/concepts/windows-server-gke#limitations
+  # The test itself calls t.Skip() when CloudProduct == "gke-autopilot".
+  # Runs parallel with e2e-feature-gates to avoid adding wall-clock time.
+  #
+  - name: e2e-runner
+    args:
+      - ${_FEATURE_WITHOUT_GATE}
+      - ${_CLOUD_PRODUCT}
+      - ${_TEST_CLUSTER_NAME}
+      - ${_TEST_CLUSTER_LOCATION}
+      - ${_REGISTRY}
+      - ${_GS_TEST_IMAGE}
+    id: e2e-windows-smoke
+    waitFor:
+      - build-e2e
+    timeout: 900s


Suggested change

#

# Run Windows GameServer smoke test.

# Windows nodes are NOT supported on GKE Autopilot:

# https://cloud.google.com/kubernetes-engine/docs/concepts/windows-server-gke#limitations

# The test itself calls t.Skip() when CloudProduct == "gke-autopilot".

# Runs parallel with e2e-feature-gates to avoid adding wall-clock time.

#

- name: e2e-runner

args:

- ${_FEATURE_WITHOUT_GATE}

- ${_CLOUD_PRODUCT}

- ${_TEST_CLUSTER_NAME}

- ${_TEST_CLUSTER_LOCATION}

- ${_REGISTRY}

- ${_GS_TEST_IMAGE}

id: e2e-windows-smoke

waitFor:

- build-e2e

timeout: 900s

timeout: 900s

markmandel · 2026-07-08T21:00:31Z

+func TestWindowsCreateConnect(t *testing.T) {
+	t.Parallel()
+
+	if framework.CloudProduct == "gke-autopilot" {


Since you have this, it should run as part of the standard set of e2e tests -- no need for the extra build step.

agones-bot · 2026-07-09T04:33:09Z

Build Failed 😭

Build Id: 2d8351fd-4531-43fd-8c89-bfbe5c8192dd

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

Sivasankaran25 · 2026-07-09T04:43:21Z

/gcbrun

agones-bot · 2026-07-09T05:29:37Z

Build Failed 😭

Build Id: 1a5bebee-8115-4f71-a368-fb94b3d61bac

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

igooch

Thanks for moving Windows GameServers to Beta! The docs and support-tier changes look right, and the Autopilot handling is correct — the test is skipped on Autopilot and the node pool is only added to the standard cluster.

However, TestWindowsCreateConnect will fail consistently on the standard e2e cluster (where it actually runs) for two reasons. Fixes are suggested inline, plus one blocker that lives outside this diff is that no Windows image is built for e2e:

cloudbuild.yaml:188 builds and pushes the e2e simple-game-server image with WITH_WINDOWS=0, so its manifest is linux/amd64-only (examples/simple-game-server/Makefile only adds the windows_amd64 layers when WITH_WINDOWS=1). The test comment claims framework.GameServerImage is multi-arch with a windows/amd64 variant — that isn't true in CI, so a Windows-pinned pod has no runnable image and never reaches Ready. This needs WITH_WINDOWS=1 for the e2e image build step:

      - 'WITH_WINDOWS=1'

Inline suggestions

Add the Windows node toleration and prefer the SkipOnCloudProduct helper (test file).
Set the Windows taint on the node pool and restore the missing trailing newline (module.tf).

Nice work overall — once the e2e image includes a Windows variant and the pod can tolerate/schedule onto the Windows node, this should go green.

igooch · 2026-07-09T17:42:21Z

+	if framework.CloudProduct == "gke-autopilot" {
+		t.Skip("Windows nodes are not supported on GKE Autopilot")
+	}
+
+	gs := framework.DefaultGameServer(framework.Namespace)
+	gs.Spec.Template.Spec.NodeSelector = map[string]string{
+		"kubernetes.io/os": "windows",
+	}
+	// framework.GameServerImage is multi-arch (linux/amd64 + windows/amd64),
+	// so no separate Windows-tagged image is required - the correct
+	// platform variant is selected automatically based on the node.
+	gs.Spec.Template.Spec.Containers[0].Image = framework.GameServerImage


🔴 GKE taints Windows nodes node.kubernetes.io/os=windows:NoSchedule, but DefaultGameServer sets no toleration, so the pod can't schedule onto the Windows node. Also prefer the existing SkipOnCloudProduct helper, and drop the multi-arch comment — the e2e image is built with WITH_WINDOWS=0 (see summary), so GameServerImage has no windows/amd64 variant today.

Suggested change

if framework.CloudProduct == "gke-autopilot" {

t.Skip("Windows nodes are not supported on GKE Autopilot")

}

gs := framework.DefaultGameServer(framework.Namespace)

gs.Spec.Template.Spec.NodeSelector = map[string]string{

"kubernetes.io/os": "windows",

}

// framework.GameServerImage is multi-arch (linux/amd64 + windows/amd64),

// so no separate Windows-tagged image is required - the correct

// platform variant is selected automatically based on the node.

gs.Spec.Template.Spec.Containers[0].Image = framework.GameServerImage

framework.SkipOnCloudProduct(t, "gke-autopilot", "Windows nodes are not supported on GKE Autopilot")

gs := framework.DefaultGameServer(framework.Namespace)

gs.Spec.Template.Spec.NodeSelector = map[string]string{

"kubernetes.io/os": "windows",

}

gs.Spec.Template.Spec.Tolerations = []corev1.Toleration{{

Key: "node.kubernetes.io/os",

Operator: corev1.TolerationOpEqual,

Value: "windows",

Effect: corev1.TaintEffectNoSchedule,

}}

gs.Spec.Template.Spec.Containers[0].Image = framework.GameServerImage

agones-bot · 2026-07-10T04:36:50Z

Build Failed 😭

Build Id: f6275ab8-4205-4849-a395-7f559c635004

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

agones-bot · 2026-07-15T05:38:47Z

Build Failed 😭

Build Id: 5915080e-be9a-40a2-bc48-888ba68fb9ab

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

Implmemnted Feature Windows Gameserver Move Alpha to beta

eea6c1f

Sivasankaran25 requested a review from a team May 11, 2026 09:01

Sivasankaran25 self-assigned this May 11, 2026

github-actions Bot added kind/feature New features for Agones size/M labels May 11, 2026

Fix e2e test cloudbuild.yam

7fef0af

Change Windows base lts

0470921

Merge branch 'main' into feature/widowsgameserver

6444191

fix: resolve CI failures in Windows GameServer Beta PR

317efc2

nikkisingh0204 approved these changes May 24, 2026

View reviewed changes

igooch reviewed May 28, 2026

View reviewed changes

Sivasankaran25 added 2 commits May 30, 2026 06:21

fix: skip Windows smoke test on GKE Autopilot

781b0a6

Merge branch 'main' into feature/widowsgameserver

acdd754

fix lint error

ee42814

fix: remove entrypoint bash from windows smoke step

9fe7d30

Sivasankaran25 added 2 commits June 12, 2026 04:58

test/e2e: skip TestWindowsCreateConnect on gke-autopilot

0a9f06e

Merge branch 'main' into feature/widowsgameserver

700ce75

Merge branch 'main' into feature/widowsgameserver

f52ead6

markmandel reviewed Jun 19, 2026

View reviewed changes

Merge branch 'main' into feature/widowsgameserver

8c8b351

fix: use multi-arch image tag for Windows GameServer, add feature sho…

98662d5

…rtcodes to docs

fix: e2e test error

114d754

markmandel reviewed Jul 8, 2026

View reviewed changes

Sivasankaran25 added 2 commits July 9, 2026 03:50

cleanup(ci): remove e2e-windows-smoke build step

09afd41

Merge branch 'main' into feature/widowsgameserver

74cde1f

igooch reviewed Jul 9, 2026

View reviewed changes

Sivasankaran25 added 2 commits July 10, 2026 03:52

fix(e2e): fix Windows GameServer scheduling in TestWindowsCreateConnect

a97b606

Merge branch 'main' into feature/widowsgameserver

f5beb1b

Merge branch 'main' into feature/widowsgameserver

9f5c4be

Uh oh!

Conversation

Sivasankaran25 commented May 11, 2026

Uh oh!

agones-bot commented May 11, 2026

Uh oh!

agones-bot commented May 11, 2026

Uh oh!

Sivasankaran25 commented May 12, 2026

Uh oh!

agones-bot commented May 12, 2026

Uh oh!

agones-bot commented May 12, 2026

Uh oh!

agones-bot commented May 24, 2026

Uh oh!

agones-bot commented May 24, 2026

Uh oh!

Sivasankaran25 commented May 26, 2026

Uh oh!

agones-bot commented May 26, 2026

Uh oh!

igooch left a comment

Choose a reason for hiding this comment

Uh oh!

agones-bot commented May 30, 2026

Uh oh!

agones-bot commented May 30, 2026

Uh oh!

agones-bot commented May 30, 2026

Uh oh!

markmandel commented Jun 12, 2026

Uh oh!

Sivasankaran25 commented Jun 12, 2026

Uh oh!

markmandel commented Jun 12, 2026

Uh oh!

Sivasankaran25 commented Jun 12, 2026

Uh oh!

agones-bot commented Jun 12, 2026

Uh oh!

agones-bot commented Jun 17, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

agones-bot commented Jul 7, 2026

Uh oh!

agones-bot commented Jul 8, 2026

Uh oh!

agones-bot commented Jul 8, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

agones-bot commented Jul 9, 2026

Uh oh!

Sivasankaran25 commented Jul 9, 2026

Uh oh!

agones-bot commented Jul 9, 2026

Uh oh!

igooch left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

igooch left a comment •

edited

Loading