Code review comment for lp:~wallyworld/juju-core/add-machine-panic

The attempt to merge lp:~wallyworld/juju-core/add-machine-panic into lp:juju-core failed. Below is the output from the failed tests.

ok launchpad.net/juju-core/agent 1.834s
ok launchpad.net/juju-core/agent/tools 25.645s
ok launchpad.net/juju-core/bzr 6.633s
ok launchpad.net/juju-core/cert 2.235s
ok launchpad.net/juju-core/charm 0.552s
? launchpad.net/juju-core/charm/hooks [no test files]
ok launchpad.net/juju-core/cloudinit 0.024s
ok launchpad.net/juju-core/cmd 0.306s
? launchpad.net/juju-core/cmd/builddb [no test files]
? 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 99.876s
ok launchpad.net/juju-core/cmd/jujud 48.141s
ok launchpad.net/juju-core/cmd/plugins/juju-metadata 0.885s
ok launchpad.net/juju-core/constraints 0.039s
ok launchpad.net/juju-core/container/lxc 0.331s
? launchpad.net/juju-core/container/lxc/mock [no test files]
ok launchpad.net/juju-core/downloader 5.282s
ok launchpad.net/juju-core/environs 1.236s
? launchpad.net/juju-core/environs/all [no test files]
ok launchpad.net/juju-core/environs/azure 3.034s
ok launchpad.net/juju-core/environs/cloudinit 0.482s
ok launchpad.net/juju-core/environs/config 0.696s
ok launchpad.net/juju-core/environs/dummy 15.957s
ok launchpad.net/juju-core/environs/ec2 181.906s
ok launchpad.net/juju-core/environs/imagemetadata 0.305s
ok launchpad.net/juju-core/environs/instances 0.238s
ok launchpad.net/juju-core/environs/jujutest 0.320s
ok launchpad.net/juju-core/environs/local 1.204s
? launchpad.net/juju-core/environs/local/storage [no test files]
ok launchpad.net/juju-core/environs/localstorage 0.203s
ok launchpad.net/juju-core/environs/maas 2.047s
ok launchpad.net/juju-core/environs/openstack 3.827s
? launchpad.net/juju-core/environs/provider [no test files]
ok launchpad.net/juju-core/environs/sync 0.273s
ok launchpad.net/juju-core/environs/testing 0.022s
? launchpad.net/juju-core/errors [no test files]
ok launchpad.net/juju-core/instance 0.032s
ok launchpad.net/juju-core/juju 11.628s
ok launchpad.net/juju-core/juju/osenv 0.237s
? launchpad.net/juju-core/juju/testing [no test files]
ok launchpad.net/juju-core/log 0.018s
ok launchpad.net/juju-core/log/syslog 0.027s
ok launchpad.net/juju-core/names 0.029s
ok launchpad.net/juju-core/rpc 0.259s
ok launchpad.net/juju-core/rpc/jsoncodec 0.222s
ok launchpad.net/juju-core/schema 0.020s
FAIL launchpad.net/juju-core/state [build failed]
? launchpad.net/juju-core/state/api [no test files]
ok launchpad.net/juju-core/state/api/agent 1.784s
? launchpad.net/juju-core/state/api/common [no test files]
ok launchpad.net/juju-core/state/api/deployer 5.224s
ok launchpad.net/juju-core/state/api/machiner 2.556s
ok launchpad.net/juju-core/state/api/params 0.032s
ok launchpad.net/juju-core/state/api/upgrader 2.869s
ok launchpad.net/juju-core/state/api/watcher 2.955s
ok launchpad.net/juju-core/state/apiserver 2.710s
ok launchpad.net/juju-core/state/apiserver/agent 2.529s
ok launchpad.net/juju-core/state/apiserver/client 13.238s
ok launchpad.net/juju-core/state/apiserver/common 0.050s
ok launchpad.net/juju-core/state/apiserver/deployer 4.181s
ok launchpad.net/juju-core/state/apiserver/machine 3.543s
? launchpad.net/juju-core/state/apiserver/testing [no test files]
ok launchpad.net/juju-core/state/apiserver/upgrader 3.770s
ok launchpad.net/juju-core/state/multiwatcher 0.957s
ok launchpad.net/juju-core/state/presence 6.686s
ok launchpad.net/juju-core/state/statecmd 6.664s
? launchpad.net/juju-core/state/testing [no test files]
ok launchpad.net/juju-core/state/watcher 5.520s
ok launchpad.net/juju-core/store 17.726s
ok launchpad.net/juju-core/testing 1.314s
ok launchpad.net/juju-core/testing/checkers 0.016s
ok launchpad.net/juju-core/thirdparty/pbkdf2 0.133s
ok launchpad.net/juju-core/upstart 0.116s
ok launchpad.net/juju-core/utils 0.969s
ok launchpad.net/juju-core/utils/fslock 1.056s
ok launchpad.net/juju-core/utils/set 0.020s
ok launchpad.net/juju-core/version 0.061s
ok launchpad.net/juju-core/worker 1.548s
ok launchpad.net/juju-core/worker/cleaner 1.123s
ok launchpad.net/juju-core/worker/deployer 3.054s
ok launchpad.net/juju-core/worker/firewaller 10.278s
ok launchpad.net/juju-core/worker/machiner 7.118s
ok launchpad.net/juju-core/worker/minunitsworker 1.129s
ok launchpad.net/juju-core/worker/provisioner 16.460s
ok launchpad.net/juju-core/worker/resumer 1.496s
ok launchpad.net/juju-core/worker/uniter 112.872s
ok launchpad.net/juju-core/worker/uniter/charm 2.346s
ok launchpad.net/juju-core/worker/uniter/hook 0.014s
ok launchpad.net/juju-core/worker/uniter/jujuc 0.468s
ok launchpad.net/juju-core/worker/uniter/relation 1.406s
ok launchpad.net/juju-core/worker/upgrader 2.338s

warning: building out-of-date packages:
 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.

# launchpad.net/juju-core/state_test
state/state_test.go:22: imported and not used: "launchpad.net/juju-core/names"