Skip to content

feat: reconstruct full storage_mount attachment state on export#289

Merged
josegonzalez merged 1 commit into
mainfrom
281-export-reconstruct-storage-mount-phases-readonly-subpath-process-type
Jul 7, 2026
Merged

feat: reconstruct full storage_mount attachment state on export#289
josegonzalez merged 1 commit into
mainfrom
281-export-reconstruct-storage-mount-phases-readonly-subpath-process-type

Conversation

@josegonzalez

Copy link
Copy Markdown
Member

The dokku_storage_mount exporter read storage:list --format json, which only exposes the entry/host and container paths and is filtered to the deploy phase, so phases, process_type, subpath, readonly, and volume_chown were dropped and run-only mounts were missed. It now reads storage:report --format json, which exposes every attachment attribute for all phases, and reconstructs each field in full while omitting values that match dokku's defaults. The drift-detection read is unified onto the same report so run-only mounts are visible and the exported recipe round-trips.

Closes #281

The dokku_storage_mount exporter read `storage:list --format json`, which
only exposes the entry/host and container paths and is filtered to the deploy
phase, so `phases`, `process_type`, `subpath`, `readonly`, and `volume_chown`
were dropped and run-only mounts were missed. It now reads
`storage:report --format json`, which exposes every attachment attribute for
all phases, and reconstructs each field in full while omitting values that
match dokku's defaults. The drift-detection read is unified onto the same
report so run-only mounts are visible and the exported recipe round-trips.
@josegonzalez josegonzalez merged commit b12d269 into main Jul 7, 2026
18 of 19 checks passed
@josegonzalez josegonzalez deleted the 281-export-reconstruct-storage-mount-phases-readonly-subpath-process-type branch July 7, 2026 16:01
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.

export: reconstruct storage_mount phases/readonly/subpath/process-type

1 participant