Skip to content

chore(paper): drop plugin-agones-paper from default PLUGINS#92

Merged
hbrombeer merged 1 commit intomainfrom
chore/paper-drop-agones-plugin
May 6, 2026
Merged

chore(paper): drop plugin-agones-paper from default PLUGINS#92
hbrombeer merged 1 commit intomainfrom
chore/paper-drop-agones-plugin

Conversation

@hbrombeer
Copy link
Copy Markdown
Member

Paper baseImage is only used by type: plugin-paper workloads — those render as plain Deployments, never as Agones GameServers. The SDK sidecar never gets injected, so plugin-agones-paper here is permanently inert dead weight (and noise on boot until plugin-agones#40 ships). If a future type: gamemode Paper-based baseImage lands, the plugin belongs there.

The paper baseImage is consumed exclusively by `type: plugin-paper`
workloads (per BASE_IMAGE_TO_TYPE in grounds-forge/src/buildrunner/
manifest.ts). Forge renders those as plain Deployments — never as
Agones-managed GameServers. The Agones SDK sidecar is only injected
on GameServer-managed pods, so plugin-agones-paper in this image
talks to a localhost:9358 that never exists. The plugin's own
docs accept this state (`Errors are logged at severe level`) and
groundsgg/plugin-agones#40 makes it inert when the env var is unset,
but the simpler fix is to not bundle it here at all.

If a future `type: gamemode` workload uses a Paper-based baseImage
(rather than the current minestom-based one), plugin-agones-paper
belongs in that image. Tracked separately.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@hbrombeer hbrombeer merged commit fddcc0e into main May 6, 2026
5 checks passed
@hbrombeer hbrombeer deleted the chore/paper-drop-agones-plugin branch May 6, 2026 08:46
hbrombeer added a commit that referenced this pull request May 6, 2026
Surfaces #92 as a release-worthy change. The chore-prefixed merge
removed plugin-agones-paper from PLUGINS= but didn't trigger
release-please; this empty commit cuts a paper@v0.5.1 so the new
image actually lands in GHCR for forge to pick up. User pods stop
shipping the dead-weight plugin once forge bumps its baseImage tag.
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.

1 participant