Code review comment for lp:~frankban/charms/precise/juju-gui/support-firewall

Revision history for this message
Richard Harding (rharding) wrote :

LGTM QA ok

For QA I did a deploy, ssh'd into the box to verify that the ppa,
apache/haproxy packages were not installed.

Checked the gui worked fine.

Then changed the config for the server. SSH'd back into the box to
verify that packages were then installed. Gui still worked fine.

https://codereview.appspot.com/14433049/

« Back to merge proposal