Skip to content
This repository has been archived by the owner on Sep 18, 2022. It is now read-only.

Support multiple instances installation #9

Closed
didierm opened this issue Oct 11, 2018 · 3 comments
Closed

Support multiple instances installation #9

didierm opened this issue Oct 11, 2018 · 3 comments

Comments

@didierm
Copy link

didierm commented Oct 11, 2018

We run multiple instances with the same codebase, by using the NEXTCLOUD_CONFIG_DIR variable (nextcloud/server#300).

In order for calcardbackup to support multiple instances, I propose the following patch :

# location of ownClouds/Nextclouds config.php:
configphp="${nextcloud_path}/config/config.php"

to

# location of ownClouds/Nextclouds config.php:
[[ -z "${NEXTCLOUD_CONFIG_DIR:-}" ]] && configphp="${nextcloud_path}/config/config.php" || configphp="${NEXTCLOUD_CONFIG_DIR}/config.php"
@BernieO
Copy link
Owner

BernieO commented Oct 11, 2018

On a first glance this looks good to me.
I will have a closer look into this the next couple of days.

@BernieO
Copy link
Owner

BernieO commented Oct 13, 2018

I saw that this is also possible with ownCloud: owncloud/core#27874

support for environment variables NEXTCLOUD_CONFIG_DIR and OWNCLOUD_CONFIG_DIR is been added with commit 279b923 (until release of the next version only in the testing branch)

However, if both environment variables are set, calcardbackup will exit with an error, because the script does not know which one to use. In this case one of the environment variables has to be unset for the script to work.

@didierm: could you check whether that does the trick for your use case? Thanks.

@didierm
Copy link
Author

didierm commented Oct 16, 2018

Tested (ver. 0.7.2-24 (13.10.2018)) with NEXTCLOUD_CONFIG_DIR ; works perfect.
Thanks a lot !

@didierm didierm closed this as completed Oct 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants