Bug 24642

Summary: Cache::Memcached::Fast::Safe must be marked as mandatory
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: 1joynelson, julian.maurice, lucas, m.de.rooy, martin.renvoize
Version: unspecifiedKeywords: dependency
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19735
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.04
Bug Depends on: 13193    
Bug Blocks:    
Attachments: Bug 24642: Make Cache::Memcached::Fast::Safe required
Bug 24642: Make Cache::Memcached::Fast::Safe required
Bug 24642: Make Cache::Memcached::Fast::Safe required

Description Jonathan Druart 2020-02-12 14:22:54 UTC
There is something wrong between the logic in Koha::Cache->new and our dependencies.

We are having Cache::Memcached required, but Cache::Memcached::Fast::Safe is optional.
However, in Koha::Cache->new we are initializing the Memcached cache only if Cache::Memcached::Fast::Safe is installed.

We could fix the logic in Koha::Cache, but it seems better to make Cache::Memcached::Fast::Safe mandatory (especially after bug 13193).

Note that after bug 13193, you are loosing the Memcached cache, as Cache::Memcached::Fast::Safe will not get installed by default.
Comment 1 Jonathan Druart 2020-02-12 14:26:35 UTC
Created attachment 98755 [details] [review]
Bug 24642: Make Cache::Memcached::Fast::Safe required

There is something wrong between the logic in Koha::Cache->new and our dependencies.

We are having Cache::Memcached required, but Cache::Memcached::Fast::Safe is optional.
However, in Koha::Cache->new we are initializing the Memcached cache only if Cache::Memcached::Fast::Safe is installed.

We could fix the logic in Koha::Cache, but it seems better to make Cache::Memcached::Fast::Safe mandatory (especially after bug 13193).

Note that after bug 13193, you are loosing the Memcached cache, as Cache::Memcached::Fast::Safe will not get installed by default.
Comment 2 Julian Maurice 2020-02-12 14:30:44 UTC
Created attachment 98756 [details] [review]
Bug 24642: Make Cache::Memcached::Fast::Safe required

There is something wrong between the logic in Koha::Cache->new and our
dependencies.

We are having Cache::Memcached required, but
Cache::Memcached::Fast::Safe is optional.
However, in Koha::Cache->new we are initializing the Memcached cache
only if Cache::Memcached::Fast::Safe is installed.

We could fix the logic in Koha::Cache, but it seems better to make
Cache::Memcached::Fast::Safe mandatory (especially after bug 13193).

Note that after bug 13193, you are loosing the Memcached cache, as
Cache::Memcached::Fast::Safe will not get installed by default.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Comment 3 Martin Renvoize 2020-02-12 16:22:52 UTC
Created attachment 98761 [details] [review]
Bug 24642: Make Cache::Memcached::Fast::Safe required

There is something wrong between the logic in Koha::Cache->new and our
dependencies.

We are having Cache::Memcached required, but
Cache::Memcached::Fast::Safe is optional.
However, in Koha::Cache->new we are initializing the Memcached cache
only if Cache::Memcached::Fast::Safe is installed.

We could fix the logic in Koha::Cache, but it seems better to make
Cache::Memcached::Fast::Safe mandatory (especially after bug 13193).

Note that after bug 13193, you are loosing the Memcached cache, as
Cache::Memcached::Fast::Safe will not get installed by default.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize 2020-02-12 16:24:53 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 5 Marcel de Rooy 2020-02-24 08:54:57 UTC
Should we add a dependency reference here for Cache::Memcached::Fast::Safe ?
Strange that it is not even mentioned anywhere on the report..
Comment 6 Joy Nelson 2020-03-05 00:55:21 UTC
enhancement - not pushed to 19.11.x  (see notes on 13193)
Comment 7 Joy Nelson 2020-03-05 23:52:47 UTC
Pushed to 19.11.x branch for 19.11.04
Comment 8 Lucas Gass 2020-03-18 16:27:58 UTC
choosing not to backport these to 19.05.x