Code review comment for lp:~thumper/juju-core/fix-lxc-destroy

The attempt to merge lp:~thumper/juju-core/fix-lxc-destroy into lp:juju-core failed. Below is the output from the failed tests.

ok launchpad.net/juju-core/agent 1.316s
ok launchpad.net/juju-core/agent/tools 0.255s
ok launchpad.net/juju-core/bzr 7.544s
ok launchpad.net/juju-core/cert 3.531s
ok launchpad.net/juju-core/charm 0.614s
? launchpad.net/juju-core/charm/hooks [no test files]
? launchpad.net/juju-core/charm/testing [no test files]
ok launchpad.net/juju-core/cloudinit 0.052s
ok launchpad.net/juju-core/cloudinit/sshinit 1.014s
ok launchpad.net/juju-core/cmd 0.222s
ok launchpad.net/juju-core/cmd/charm-admin 0.880s
? launchpad.net/juju-core/cmd/charmd [no test files]
? launchpad.net/juju-core/cmd/charmload [no test files]
ok launchpad.net/juju-core/cmd/juju 241.599s
ok launchpad.net/juju-core/cmd/jujud 62.346s
ok launchpad.net/juju-core/cmd/plugins/juju-metadata 14.922s
? launchpad.net/juju-core/cmd/plugins/juju-restore [no test files]
ok launchpad.net/juju-core/constraints 0.027s
ok launchpad.net/juju-core/container 0.037s
ok launchpad.net/juju-core/container/factory 0.065s
ok launchpad.net/juju-core/container/kvm 0.290s
ok launchpad.net/juju-core/container/kvm/mock 0.060s
? launchpad.net/juju-core/container/kvm/testing [no test files]
ok launchpad.net/juju-core/container/lxc 0.356s
? launchpad.net/juju-core/container/lxc/mock [no test files]
? launchpad.net/juju-core/container/lxc/testing [no test files]
? launchpad.net/juju-core/container/testing [no test files]
ok launchpad.net/juju-core/downloader 5.343s
ok launchpad.net/juju-core/environs 3.339s
ok launchpad.net/juju-core/environs/bootstrap 4.637s
ok launchpad.net/juju-core/environs/cloudinit 0.637s
ok launchpad.net/juju-core/environs/config 3.166s
ok launchpad.net/juju-core/environs/configstore 0.042s
ok launchpad.net/juju-core/environs/filestorage 0.032s
ok launchpad.net/juju-core/environs/httpstorage 0.941s
ok launchpad.net/juju-core/environs/imagemetadata 0.660s
? launchpad.net/juju-core/environs/imagemetadata/testing [no test files]
ok launchpad.net/juju-core/environs/instances 0.061s
ok launchpad.net/juju-core/environs/jujutest 0.234s
ok launchpad.net/juju-core/environs/manual 10.738s
ok launchpad.net/juju-core/environs/simplestreams 0.323s
? launchpad.net/juju-core/environs/simplestreams/testing [no test files]
ok launchpad.net/juju-core/environs/sshstorage 1.235s
ok launchpad.net/juju-core/environs/storage 1.152s
ok launchpad.net/juju-core/environs/sync 34.858s
ok launchpad.net/juju-core/environs/testing 0.199s
ok launchpad.net/juju-core/environs/tools 7.066s
? launchpad.net/juju-core/environs/tools/testing [no test files]
ok launchpad.net/juju-core/errors 0.016s
ok launchpad.net/juju-core/instance 0.024s
? launchpad.net/juju-core/instance/testing [no test files]
ok launchpad.net/juju-core/juju 22.993s
ok launchpad.net/juju-core/juju/osenv 0.021s
? launchpad.net/juju-core/juju/testing [no test files]
ok launchpad.net/juju-core/log 0.016s
ok launchpad.net/juju-core/log/syslog 0.026s
ok launchpad.net/juju-core/names 0.027s
? launchpad.net/juju-core/provider [no test files]
? launchpad.net/juju-core/provider/all [no test files]
ok launchpad.net/juju-core/provider/azure 6.556s
ok launchpad.net/juju-core/provider/common 6.285s
ok launchpad.net/juju-core/provider/dummy 24.366s
ok launchpad.net/juju-core/provider/ec2 19.170s
ok launchpad.net/juju-core/provider/joyent 5.272s
ok launchpad.net/juju-core/provider/local 5.310s
ok launchpad.net/juju-core/provider/maas 20.679s
ok launchpad.net/juju-core/provider/null 1.258s
ok launchpad.net/juju-core/provider/openstack 32.138s

