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

Add support for s3 external storage. #157

Closed
pierreozoux opened this issue Jan 13, 2021 · 4 comments
Closed

Add support for s3 external storage. #157

pierreozoux opened this issue Jan 13, 2021 · 4 comments

Comments

@pierreozoux
Copy link
Member

Hi,

I saw the other issues. I just wanted to open one to track support for it.

Thanks for your work, and for considering this enhancement :)

@pierreozoux pierreozoux changed the title Add support with s3 external storage. Add support for s3 external storage. Jan 13, 2021
@PhrozenByte
Copy link
Collaborator

Pico currently requires direct filesystem access, thus Pico CMS for Nextcloud can't support non-local storages. See #37 for more info. As soon as we have a solution for that, it doesn't really matter what non-local stroage is used, i.e. s3 external storage would work, too. This feature request is tracked in #52

@pierreozoux
Copy link
Member Author

if it is already tracked, then I close, just the title of #52 is not really clear.

@PhrozenByte
Copy link
Collaborator

PhrozenByte commented Jan 14, 2021

just the title of #52 is not really clear.

Yeah, that's because there are currently no plans to add filesystem abstraction to Pico, as it kinda objects the "stupidly simple" approach regarding Pico's own code. Thus it's unlikely that this will be implemented on Pico's side. However, another solution is to let Nextcloud cache external storages locally, allowing Pico to access the files locally. That's what persistent federated data basically is about. Thus it's tracked below #52. It's another solution for the same problem that is just more likely to be implemented sometime. However, this is nothing we can carry out ourselves. Also see nextcloud/server#8459.

@pierreozoux
Copy link
Member Author

Thanks for your kind answer :) have a nice evening :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants