support multiple cache-valid(ity)/proxy_cache_valid directives
Bug #1873116 reported by
Paul Collins
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Content Cache Charm |
Fix Released
|
High
|
Haw Loeung |
Bug Description
Currently cache-valid(ity) has to be a single item:
- cache-valid: 200 1h
or
- cache-valid: 200 401 1h
which maps to
proxy_cache_valid 200 1h;
or
proxy_cache_valid 200 401 1h;
Ideally we could also supply a list so that, say
- cache-valid:
- 200 301 1h
- 401 15m
would generate
proxy_cache_valid 200 301 1h;
proxy_cache_valid 401 15m;
which seems to be what is needed to get different lifetimes: https:/
Related branches
~hloeung/content-cache-charm:nginx-config
- Stuart Bishop (community): Approve
- Content Cache Charmers: Pending requested
-
Diff: 137 lines (+33/-7)6 files modifiedlib/nginx.py (+17/-2)
templates/nginx_cfg.tmpl (+7/-1)
tests/unit/files/config_test_config.txt (+3/-0)
tests/unit/files/nginx_config_rendered_test_output-site3.local.txt (+1/-1)
tests/unit/files/nginx_config_rendered_test_output-site8.local.txt (+3/-2)
tests/unit/files/nginx_config_rendered_test_output-site9.local.txt (+2/-1)
description: | updated |
Changed in content-cache-charm: | |
status: | New → In Progress |
importance: | Undecided → Medium |
assignee: | nobody → Haw Loeung (hloeung) |
importance: | Medium → High |
Changed in content-cache-charm: | |
status: | In Progress → Fix Released |
To post a comment you must log in.