Comment 4 for bug 1392740

Revision history for this message
Whit Morriss (whitmo) wrote :

Howdy Wes!

To echo Marco, the tests are failing on the ansible syntax check since there is not a make target that installs ansible and puts the ansible-playbook command on the path. I'd suggest conditional creating a virtualenv for ansible in the makefile.

With an ansible based charm, unit-tests might not make any sense, so you could remove the test tar get and the generated unit-test directory. What would be super helpful is a few functional tests using amulet which will help us keep up with changes over time in the automated testing system.

Anyway, thanks for your work on this!

-w