Merge lp:~brad-marshall/charms/trusty/swift-proxy/haproxy-add-timeout into lp:~openstack-charmers-archive/charms/trusty/swift-proxy/next

Proposed by Brad Marshall
Status: Merged
Merged at revision: 128
Proposed branch: lp:~brad-marshall/charms/trusty/swift-proxy/haproxy-add-timeout
Merge into: lp:~openstack-charmers-archive/charms/trusty/swift-proxy/next
Diff against target: 19 lines (+12/-0)
1 file modified
config.yaml (+12/-0)
To merge this branch: bzr merge lp:~brad-marshall/charms/trusty/swift-proxy/haproxy-add-timeout
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Review via email: mp+278670@code.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 #14419 swift-proxy-next for brad-marshall mp278670
    LINT OK: passed

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

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

charm_unit_test #13441 swift-proxy-next for brad-marshall mp278670
    UNIT OK: passed

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

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

charm_amulet_test #8034 swift-proxy-next for brad-marshall mp278670
    AMULET FAIL: amulet-test failed

AMULET Results (max last 2 lines):
Timeout occurred (2700s), printing juju status...environment: osci-sv08
ERROR:root:Make target returned non-zero.

Full amulet test output: http://paste.ubuntu.com/13513054/
Build: http://10.245.162.77:8080/job/charm_amulet_test/8034/

Revision history for this message
Ryan Beisner (1chb1n) wrote :

FYI, undercloud issue caused amulet fail on #8034.

Rerunning tests...

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

charm_amulet_test #8053 swift-proxy-next for brad-marshall mp278670
    AMULET OK: passed

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

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

Approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'config.yaml'
--- config.yaml 2015-09-23 16:28:19 +0000
+++ config.yaml 2015-11-26 07:26:47 +0000
@@ -265,3 +265,15 @@
265 wait for you to execute the openstack-upgrade action for this charm on265 wait for you to execute the openstack-upgrade action for this charm on
266 each unit. If False it will revert to existing behavior of upgrading266 each unit. If False it will revert to existing behavior of upgrading
267 all units on config change.267 all units on config change.
268 haproxy-server-timeout:
269 type: int
270 default:
271 description: |
272 Server timeout configuration in ms for haproxy, used in HA configurations.
273 If not provided, default value of 30000ms is used.
274 haproxy-client-timeout:
275 type: int
276 default:
277 description: |
278 Client timeout configuration in ms for haproxy, used in HA
279 configurations. If not provided, default value of 30000ms is used.

Subscribers

People subscribed via source and target branches