Bug 1319743

Summary: Enable memory cache on default configuration
Product: [Fedora] Fedora Reporter: Remi Collet <fedora>
Component: owncloudAssignee: Igor Gnatenko <ignatenko>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: awilliam, ignatenko, james.hogarth, shawn
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-03-22 09:36:07 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Remi Collet 2016-03-21 12:12:36 UTC
After installation, on administration page, owncloud display a warning about missing cache, which seems recommended for performance.

See: https://doc.owncloud.org/server/8.1/admin_manual/configuration_server/caching_configuration.html

Proposal: 

In config.php, addd

    'memcache.local' => '\OC\Memcache\APCu',

In spec file, add

     Requires: php-pecl(apcu) >= 4.0.6

Comment 1 James Hogarth 2016-03-21 12:43:24 UTC
I'm not keen on this change.

There's a bunch of things that should be done after an install by a user - only one of which is to possibly enable caching.

There's multiple different ways of carrying caching and if the administrator chooses to use redis, for instance, there would be a pointless php-pecl-aocu package installed.

Note that redis is the required caching layer for Transactional File Locking

https://doc.owncloud.org/server/8.1/admin_manual/configuration_files/files_locking_transactional.html

If APCu is enabled by default there there would be no notification on the admin page pointing the new admin to the caching documentation where they would learn about this.

Unless a *really* good reason can be provided for it I'm closing this WONTFIX NOTABUG

Comment 2 James Hogarth 2016-03-22 09:36:07 UTC
After sleeping on this I'd rather stick to no default configured so the message appears in the admin panel and the administrator of the owncloud installation will be directed to the docs to pick the best case for them, rather than hiding it from them.

I will add a section to the README though that has the correct dnf install to pick up the relevant package properly.