Skip to content

feat(deployments): switch ocis_full example to bind-mounted ./data/ volumes - #12801

Draft
LukasHirt wants to merge 1 commit into
masterfrom
feat-ocis-full-bind-mounts
Draft

feat(deployments): switch ocis_full example to bind-mounted ./data/ volumes#12801
LukasHirt wants to merge 1 commit into
masterfrom
feat-ocis-full-bind-mounts

Conversation

@LukasHirt

Copy link
Copy Markdown
Contributor

Summary

  • Switch the ocis_full deployment example from docker-managed named volumes to ./data/-relative bind mounts so persisted data is directly accessible on the host
  • Add migrate-volumes.sh, a one-time helper that copies data from the old named volumes into the new bind-mount paths for existing deployments upgrading in place
  • Update the example README and deployment docs accordingly

Test plan

  • docker compose config validates cleanly for the full stack plus enabled web extensions
  • Fresh install: docker compose up creates and populates ./data/* directories as expected
  • Upgrade path: run migrate-volumes.sh against a deployment with existing named volumes (including ocis-apps and companion-data) and confirm data lands in the correct ./data/* paths
  • migrate-volumes.sh runs successfully under both bash 5.x and bash 3.2 (stock macOS /bin/bash)

…olumes

Replace docker-managed named volumes with ./data/ bind mounts in the
ocis_full deployment example so persisted data is directly accessible
on the host, and add a migrate-volumes.sh helper to copy existing
named-volume data across for upgrading deployments.

Signed-off-by: Lukas Hirt <info@hirt.cz>
@update-docs

update-docs Bot commented Aug 14, 2026

Copy link
Copy Markdown

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@LukasHirt LukasHirt self-assigned this Aug 14, 2026
@kw-security

kw-security commented Aug 14, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@LukasHirt

Copy link
Copy Markdown
Contributor Author

FYI @mmattel I keep this now as a draft. It's quite a bigger change so I need to do some more testing first and also I do not want to merge this before the Saturdays workshop so that it does not break anything for that (and does not force updating the current slides/notes).

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.

2 participants