aaron/infra-core
1
0
Fork 2
Code Issues Pull requests 12 Projects Releases 1 Packages Wiki Activity Actions

HOMELAB-954: fix(workadventure): use correct chart schema for mapstorage PVC #180

Merged
aaron merged 1 commit from plane/HOMELAB-954-mapstorage-pvc into live 2026-04-23 06:22:57 +00:00
Conversation 0 Commits 1 Files changed 1 +6 -2
aaron commented 2026-04-23 06:22:52 +00:00
Owner
Copy link

Upstream workadventure chart v1.30.5 expects mapstorage.persistence.{enabled,storageClass,accessMode,storageSize} — the previous values used mapstorage.persistentVolume.{enabled,storageClass,size}, which is silently ignored, so map-storage was running on the default emptyDir. This violated the plan acceptance criterion that map uploads must persist across pod restart.

Rename to the correct schema. Longhorn will provision a 10Gi RWO PVC workadventure-mapstorage with helm.sh/resource-policy: keep, and the pod will mount it at /maps.

Upstream workadventure chart v1.30.5 expects `mapstorage.persistence.{enabled,storageClass,accessMode,storageSize}` — the previous values used `mapstorage.persistentVolume.{enabled,storageClass,size}`, which is silently ignored, so map-storage was running on the default emptyDir. This violated the plan acceptance criterion that map uploads must persist across pod restart. Rename to the correct schema. Longhorn will provision a 10Gi RWO PVC `workadventure-mapstorage` with `helm.sh/resource-policy: keep`, and the pod will mount it at /maps.
HOMELAB-954: fix(workadventure): use correct chart schema for mapstorage PVC
Some checks failed
CI Review / pr-title (pull_request) Has been cancelled
Details
CI Review / helm-validate (pull_request) Has been cancelled
Details
CI Review / ai-review (pull_request) Has been cancelled
Details
Lint & Validate / terraform-validate (pull_request) Has been cancelled
Details
Lint & Validate / yaml-lint (pull_request) Has been cancelled
Details
Lint & Validate / shellcheck (pull_request) Has been cancelled
Details
5c5390b337 
The upstream workadventure chart v1.30.5 expects `mapstorage.persistence.*`
(with `storageSize`), not `mapstorage.persistentVolume.*` (with `size`) — the
wrong keys were silently ignored, so the Deployment fell through to the
chart's default emptyDir volume. This meant map uploads didn't survive a pod
restart, violating the plan's acceptance criterion.

Rename to the chart's real schema so Longhorn provisions a 10Gi RWO PVC
(workadventure-mapstorage) and the pod mounts it at /maps. Upstream template
annotates the PVC with helm.sh/resource-policy: keep so a future uninstall
won't drop the data.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
aaron merged commit 40449d5e9c into live 2026-04-23 06:22:57 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
aaron/infra-core!180
No description provided.