Juju Charms Collection
ceph-radosgw package

Merge lp://qastaging/~brad-marshall/charms/trusty/ceph-radosgw/haproxy-add-timeout into lp://qastaging/~openstack-charmers-archive/charms/trusty/ceph-radosgw/next

  1. Trusty Tahr (14.04)
  2. haproxy-add-timeout
  3. Merge into next
Proposed by Brad Marshall
Status: Needs review
Proposed branch: lp://qastaging/~brad-marshall/charms/trusty/ceph-radosgw/haproxy-add-timeout
Merge into: lp://qastaging/~openstack-charmers-archive/charms/trusty/ceph-radosgw/next
Diff against target: 19 lines (+12/-0)
1 file modified
config.yaml (+12/-0)
To merge this branch: bzr merge lp://qastaging/~brad-marshall/charms/trusty/ceph-radosgw/haproxy-add-timeout
Reviewer Review Type Date Requested Status
Liam Young (community) Disapprove
Review via email: mp+279715@code.qastaging.launchpad.net

Description of the change

Expose haproxy server and client timeouts via config.

To post a comment you must log in.
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_lint_check #15037 ceph-radosgw-next for brad-marshall mp279715
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/15037/

Reply
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #14025 ceph-radosgw-next for brad-marshall mp279715
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/14025/

Reply
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_amulet_test #8200 ceph-radosgw-next for brad-marshall mp279715
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/8200/

Reply
Revision history for this message
David Ames (thedac) wrote :

Brad,

Seems we have been working on the same issue at the same time.

All the MPs associated with this bug cover your MPs.
https://bugs.launchpad.net/charms/+source/cinder/+bug/1522130

I have also exposed connect and queue timeouts.

Those should land in the next couple of days.

Reply
Revision history for this message
Liam Young (gnuoy) wrote :

thedacs branch has landed into /next which supercedes this one, but thank you for taking the time to raise the mp.

review: Disapprove
Reply

Unmerged revisions

53. By Brad Marshall

[brad-marshall] Expose haproxy server and client timeouts via config.

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