Closed Bug 1377807 Opened 7 years ago Closed 7 years ago

Move memcached options from data/params to localconfig

Categories

(bugzilla.mozilla.org :: General, enhancement)

Production
enhancement
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: dylan, Assigned: umohm12)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

45 bytes, text/x-github-pull-request
dylan
: review+
Details | Review
So in order to get bug 1355463 done, we'll need to move this out of data/params
and into the localconfig file. 

remove Bugzilla/Config/Memcached.pm
remove template/en/default/admin/params/memcached.html.tmpl
remove mention from docs/en/rst/administering/parameters.rst

edit Bugzilla/Install/Localconfig.pm add memcached_servers and memcached_namespace to LOCALCONFIG_VARS
add description of both options to ./template/en/default/setup/strings.txt.pl.

Bugzilla/Memcached.pm will need to be updated to use Bugzilla->localconfig->{memcached_servers} and Bugzilla->localconfig->{memcached_namespace}.

Note that Memcached.pm uses Bugzilla->get_param_with_override() currently -- that can go away.
Attached file PR
Attachment #8883063 - Flags: review?(dylan)
Attachment #8883063 - Flags: review?(dylan) → review-
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Attachment #8883063 - Flags: review- → review+
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: