Code review comment for lp:~freyes/charms/trusty/percona-cluster/lp1426508

Revision history for this message
James Page (james-page) wrote :

Felipe

Thinking about 'local.yaml' - that's a bit tricky for our automated testing tooling - however using a environment variable is not - (I see 'VIP' in the code already).

Please could you scope that to be AMULET_OS_VIP and pass it through in the Makefile:

   @juju test -v -p AMULET_HTTP_PROXY,AMULET_OS_VIP --timeout 900

review: Needs Fixing

« Back to merge proposal