all: vet, fix unreachable errors
fix spurious report of passing a lock (embedded in the struct value) by value
gofmt
Merge pull request #228 from davecheney/123-fix-vet-unreachable-errors

all: vet, fix unreachable errors

Fix a bunch of simple vet errors obscuring some more important ones.
update goose dependency, fix lp 1336634
Merge pull request #231 from davecheney/125-update-goose-dependency-fix-lp-1336634

update goose dependency, fix lp 1336634
Modified config file to create all-machines.log with 0600 permissions.
juju/testing: do not tear down connection if setup fails
More fixes
Merge pull request #232 from katco-/b1286518-log-perms

Modified config file to create all-machines.log with 0600 permissions.

https://bugs.launchpad.net/juju-core/+bug/1286518
responding to review feedback
Merge pull request #233 from davecheney/126-do-not-close-state-and-apiserver-connections-if-setup-fails

juju/testing: do not tear down connection if setup fails

Our test suite helpers are a double edged sword. They tend to expect the best, and blow up when the worst happens. This change is an example of this, if the setup phase failed, then what was a c.Assert("couldn't talk to mongo") becomes a panic when the tear down phase tries to close a connection which was never opened.
Merge branch 'master' of https://github.com/juju/juju into 119-version-tests
Responding to review feedback