Content Cache Charm

Allow cache_inactive_time on per site basis

Bug #1922927 reported by Benjamin Allot
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Content Cache Charm
Fix Released
High
Haw Loeung

Bug Description

The current cache_inactive_time setting is used for all locations/sites served by the charm.

We should be able to tweak it on a site basis.
I think a key "cache_inactive_time" under the site key itself would do, this latter overwriting the global one if set.

Related branches

~hloeung/content-cache-charm:nginx-site-cache-inactive-time
James Simpson: Approve
Canonical IS Reviewers: Pending requested
Diff: 91 lines (+11/-7)
4 files modified
reactive/content_cache.py (+3/-2)
tests/unit/files/config_test_config.txt (+1/-0)
tests/unit/files/nginx_config_rendered_test_output-site11.local.txt (+1/-1)
tests/unit/test_nginx.py (+6/-4)
~hloeung/content-cache-charm:nginx-extra-server-configs
James Simpson: Approve
Guillermo Gonzalez (community): Approve
Canonical IS Reviewers: Pending requested
Diff: 584 lines (+256/-16)
15 files modified
lib/nginx.py (+10/-5)
reactive/content_cache.py (+1/-0)
templates/nginx_cfg.tmpl (+8/-0)
tests/unit/files/config_test_config.txt (+12/-0)
tests/unit/files/content_cache_rendered_haproxy_test_output.txt (+26/-2)
tests/unit/files/content_cache_rendered_haproxy_test_output_auto_maxconns.txt (+26/-2)
tests/unit/files/content_cache_rendered_haproxy_test_output_load_balancing_algorithm.txt (+26/-2)
tests/unit/files/content_cache_rendered_haproxy_test_output_processes_and_threads.txt (+26/-2)
tests/unit/files/content_cache_rendered_haproxy_test_output_processes_and_threads_haproxy2.txt (+27/-2)
tests/unit/files/haproxy_config_rendered_backends_stanzas_test_output.txt (+10/-0)
tests/unit/files/haproxy_config_rendered_listen_stanzas_test_output.txt (+1/-0)
tests/unit/files/haproxy_config_rendered_test_output.txt (+11/-0)
tests/unit/files/haproxy_config_rendered_test_output2.txt (+11/-0)
tests/unit/files/nginx_config_rendered_test_output-site11.local.txt (+39/-0)
tests/unit/test_nginx.py (+22/-1)
Revision history for this message
Haw Loeung (hloeung) wrote :

https://code.launchpad.net/~hloeung/content-cache-charm/+git/content-cache-charm/+merge/414774 allows specifying extra configs such as "cache_inactive_time" per site/server. But I think we can do one better and use that to override cache_inactive_time for each of the proxy_cache_path.

Changed in content-cache-charm:
status: New → In Progress
assignee: nobody → Haw Loeung (hloeung)
importance: Undecided → High
Haw Loeung (hloeung)
Changed in content-cache-charm:
status: In Progress → Fix Committed
Revision history for this message
Haw Loeung (hloeung) wrote :

Released as cs:~content-cache-charmers/content-cache-94

Changed in content-cache-charm:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.