Skip to content

Commit

Permalink
Best practices for game server shutdown (googleforgames#1752)
Browse files Browse the repository at this point in the history
Based on conversations in Slack, it was not clear on best practices on
how to handle graceful shutdown in the game server process.

Wrote some docs that both point to the Kubernetes documentation, but
also provides a rule of thumb that should apply for most all cases.

Co-authored-by: Alexander Apalikov <alexander.apalikov@globant.com>
  • Loading branch information
2 people authored and ilkercelikyilmaz committed Oct 23, 2020
1 parent 6ed9485 commit 855406a
Showing 1 changed file with 11 additions and 3 deletions.
14 changes: 11 additions & 3 deletions site/content/en/docs/Guides/Client SDKs/_index.md
Original file line number Diff line number Diff line change
Expand Up @@ -121,9 +121,17 @@ relinquish control to an external service which likely doesn't have as much info
{{< /alert >}}

#### Shutdown()
This tells Agones to shut down the currently running game server.
The GameServer state will be set `Shutdown` and the
backing Pod will be deleted, if they have not shut themselves down already.
This tells Agones to shut down the currently running game server. The GameServer state will be set `Shutdown` and the
backing Pod will be Terminated.

It's worth reading the [Termination of Pods](https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/#pod-termination)
Kubernetes documentation, to understand the termination process, and the related configuration options.

As a rule of thumb, implement a graceful shutdown in your game sever process when it receives the TERM signal
from Kubernetes when the backing Pod goes into Termination state.

Be aware that if you use a variation of `System.exit(0)` after calling SDK.Shutdown(), your game server container may
restart for a brief period, inline with our [Health Checking]({{% ref "/docs/Guides/health-checking.md#health-failure-strategy" %}}) policies.

### Configuration Retrieval

Expand Down

0 comments on commit 855406a

Please sign in to comment.