Fix bind-mounted app directories being overridden by anonymous volumes#444
Fix bind-mounted app directories being overridden by anonymous volumes#444pulsarf0x wants to merge 1 commit into
Conversation
|
Thanks for you PR, to me this is intended behaviour as this writable app directory was for me only used for testing app installation through the apps management but not for development. Is there any reason why you would not just clone the app you need in your local setup? |
|
Hello @juliusknorr, I agree that for pure development work, cloning an app directly into apps/ or apps-extra is often the right approach. However, in practice, apps-writable is not only used for “testing installation” but is the default target for apps installed via the App Store or occ app:install, which is a very common workflow when:
Mounting apps-writable does not change the default behaviour or force developers to use it for development, but it makes installed apps visible on the host filesystem, which greatly improves debugging and inspection of apps without impacting users who prefer cloning apps manually. What do you think ? |
Hello Team,
Here's a little PR that could make DX better :
Problem
When using
nextcloud-docker-devfor local development, apps installed viaocc app:installor the App Store were not appearing on the host filesystem.Although bind-mounts for app directories were defined, they were overridden
by anonymous Docker volumes declared later in the service configuration.
As a result, Nextcloud was writing apps into anonymous volumes instead of
the intended local directories, making them invisible to IDEs.
Root cause
Docker gives precedence to the last volume mounted on a given path.
Anonymous volumes declared for:
/var/www/html/apps-writablewere overriding explicit bind-mounts for development.
Solution
This PR removes the anonymous volumes that shadow bind-mounted directories.
With this change:
occare written to the bind-mountedapps-writabledirectoryImpact