Partial fix for LP-1351089
Fixed incorrect mgo import
Merge pull request #451 from mjs/fix-mgo-import

Fixed incorrect mgo import
LP-1351089 :: Isolate tests from users BASH_ENV
Add comments per suggestion from @axw
Fix various issues in live tests, dummy provider

MgoSuite was missing from dummy provider. Various
tests failed to bootstrap correctly, relying on
previous tests to do so; various tests were not
releasing resources properly, leaking mgo sessions.
provider/ec2: Fix fallout from live test changes
juju: add missing MgoSuite
juju/testing: keep tearing down if closing state fails

mongod processes were being left around because of
failures to close State objects. We fix this by
changing the Assert to Check.
Add reviewing checklist to docs/contributions. There is some overlap with the style guide. Where the style guide aims to be descriptive, this document is simply a list of questions for the reviewer to ask when when reading a PR.
Merge pull request #457 from axw/fix-tests-mgo-leakage

Fix tests mgo leakage

Various fixes to tests, mostly relating to usage of MgoSuite usage.
Merge branch 'master' into review-doc
Merge pull request #441 from waigani/review-doc

Add reviewing checklist to docs/contributions.

There is some overlap with the style guide. Where the style guide aims to be descriptive, this document is simply a list of questions for the reviewer to ask when when reading a PR.
Revert "Merge pull request #416 from axw/mongo-journalenabled"

This reverts commit 62e172632c3e9d8496805ed5223f9f4acc28986a, reversing
changes made to 8275fa461e1644ac073db177c5e456db24991a0c.
Merge pull request #454 from johnweldon/lp-1351089-isolate

LP-1351089 :: Isolate tests from users BASH_ENV

By suppressing bash from loading the BASH_ENV files
Merge pull request #450 from johnweldon/lp-1351089

Partial fix for LP-1351089

If the environment has `set -o noclobber` then `cat > $TMPFILE` will fail if $TMPFILE exists.

The test referenced in [lp-1351089](https://bugs.launchpad.net/juju-core/+bug/1351089) should still be isolated better from the environment, but this seems like a good change to make anyway.
Merge pull request #472 from mjs/fix-mongo-craziness

Revert "Merge pull request #416 from axw/mongo-journalenabled"

This reverts commit 62e172632c3e9d8496805ed5223f9f4acc28986a, reversing
changes made to 8275fa461e1644ac073db177c5e456db24991a0c.

The commit being reverted appears to cause mongo to consume large amounts of resources and get stuck when "juju ensure-availability" is run, effectively trashing the environment.
provider/manual: don't change config in Open/SetConfig

The manual provider's Open and SetConfig methods were
both using the result of "validate", which replaces
missing manual-specific attributes with their defaults.
Open and SetConfig should not alter the input config.

Fixes https://bugs.launchpad.net/juju-core/+bug/1347715
Increase socket timeout to 1 minute during bootstrap

We increase the socket timeout during bootstrap to
cope with slow disk I/O during machine initialisation.
Empirical evidence has shown this to decrease the
bootstrap failure rate on Microsoft Azure.

Fixes https://bugs.launchpad.net/juju-core/+bug/1351101
cmd/jujud: add test for InitializeState params