Code review comment for ~hloeung/content-cache-charm:master

Revision history for this message
Joel Sing (jsing) wrote :

This has already been landed, however the naming seems a bit backwards - IMO it would be better to have background_cache_updates (even if that defaults to true), rather than a disable_cache_background_update that defaults to false (even enable_background_cache_updates would at least be consistent with the following option).

« Back to merge proposal