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

External local storage is not picked up #154

Open
rajil opened this issue Jan 25, 2020 · 9 comments
Open

External local storage is not picked up #154

rajil opened this issue Jan 25, 2020 · 9 comments

Comments

@rajil
Copy link

@rajil rajil commented Jan 25, 2020

Hello,
I have an local external storage where i am exposing my digikam organized photos. Unfortunately, the Photos app does not pick up this folder. Is there a setting any where to select folders for the Photo app?

@skjnldsv

This comment has been minimized.

Copy link
Member

@skjnldsv skjnldsv commented Jan 25, 2020

Hello!
This app is indeed not compatible with external storage for performances reasons.
I'm still not sure thi sis something we'll be able to do or if we even want to dive into this yet :)

cc @rullzer @icewind1991 & @jancborchardt

@Dragon988

This comment has been minimized.

Copy link

@Dragon988 Dragon988 commented Jan 25, 2020

That's disappointing. Why not allow users to choose which external storage will work on photos instead? It doesn't have to be a universal setting.

For example I might keep photos on another server and connect through NAS. Since my primary storage is S3 I'm not able to mount or symbolic link the folder into my nextcloud data directory.

Or maybe I'm keeping them on wasabi because API requests and download is free.

@S74HK9hV

This comment has been minimized.

Copy link

@S74HK9hV S74HK9hV commented Jan 27, 2020

I upgraded one of my instances to NC18 already and didn't realize that external shares aren't working.

I'm glad I saw this post before upgrading my main NC server. This basically breaks Nextcloud for me in terms of photos.

There should be a big red flag in the NC18 changelog about this. The fact that there is no option to use the old gallery app, and the new one basically won't work for a lot of us

I imagine there are lots of people who share their photo collection via external storage. Photo collections are generally quite large and better suited for storage on a dedicated NAS device. I would also imagine there are a lot of other people with inexpensive NAS that aren't capable of running nextcloud on the same hardware.

@skjnldsv what are the performance issues with external storage? the old gallery app seemed to work just fine. admittedly, it wasn't the fastest but it worked well enough to make it a compelling alternative to uploading/sharing things on google

@lgrawet

This comment has been minimized.

Copy link

@lgrawet lgrawet commented Jan 29, 2020

In my case I use "external storage" to connect local samba storage. I need to access the storage booth ways (nextcloud and samba) and I also need to preserve user permissions at OS level.

@Dragon988

This comment has been minimized.

Copy link

@Dragon988 Dragon988 commented Jan 29, 2020

Hello!
This app is indeed not compatible with external storage for performances reasons.
I'm still not sure thi sis something we'll be able to do or if we even want to dive into this yet :)

cc @rullzer @icewind1991 & @jancborchardt

@skjnldsv I would appreciate it if you expanded on this.

@rajil

This comment has been minimized.

Copy link
Author

@rajil rajil commented Jan 30, 2020

My nextcloud data directory and the local photos/digikam folder share the same file system. If I do a symlink my external local directory somewhere in the data directory, will photos index it?

@skjnldsv

This comment has been minimized.

Copy link
Member

@skjnldsv skjnldsv commented Jan 30, 2020

My nextcloud data directory and the local photos/digikam folder share the same file system. If I do a symlink my external local directory somewhere in the data directory, will photos index it?

Hey, if your data directory is just hard linked, yes, this is the job of your underlying file system, same goes for any external storage mounted in the os level.
Only the external storage app is not used here :)

@Dragon988

This comment has been minimized.

Copy link

@Dragon988 Dragon988 commented Feb 16, 2020

Can we get clarification on the performance impact external storage has on this application? I can live with it if there is a legitimate performance concern but if the external storage is on the local filesystem/LAN or low latency I can't see it being a problem.

Is the performance concern related to external storage or the method in which nextcloud handles the external storage app? Is it a privacy issue? Gallery was slow at generating previews but once they were generated I never noticed a performance impact on external storage.

Thank you.

@alarmix

This comment has been minimized.

Copy link

@alarmix alarmix commented Feb 19, 2020

Reading this too late, is there a painless way to revert to NC17? For me, as presumably for many others, the new photo app is absolutely useless without external storage support!
Agree that there SHOULD have been a VERY big red notice about it in the changelog!!

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

Successfully merging a pull request may close this issue.

None yet
6 participants
You can’t perform that action at this time.