Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding documentation to change nextcloud data path #670

Open
wants to merge 6 commits into
base: testing
Choose a base branch
from

Conversation

Alexis-IMBERT
Copy link

@Alexis-IMBERT Alexis-IMBERT commented Feb 27, 2024

close : #674

Problem

  • Trouble finding a proper way to change the location of Nextcloud's data folder

Solution

  • Adding the command to change the data path of Nextcloud in admin documentation (with french translation)

PR Status

  • Code finished and ready to be reviewed/tested
  • The fix/enhancement were manually tested (if applicable)

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

@Alexis-IMBERT
Copy link
Author

!testme

@ericgaspar ericgaspar changed the base branch from master to testing February 27, 2024 21:58
doc/ADMIN.md Outdated Show resolved Hide resolved
doc/ADMIN.md Outdated Show resolved Hide resolved
Co-authored-by: eric_G <[email protected]>
doc/ADMIN_fr.md Outdated Show resolved Hide resolved
@Alexis-IMBERT
Copy link
Author

Hi!
I read this doc https://yunohost.org/fr/packaging_apps_resources#data-dir

When the mirgation is effective ?

I reinstalled my server but when I did the command the folder was not created, and when I create a new files in nextcloud, the file is created on the first folder

@Alexis-IMBERT Alexis-IMBERT requested a review from OniriCorpe March 1, 2024 23:51
sudo -u __APP__ php__PHPVERSION__ --define apc.enable_cli=1 __INSTALL_DIR__/occ ...
```

Alternatively, you may open a 'Nextcloud shell' with `sudo yunohost app shell __APP__`, then run `php occ ...`

### Change data folder location
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Related to https://github.com/YunoHost/doc/blob/master/pages/04.applications/10.docs/nextcloud/app_nextcloud.md#ii-migrate-nextcloud-data-to-a-larger-partition which is actually supposed to be removed and/or migrated to the app's ADMIN.md as a consequence of YunoHost/doc#2417

This old piece of doc actually seems to recommend a savage cp + edition of the config.php + the data dir setting

The symlink solution seems better but I don't remember why exactly but this ultimately also causes issues later on (maybe with backups if i remember correctly)

There is also https://yunohost.org/en/administer/tutorials/external_storage which to me sounds like the most reliable solution and uses bind mounts

Ideally we should harmonize all this because this is quite a reccuring question/issue and currently there's no way to move apps data in a simple yet long-term robust way

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.

4 participants