Skip to content
This repository has been archived by the owner on Feb 4, 2021. It is now read-only.

Add support to nextcloud image to use an S3 bucket as primary storage #224

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rendhalver
Copy link

Let's try that again.

@rendhalver rendhalver closed this Sep 1, 2017
@rendhalver rendhalver reopened this Sep 1, 2017
@rendhalver
Copy link
Author

I actually got this working and it seems like a useful feature
Original PR was #223

@ghost
Copy link

ghost commented Sep 1, 2017

This is awesome. How would I get this working in an EC2 Ubuntu server instance? If there is something that can be added to the config.php to enable S3 storage, that would be awesome. I know there is a way to set up "External Storage" and add S3 -- but I am trying to have the NC_DATA directory for the normal files stored in S3; instead of within in the server. Thanks @rendhalver

Copy link
Owner

@Wonderfall Wonderfall left a comment

Choose a reason for hiding this comment

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

Please add documentation, then I'll merge. 😉

@rendhalver rendhalver force-pushed the feature/nextcloud_s3_objectstore branch from dd45435 to e7bcc6b Compare September 5, 2017 12:34
@rendhalver
Copy link
Author

@Wonderfall Basic docs added.
If you would like more detail let me know and I will expand them.

'key' => '${DATASTORE_KEY}',
'secret' => '${DATASTORE_SECRET}',
'hostname' => '${DATASTORE_HOST}',
'use_ssl' => true,
Copy link

Choose a reason for hiding this comment

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

Duped key here, use_ssl is defined below again.

- **DATASTORE_KEY** : S3 Key for about bucket *(default : none)*
- **DATASTORE_SECRET** : S3 secret for about bucket *(default : none)*
- **DATASTORE_HOST** : S3 host for above bucket *(default : none)*
- **DATASTORE_PORT** : S3 port for above bucket *(default : 443)*
Don't forget to use a **strong password** for the admin account!
Copy link

@djtaylor djtaylor Sep 7, 2017

Choose a reason for hiding this comment

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

You might want to add DATASTORE_USE_PATH_STYLE with default as false, since AWS buckets currently don't use path style. Path style tends to be for very specific 3rd party implementations of S3.

@rendhalver rendhalver force-pushed the feature/nextcloud_s3_objectstore branch 2 times, most recently from 5250441 to 107262d Compare September 7, 2017 18:26
@rendhalver rendhalver force-pushed the feature/nextcloud_s3_objectstore branch from 107262d to a047b8f Compare September 7, 2017 18:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants