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

Backup/restore scripts #23

Merged
merged 4 commits into from
Mar 4, 2017
Merged

Conversation

rokaz
Copy link
Contributor

@rokaz rokaz commented Feb 27, 2017

Hi,

I wrote these basic backup/restore scripts. I tested them and they seem to be working as expected.

@lapineige
Copy link
Member

Sorry for the delay.
I'm testing those scripts right now :)

@lapineige
Copy link
Member

lapineige commented Mar 3, 2017

Maybe it's because of my installation, but when making a backup, removing shaarli, and restoring later, it raises and error like: An error happened during LDAP operation

I'm on Yunohost 2.5.6.

@rokaz
Copy link
Contributor Author

rokaz commented Mar 3, 2017

I have the same error when backing up and restoring both hooks and other official apps (tested with wallabag and dokuwiki). Backup/restore without the hooks (only the app) works on my test system (the ynh_dev script, on stable), as for the other apps and the hooks alone. I didn't take the time to talk with the ynh devs of this issue, which doesn't seem to be triggered by this only app.

@lapineige
Copy link
Member

I did it just for shaarli app, but I don't remember if I added the hooks too.
I'll test again.

If it works, and as it seems to be a known bug, I'll merge your PR and add a warning in the README :)

@lapineige
Copy link
Member

Ok, if I had the --ignore-hooks option it works. But I have a warning for Insufficient permissions in "cache" , "data", "pagecache" and"tmp/" directories.
Looking at the script, it setup the owner as root (same for the group). Shouldn't it be www-data ? (as during install)

It works like that on your config ?

@rokaz
Copy link
Contributor Author

rokaz commented Mar 4, 2017

No, it is a mistake from myself. I pushed the correction.

@lapineige lapineige merged commit 0293bf1 into YunoHost-Apps:master Mar 4, 2017
@lapineige
Copy link
Member

lapineige commented Mar 4, 2017

Everything works right now :)
Thanks a lot for your work !

@lapineige lapineige mentioned this pull request Mar 4, 2017
3 tasks
@rokaz rokaz deleted the backup_script branch March 4, 2017 10:19
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