Override cache settings during rb-site invocation.

Review Request #12457 — Created July 11, 2022 and submitted — Latest diff uploaded

Information

Review Board
release-5.0.x

Reviewers

The change to move from MemcachedCache to PymemcacheCache broke
things when upgrading from a Python 2-based Review Board 3.0.x to Python
3-based 5.0 beta 1. The problem is that various code paths during
django.start() ended up triggering cache operations, and trying to set
up the old (pre-migrated) MemcachedCache would fail due to a missing
module. We migrate this setting during the upgrade process, but we never
got that far.

This change fixes rb-site to override any stored cache settings and just
use the local-memory cache for the duration of the command. This is
guaranteed to work regardless of what modules happen to be installed.

Ran rb-site upgrade on a site where the old environment used the
memcached package, and the new environment used the pymemcache
package.

Commits

Files