Merge ~hloeung/content-cache-charm:fixes into content-cache-charm:master

Proposed by Haw Loeung
Status: Merged
Approved by: Haw Loeung
Approved revision: 913574a7f928222be364355fdfc4ba963cd9ff5b
Merged at revision: 17b3f36cc4a16f47d9ab3697d058ad8a71eb2396
Proposed branch: ~hloeung/content-cache-charm:fixes
Merge into: content-cache-charm:master
Diff against target: 179 lines (+128/-4)
3 files modified
reactive/content_cache.py (+8/-4)
tests/unit/files/content_cache_rendered_haproxy_test_output_override_maxconns.txt (+92/-0)
tests/unit/test_content_cache.py (+28/-0)
Reviewer Review Type Date Requested Status
James Simpson Approve
Canonical IS Reviewers Pending
Review via email: mp+440894@code.qastaging.launchpad.net

Commit message

Allow overriding maxconn for sites/locations without backends - LP:2015989

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
James Simpson (jsimpso) wrote :

LGTM

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 17b3f36cc4a16f47d9ab3697d058ad8a71eb2396

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches