Comment 7 for bug 1413775

Revision history for this message
Cory Johns (johnsca) wrote :

Charles,

Thank you for this contribution, and this is an exciting step toward getting better Docker integration with Juju. I had a couple of minor notes, but overall this seems good. +1

* The deployment instructions in the README are out of date and still direct to install --to flannel

* The Amulet tests contain `amulet.raise_status()` statements, which don't play well with the unittest.TestCase pattern. Those should be changed to `self.assertEqual()` or at least `self.fail()`.