Skip to content
This repository was 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
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants