Bug 1319743 - Enable memory cache on default configuration
Summary: Enable memory cache on default configuration
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: owncloud
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Igor Gnatenko
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-03-21 12:12 UTC by Remi Collet
Modified: 2016-03-22 09:36 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-03-22 09:36:07 UTC
Type: Bug


Attachments (Terms of Use)

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.


Note You need to log in before you can comment on or make changes to this bug.