search

drupalwxt / wxt

Drupal 10 variant of the Web Experience Toolkit (WxT).
https://drupalwxt.github.io
GNU General Public License v2.0
25 stars 27 forks source link

S3FS? #300

Closed liquidcms closed 1 year ago

liquidcms commented 1 year ago

I see the WxT composer.json has patches (which are not current) for S3FS, even though it does not explicitly load S3FS. It is possible that S3FS it is a dependency of some other project which is loaded (poor way to manage with composer); but i suspect not the case.

sylus commented 1 year ago

Hi there ^_^

This was from quite a while ago when we had s3fs bundled before reduced the size of distro and also there were better ways of handling persistent storage via Azure Files NFS mounting so this is no longer needed.

I removed the patches since they should no longer be there :)

liquidcms commented 1 year ago

We'll likely still end up using S3FS as i dont think an NFS mount will offer the same solution. We are trying to not have default scaffold backup system backup the private files folder. A solution is to slice off specific folders to S3 and as such; they will not be included in a tar script; whereas an NFS mount would still be seen by tar.

But that patch is old, so thanks for removing.

liquidcms commented 8 months ago

but crept back in to 5.x

sylus commented 8 months ago

Thank you!

I removed this again, my apologies!