Code review comment for lp:~thumper/juju-core/bootstrap-state-no-constraints-trunk

Revision history for this message
Andrew Wilkins (axwalk) wrote :

LGTM. Should log a couple of bugs: one to investigate why the quotes are
lost, and one one to fix the testing situation.

https://codereview.appspot.com/14494054/diff/4001/environs/cloudinit/cloudinit_test.go
File environs/cloudinit/cloudinit_test.go (right):

https://codereview.appspot.com/14494054/diff/4001/environs/cloudinit/cloudinit_test.go#newcode123
environs/cloudinit/cloudinit_test.go:123: // NOTE: this is terrible,
only want to test part of the results...
Indeed :(

https://codereview.appspot.com/14494054/

« Back to merge proposal