znuny / Znuny

Znuny/Znuny LTS is a fork of the ((OTRS)) Community Edition, one of the most flexible web-based ticketing systems used for Customer Service, Help Desk, IT Service Management.
https://www.znuny.org
GNU General Public License v3.0
343 stars 83 forks source link

Memcached cache module for Znuny #417

Open pboguslawski opened 1 year ago

pboguslawski commented 1 year ago

Proposed change

This mod allows one to implement enterprise, multiserver Znuny installations with shared cache moved from slow NFS-like solutions to faster Memcached server.

Requires Cache::Memcached::Fast installed (libcache-memcached-fast-perl package in Debian).

Cache benchmarking possible with otrs.Console.pl Dev::Tools::CacheBenchmark.

Note: memcached service (i.e. memcached -m 256 -I 16m from official image) must be listening at address configured with Cache::Module::Memcached###Servers for cache unit tests to pass. This CI adjustment is not a part of this PR.

Type of change

Additional information

Author-Change-Id: IB#1046574

Checklist

joserribeiro26 commented 11 months ago

@rkaldung @dennykorsukewitz Is this feature on the roadmap for the next versions? It's very interesting for scalability, we use it here when we set up a good environment with multiple front-ends.

rkaldung commented 9 months ago

@rkaldung @dennykorsukewitz Is this feature on the roadmap for the next versions? It's very interesting for scalability, we use it here when we set up a good environment with multiple front-ends.

@joserribeiro26 Either this PR here or https://github.com/znuny/Znuny/pull/112 would be great. But both PRs have any unit tests and this is no-go.

pboguslawski commented 9 months ago

But both PRs have any unit tests and this is no-go.

Redis is not Memcached and Memcached is not Redis so both PR's should be considered as separate features. Standard unit test checks all availabe cache backends so no need for separate unit tests for this PR.

rkaldung commented 9 months ago

But both PRs have any unit tests and this is no-go.

Redis is not Memcached and Memcached is not Redis so both PR's should be considered as separate features. Standard unit test checks all availabe cache backends so no need for separate unit tests for this PR.

👍 Who said they are identical? If all backends are covered by the default unit test that's fine, never checked this. @dennykorsukewitz Can you confirm that existing unit tests cover also this PR?