promote haproxy charm to trusty version in the charm store

Bug #1373081 reported by Darryl Weaver
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
haproxy (Juju Charms Collection)
Fix Released
Undecided
Unassigned

Bug Description

Please promote the HAproxy charm to trusty version in the charm store. For trusty only environments we need an official HAproxy charm to deploy from the charm store.

Revision history for this message
Charles Butler (lazypower) wrote :

Attached, verified as working with a 2 node ghost setup behind haproxy on trusty.

Attached minor cleanup to pass bundletester.

Changed in haproxy (Juju Charms Collection):
status: New → Fix Released
Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

@lazypower,

Overall looks good. I just ran a deploy of 3 units in front of apache, everything worked as expected. Please note that i didn't
tried any specific configuration option, just the vanilla installation.

Proofing charm.. OK
Tests runs .. OK
LInt ... OK

The test coverage , looks pretty good ;

hooks 553 80 86%

Only a few comments:

1) By default my system uses python 3.4 instead of 2.7, and this causes the 'make test' command to fail, since
the synchronized charmhelpers and the hooks code are not compatible with python 3.4, adding the following
lines will force the usage of 2.7.

=== modified file 'Makefile'
--- Makefile 2014-09-11 18:50:46 +0000
+++ Makefile 2014-10-01 22:29:02 +0000
@@ -19,7 +19,7 @@

 .venv:
  sudo apt-get install -y python-apt python-virtualenv
- virtualenv .venv --system-site-packages
+ virtualenv --python=python2.7 .venv --system-site-packages
  .venv/bin/pip install -I nose testtools mock pyyaml

 test: .venv

2) The following lines could be factorized using the @hooks.hook("hook_name") decorator:

    if hook_name == "install":
        install_hook()
    elif hook_name == "upgrade-charm":
        install_hook()
        config_changed()
        update_nrpe_config()
    elif hook_name == "config-changed":
        config_changed()
        update_nrpe_config()
    elif hook_name == "start":
        start_hook()
    elif hook_name == "stop":
        stop_hook()
    elif hook_name == "reverseproxy-relation-broken":
        config_changed()
    elif hook_name == "reverseproxy-relation-changed":
        reverseproxy_interface("changed")
    elif hook_name == "reverseproxy-relation-departed":
        reverseproxy_interface("departed")
    elif hook_name == "website-relation-joined":
        website_interface("joined")
    elif hook_name == "website-relation-changed":
        website_interface("changed")
    elif hook_name == "peer-relation-joined":
        website_interface("joined")
    elif hook_name == "peer-relation-changed":
        reverseproxy_interface("changed")
    elif hook_name in ("nrpe-external-master-relation-joined",
                       "local-monitors-relation-joined"):
        update_nrpe_config()
    else:
        print "Unknown hook

3) Method's documentation is not complaining with python docstrings ( http://legacy.python.org/dev/peps/pep-0257/ )

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.