Juju Charms Collection
squid-reverseproxy package

Merge lp://staging/~michael.nelson/charms/trusty/squid-reverseproxy/add-max-obj-size-mem into lp://staging/charms/trusty/squid-reverseproxy

  1. Trusty Tahr (14.04)
  2. add-max-obj-size-mem
  3. Merge into trunk
Proposed by Michael Nelson
Status: Merged
Merged at revision: 56
Proposed branch: lp://staging/~michael.nelson/charms/trusty/squid-reverseproxy/add-max-obj-size-mem
Merge into: lp://staging/charms/trusty/squid-reverseproxy
Diff against target: 26 lines (+5/-0)
2 files modified
config.yaml (+4/-0)
templates/main_config.template (+1/-0)
To merge this branch: bzr merge lp://staging/~michael.nelson/charms/trusty/squid-reverseproxy/add-max-obj-size-mem
Reviewer Review Type Date Requested Status
Charles Butler (community) Approve
Review via email: mp+264664@code.staging.launchpad.net

Commit message

Add max_obj_size_in_mem_kb option. Bug 1474176

Description of the change

Tested locally with:

$ juju ssh sca-cache/0 "sudo cat /etc/squid3/squid.conf | grep maximum_object"
maximum_object_size 8192 KB

$ make manifests/upgrade-deployment
...
########################
# Succesfully verified #
########################
########################
# Nagios Checks Passed #
########################

$ juju ssh sca-cache/0 "sudo cat /etc/squid3/squid.conf | grep maximum_object"
maximum_object_size 8192 KB
maximum_object_size_in_memory 512 KB

To post a comment you must log in.
Revision history for this message
Charles Butler (lazypower) wrote :

+1 LGTM. Thank you for taking the time to submit this fix for the charm store. We appreciate your work. I've merged this branch and it should be available in the charm store after the next ingestion.

If you have any questions/comments/concerns about the review contact us in #juju on irc.freenode.net or email the mailing list <email address hidden>, or ask a question tagged with "juju" on http://askubuntu.com.

review: Approve

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

to all changes: