Skip to content

Return GameServerAllocationUnAllocated when an game server update error occurs#4267

Merged
markmandel merged 4 commits intoagones-dev:mainfrom
miai10:fix/conflict-result
Nov 18, 2025
Merged

Return GameServerAllocationUnAllocated when an game server update error occurs#4267
markmandel merged 4 commits intoagones-dev:mainfrom
miai10:fix/conflict-result

Conversation

@miai10
Copy link
Copy Markdown
Contributor

@miai10 miai10 commented Sep 5, 2025

What type of PR is this?
/kind bug

What this PR does / Why we need it:
End the game server allocation with the state GameServerAllocationUnAllocated when an update error occurs

Which issue(s) this PR fixes:

Closes #4258

Special notes for your reviewer:

@github-actions github-actions bot added kind/bug These are bugs. size/S labels Sep 5, 2025
@miai10
Copy link
Copy Markdown
Contributor Author

miai10 commented Sep 5, 2025

While doing the feature, I tried to understand this.

ErrConflictInGameServerSelection seems similar to the new ErrGameServerUpdateConflict but it is used when removing an already removed server from the cache, here. Maybe it is code that should have been cleaned up because I don't see a way to double remove. Also it is strange to translate that into contention.

@miai10 miai10 force-pushed the fix/conflict-result branch from 52aab65 to 0fe9b1c Compare November 10, 2025 15:56
@markmandel
Copy link
Copy Markdown
Collaborator

/gcbrun

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Succeeded 🥳

Build Id: cb4ade0a-1fbf-44db-82eb-f9c8f3740bcd

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4267/head:pr_4267 && git checkout pr_4267
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.54.0-dev-0fe9b1c

@miai10 miai10 force-pushed the fix/conflict-result branch from c4de022 to 26bc876 Compare November 12, 2025 11:18
Copy link
Copy Markdown
Collaborator

@markmandel markmandel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noice.

@markmandel markmandel enabled auto-merge (squash) November 18, 2025 16:04
@markmandel
Copy link
Copy Markdown
Collaborator

/gcbrun

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Succeeded 🥳

Build Id: 74337f55-5dbd-40fe-9b99-66d6ddd67482

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4267/head:pr_4267 && git checkout pr_4267
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.54.0-dev-846793b

@markmandel markmandel merged commit 5ebc2d9 into agones-dev:main Nov 18, 2025
4 checks passed
@markmandel markmandel added the kind/breaking Breaking change label Nov 18, 2025
mnthe pushed a commit to mnthe/agones that referenced this pull request Mar 23, 2026
…or occurs (agones-dev#4267)

* Return GameServerAllocationUnAllocated when an update error occurrs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/breaking Breaking change kind/bug These are bugs. size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Proposal] - End the game server allocation with the state GameServerAllocationUnAllocated when an update conflict occurs

4 participants