Return GameServerAllocationUnAllocated when an game server update error occurs#4267
Conversation
|
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. |
52aab65 to
0fe9b1c
Compare
|
/gcbrun |
|
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: |
c4de022 to
26bc876
Compare
|
/gcbrun |
|
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: |
…or occurs (agones-dev#4267) * Return GameServerAllocationUnAllocated when an update error occurrs
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: