Skip to content

feat: add allocator and ping discovery service#318

Open
Christian R (antiphp) wants to merge 14 commits into
mainfrom
cr/alloc-ping
Open

feat: add allocator and ping discovery service#318
Christian R (antiphp) wants to merge 14 commits into
mainfrom
cr/alloc-ping

Conversation

@antiphp

Copy link
Copy Markdown
Contributor

Goal of this PR

Describe provisioned Allocator and Ping (Discovery) Services and how to make use of them.

Additional context

@antiphp Christian R (antiphp) self-assigned this Jul 7, 2026
Copilot AI review requested due to automatic review settings July 7, 2026 12:54
@antiphp Christian R (antiphp) requested a review from a team as a code owner July 7, 2026 12:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation for the newly provisioned managed Allocator and Ping (Discovery) services within the Multiplayer Servers architecture docs, and updates existing allocation guides to reference the managed behavior.

Changes:

  • Added new architecture pages for Allocators and Ping Services, including UI navigation, lifecycle phases, and token guidance.
  • Documented Region-to-Allocator attachment behavior and automatic environment variable injection, including migration guidance.
  • Updated server allocation docs to clarify sidecar background and to note that managed allocators inject ALLOC_* variables automatically.

Reviewed changes

Copilot reviewed 6 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/multiplayer-servers/multiplayer-services/server-allocation/manually-registering-game-servers.md Adds a sidecar explainer callout linking to the Sidecar Containers architecture page.
src/multiplayer-servers/multiplayer-services/server-allocation/automatically-registering-game-servers.md Documents managed allocator auto-injection of ALLOC_* env vars and links to migration guidance.
src/multiplayer-servers/architecture/sidebar.json Adds navigation entries for the new Allocators and Ping Services pages.
src/multiplayer-servers/architecture/regions-sites-and-locations.md Adds a new section describing allocator attachment to Regions and updates Region operations to include allocator assignment.
src/multiplayer-servers/architecture/ping-services.md New page documenting Ping Services concept, endpoint/tokens, phases, navigation, and filtering.
src/multiplayer-servers/architecture/allocators.md New page documenting Allocators, endpoints/tokens, Region attachment, env var injection/precedence, filtering, and migration considerations.

Comment thread src/multiplayer-servers/architecture/images/allocators-expanded.png
Copilot AI review requested due to automatic review settings July 8, 2026 13:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 14 changed files in this pull request and generated no new comments.

Comment thread src/multiplayer-servers/architecture/allocators.md Outdated
Comment thread src/multiplayer-servers/architecture/images/allocators-expanded.png
Comment thread src/multiplayer-servers/architecture/allocators.md Outdated
Comment thread src/multiplayer-servers/architecture/ping-services.md
@antiphp

Copy link
Copy Markdown
Contributor Author

those URLs differ from the usual schema by quite a lot, but i suppose for exemplary purposes that's fine

See #318 (comment).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 14 changed files in this pull request and generated 2 comments.

Comment thread src/multiplayer-servers/architecture/allocators.md
Comment thread src/multiplayer-servers/architecture/regions-sites-and-locations.md
Copilot AI review requested due to automatic review settings July 9, 2026 15:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 14 changed files in this pull request and generated 3 comments.

Comment thread src/multiplayer-servers/architecture/allocators.md
Comment thread src/multiplayer-servers/architecture/allocators.md Outdated
Comment thread src/multiplayer-servers/architecture/regions-sites-and-locations.md
Copilot AI review requested due to automatic review settings July 9, 2026 16:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 14 changed files in this pull request and generated 5 comments.

Comment thread src/multiplayer-servers/architecture/allocators.md Outdated
Comment thread src/multiplayer-servers/architecture/allocators.md Outdated
Comment thread src/multiplayer-servers/architecture/allocators.md
Comment thread src/multiplayer-servers/architecture/ping-services.md Outdated
Comment thread src/multiplayer-servers/architecture/allocators.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 9, 2026 16:17
Christian R (antiphp) and others added 2 commits July 9, 2026 18:17
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 14 changed files in this pull request and generated 3 comments.

Comment thread src/multiplayer-servers/architecture/allocators.md Outdated
Comment thread src/multiplayer-servers/architecture/allocators.md
Comment thread src/api/sidebar.json
Copilot AI review requested due to automatic review settings July 9, 2026 16:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 14 changed files in this pull request and generated 1 comment.

Comment thread src/multiplayer-servers/architecture/allocators.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants