feat(terraform): support multiple EFS volumes per game server

[CoderCoco]

Summary

Refactored the EFS volume configuration to support multiple mount points per game server, replacing the single efs_path field with a flexible volumes list. Each volume entry now creates its own EFS access point with isolated directory structure.

Key Changes

• Variable schema update: Replaced efs_path: string with volumes: list(object({ name, container_path })) in game_servers variable definition
• New local value: Added game_volumes local that flattens game-volume pairs into a keyed map (${game}-${volume.name}) for EFS access point creation
• EFS access point refactor: Changed from one access point per game to one per game-volume pair, with root paths now structured as /${game}/${volume_name} instead of /${game}
• Task definition volumes: Converted static volume block to dynamic "volume" block that iterates over each.value.volumes, creating one volume mount per configured volume
• Container mount points: Updated mountPoints from a single static entry to a dynamic list comprehension that maps each volume to its container path
• Documentation updates: Updated setup guide, example configs, and component docs to reflect the new volumes structure and explain the per-volume access point isolation

Implementation Details

• The flattening logic in game_volumes ensures each game-volume combination gets a unique key while preserving all necessary metadata (game name, volume name, container path)
• EFS access points maintain UID/GID 1000 ownership per volume, with each volume isolated under its own directory hierarchy
• The change is backward compatible in behavior—single-volume games simply have one entry in the volumes list
• All example configurations updated to demonstrate the new structure (Palworld, Satisfactory, FoundryVTT)

https://claude.ai/code/session_01SkWgmzBi8EPkSvpf3shwno

[Copilot]

Pull request overview

Refactors the Terraform EFS configuration for game servers to support multiple EFS-backed mount points per game by replacing the single efs_path setting with a volumes list, and updates docs/examples accordingly.

Changes:

• Update game_servers variable schema from efs_path to volumes[] (name, container_path) and refresh example configs.
• Add a local.game_volumes flattening map to create one EFS access point per game-volume pair and update ECS task definition volume/mountPoint generation.
• Update setup and Terraform component docs to describe the new volumes structure and access point isolation model.

Reviewed changes

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

Show a summary per file

File	Description
terraform/variables.tf	Updates game_servers schema to volumes[] and adjusts defaults.
terraform/terraform.tfvars.example	Updates example config to the new volumes[] format with explanatory comments.
terraform/main.tf	Implements per-volume EFS access points and dynamic ECS volumes/mount points.
docs/docs/setup.md	Updates setup guide examples and rules to refer to volumes[].
docs/docs/components/terraform.md	Updates variable documentation to reflect volumes[] and per-volume access point behavior.

[Copilot]

Pull request overview

Refactors the Terraform EFS + ECS task definition wiring to allow multiple EFS-backed mount points per game server, updating docs/examples accordingly.

Changes:

• Replaced per-game efs_path with volumes[] (name + container mount path) in game_servers.
• Creates one EFS access point per game-volume pair and mounts all configured volumes into the container.
• Updates Terraform outputs and documentation/examples to match the new multi-volume model.

Reviewed changes

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

Show a summary per file

File	Description
terraform/variables.tf	Updates game_servers schema to use volumes[] and adjusts defaults.
terraform/terraform.tfvars.example	Updates example config and adds guidance for volumes[].
terraform/outputs.tf	Keeps efs_access_points output shape (game → AP) by selecting the first volume’s AP.
terraform/main.tf	Adds local.game_volumes, refactors EFS access point creation and ECS volume/mount configuration.
docs/docs/setup.md	Updates setup instructions from efs_path to volumes[] with new semantics.
docs/docs/components/terraform.md	Updates variable documentation to reflect volumes[] and per-volume access points.

[CoderCoco]

Declined — the cross-game collision requires two games whose names and volume names split across - in the exact same way, which is a purely theoretical concern with normal identifiers like palworld, satisfactory, saves, config. A duplicate vol.name within the same game would cause Terraform to error on the duplicate for key, not silently overwrite — so that case is already caught loudly.

---

Generated by Claude Code

[CoderCoco]

Fixed in f01fc01 — added a validation block to the game_servers variable enforcing at least one volume entry with non-empty name and container_path. This fires at variable evaluation time with a clear message rather than producing an obscure Invalid index during plan.

---

Generated by Claude Code