Add vfs-cache-max-size and vfs-cache-mode

[ghost]

Added more/ better notes and defaults.

BTW vfs-cache settings have had 0 effect and thus only currently affected gcrypt mounts.

This PR changes it so the cache mode is shared on all mounts.

Currently cache mode is set to write on the crypt mounts, that change was made a few months ago due to a user bug report, however prior to that change it used to be off for crypt mounts. It's always been off for unencrypted mounts.

This PR changes the existing behavior, this is reverting the crypt mounts to have cache mode off again, which should be fine for most people. I only enabled write in the first place was because of a user report where the error would be resolved by changing the cache mode, however that user may have been doing things outside of normal operations. Regardless, it should be tested with cache mode off when using encrypt.

if there is a problem where you want crypt mount to have a different cache mode than the unencrypt mount. I could add a 2nd cache mode setting, but perhaps it would be better to check if encrypt is used, then default to "write" instead. Are tdrive and tcrypt are used at the same time if using encrypted?

[ghost]

@Admin9705 please review and test

[Admin9705]

works fine. nice add!

[ookla-ariel-ride]

bazarr requires the write mode enabled, this isn't doing something outside of normal operations.

[antigravity83]

Agree with @ookla-ariel-ride. Bazaar smashes your drive and triggers API bans without cache.

[ghost]

@ookla-ariel-ride @antigravity83 What I meant is it's not needed for the core apps, and thus not a default. It's actually a disadvantage to have it on, so the PG default will be set to "off". It's always been off for unencrypted mounts.

it's only needed for bazarr because it opens the files for reading and writing .For those that use bazarr or other programs that write while reading, you can now set the cache mode to be compatible with this software.

[antigravity83]

@robertbaker Ah ok, so you can redeploy with VFS cache active if you use Bazarr?

[ghost]

@antigravity83 yes, it's now configurable in the vfs options in 8.6.x

[Admin9705]

outstanding work @robertbaker i can tell the difference :D

[Davst]

For people that aren't aware of Bazaar issues, is it included in the deploy script of Bazaar to set cache on?

If not that should be considered.

[Admin9705]

so robert, i changed the 72h on my install to 1h and turn the thing onto full and everything instantly plays. my new stuff constantly buffers with it set to 72h for some reason, i think prior i had 2 minutes. but awesome job on that cache settings. makes a difference @robertbaker

[ghost]

Do you mean the dir cache time or vfs cache max age? I don't know why either of those 2 would make a difference.

The cache mode would affect it, when it's set to full, it downloads the file, not sure if it does that first before playing but it doesn't sound like it waits.

[m1lkman]

for those of us using encrypted mounts what are the recommended vfs settings now with the latest release?

[ghost]

@Admin9705 So I just reviewed and I forgot to edit the drive services in the PR. So cache mode itself has had 0 affect so far... oops

Did you test that with encrypt or unencrypt? Odd that the max age would affect things when the cache was either set to 'write' or 'off'

[ghost]

@m1lkman writes should probably be on by default for everybody, this offers max compat and it's needed for bazarr or if you enabled use nfo files in arr.

We all should do some testing on various configs once my PRs get merged.