----------------------------------------------------------------------
PANIC: replicaset_test.go:295: MongoSuite.TestCurrentStatus

... Panic: runtime error: invalid memory address or nil pointer dereference (PC=0x413E71)

/usr/lib/go/src/pkg/runtime/panic.c:229
  in panic
/usr/lib/go/src/pkg/runtime/panic.c:487
  in panicstring
/usr/lib/go/src/pkg/runtime/os_linux.c:236
  in sigpanic
replicaset_test.go:359
  in MongoSuite.TestCurrentStatus
OOPS: 2 passed, 1 PANICKED
--- FAIL: TestPackage (17.99 seconds)
FAIL
FAIL launchpad.net/juju-core/replicaset 18.306s
ok launchpad.net/juju-core/rpc 0.086s
ok launchpad.net/juju-core/rpc/jsoncodec 0.031s
? launchpad.net/juju-core/rpc/rpcreflect [no test files]
ok launchpad.net/juju-core/schema 0.029s
ok launchpad.net/juju-core/state 97.691s
ok launchpad.net/juju-core/state/api 1.948s
ok launchpad.net/juju-core/state/api/agent 2.075s
? launchpad.net/juju-core/state/api/base [no test files]
ok launchpad.net/juju-core/state/api/charmrevisionupdater 1.517s
? launchpad.net/juju-core/state/api/common [no test files]
? launchpad.net/juju-core/state/api/common/testing [no test files]
ok launchpad.net/juju-core/state/api/deployer 6.116s
ok launchpad.net/juju-core/state/api/environment 1.856s
ok launchpad.net/juju-core/state/api/firewaller 11.488s
ok launchpad.net/juju-core/state/api/keymanager 4.255s
ok launchpad.net/juju-core/state/api/keyupdater 3.786s
ok launchpad.net/juju-core/state/api/logger 3.969s
ok launchpad.net/juju-core/state/api/machiner 4.039s
ok launchpad.net/juju-core/state/api/params 0.047s
ok launchpad.net/juju-core/state/api/provisioner 10.280s
ok launchpad.net/juju-core/state/api/uniter 28.998s
ok launchpad.net/juju-core/state/api/upgrader 3.955s
ok launchpad.net/juju-core/state/api/watcher 3.325s
ok launchpad.net/juju-core/state/apiserver 8.282s
ok launchpad.net/juju-core/state/apiserver/agent 3.089s
ok launchpad.net/juju-core/state/apiserver/charmrevisionupdater 2.493s
? launchpad.net/juju-core/state/apiserver/charmrevisionupdater/testing [no test files]
ok launchpad.net/juju-core/state/apiserver/client 43.608s
ok launchpad.net/juju-core/state/apiserver/common 2.059s
? launchpad.net/juju-core/state/apiserver/common/testing [no test files]
ok launchpad.net/juju-core/state/apiserver/deployer 4.700s
ok launchpad.net/juju-core/state/apiserver/environment 1.614s
ok launchpad.net/juju-core/state/apiserver/firewaller 6.186s
ok launchpad.net/juju-core/state/apiserver/keymanager 4.313s
? launchpad.net/juju-core/state/apiserver/keymanager/testing [no test files]
ok launchpad.net/juju-core/state/apiserver/keyupdater 3.100s
ok launchpad.net/juju-core/state/apiserver/logger 3.764s
ok launchpad.net/juju-core/state/apiserver/machine 3.221s
ok launchpad.net/juju-core/state/apiserver/provisioner 13.109s
? launchpad.net/juju-core/state/apiserver/testing [no test files]
ok launchpad.net/juju-core/state/apiserver/uniter 20.695s
ok launchpad.net/juju-core/state/apiserver/upgrader 5.991s
ok launchpad.net/juju-core/state/multiwatcher 0.754s
ok launchpad.net/juju-core/state/presence 6.357s
ok launchpad.net/juju-core/state/statecmd 1.798s
? launchpad.net/juju-core/state/testing [no test files]
ok launchpad.net/juju-core/state/watcher 5.856s
ok launchpad.net/juju-core/store 20.005s
ok launchpad.net/juju-core/testing 1.366s
ok launchpad.net/juju-core/testing/checkers 0.026s
ok launchpad.net/juju-core/testing/testbase 0.020s
ok launchpad.net/juju-core/thirdparty/pbkdf2 0.136s
ok launchpad.net/juju-core/tools 0.035s
ok launchpad.net/juju-core/upstart 1.317s
ok launchpad.net/juju-core/utils 2.403s
ok launchpad.net/juju-core/utils/exec 0.069s
ok launchpad.net/juju-core/utils/fslock 1.363s
ok launchpad.net/juju-core/utils/parallel 0.935s
ok launchpad.net/juju-core/utils/set 0.022s
ok launchpad.net/juju-core/utils/ssh 23.896s
? launchpad.net/juju-core/utils/ssh/testing [no test files]
ok launchpad.net/juju-core/utils/tailer 1.004s
ok launchpad.net/juju-core/version 0.068s
ok launchpad.net/juju-core/worker 6.626s
ok launchpad.net/juju-core/worker/authenticationworker 3.386s
ok launchpad.net/juju-core/worker/charmrevisionworker 2.027s
ok launchpad.net/juju-core/worker/cleaner 1.298s
ok launchpad.net/juju-core/worker/deployer 3.859s
ok launchpad.net/juju-core/worker/firewaller 12.800s
ok launchpad.net/juju-core/worker/instancepoller 2.348s
ok launchpad.net/juju-core/worker/localstorage 0.072s
ok launchpad.net/juju-core/worker/logger 1.652s
ok launchpad.net/juju-core/worker/machineenvironmentworker 7.949s
ok launchpad.net/juju-core/worker/machiner 3.267s
ok launchpad.net/juju-core/worker/minunitsworker 1.324s
ok launchpad.net/juju-core/worker/provisioner 30.815s
ok launchpad.net/juju-core/worker/resumer 1.727s
ok launchpad.net/juju-core/worker/terminationworker 0.100s
ok launchpad.net/juju-core/worker/uniter 180.150s
ok launchpad.net/juju-core/worker/uniter/charm 2.917s
ok launchpad.net/juju-core/worker/uniter/debug 1.139s
ok launchpad.net/juju-core/worker/uniter/hook 0.015s
ok launchpad.net/juju-core/worker/uniter/jujuc 0.481s
ok launchpad.net/juju-core/worker/uniter/relation 1.444s
ok launchpad.net/juju-core/worker/upgrader 3.576s

warning: building out-of-date packages:
 launchpad.net/lpad
 launchpad.net/goamz/ec2/ec2test
 launchpad.net/goamz/s3/s3test
 launchpad.net/goose/testservices/hook
 launchpad.net/goose/testservices/identityservice
 launchpad.net/goose/testservices
 launchpad.net/goose/testservices/novaservice
 launchpad.net/goose/testservices/swiftservice
 launchpad.net/goose/testservices/openstackservice
installing these packages with 'go test -i ./...' will speed future tests.