Code review comment for lp:~thumper/juju-core/better-local-destroy-environ

Revision history for this message
Tim Penhey (thumper) wrote :

Reviewers: mp+212544_code.launchpad.net,

Message:
Please take a look.

Description:
Add some tests for local provider destroy

During the force destroy part, forcably remove the
mongo and machine agent upstart scripts.

Tests are added to show that these are removed.
A drive by test was added to show that the containers
are destroyed too.

https://code.launchpad.net/~thumper/juju-core/better-local-destroy-environ/+merge/212544

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/79690043/

Affected files (+95, -25 lines):
   A [revision details]
   M provider/local/config_test.go
   M provider/local/environ.go
   M provider/local/environ_test.go
   M provider/local/export_test.go

« Back to merge proposal