Code review comment for lp:goose

The attempt to merge lp:goose into lp:~go-bot/goose/test failed. Below is the output from the failed tests.

cd /home/tarmac/trees/src/launchpad.net/juju-core
bzr pull --overwrite
cd /home/tarmac/trees/src/launchpad.net/juju-core
bzr tags
cd /home/tarmac/trees/src/launchpad.net/juju-core
bzr update -r revno:-1
cd /home/tarmac/trees/src/labix.org/v2/mgo
bzr pull --overwrite
cd /home/tarmac/trees/src/labix.org/v2/mgo
bzr tags
cd /home/tarmac/trees/src/labix.org/v2/mgo
bzr update -r go1
cd /home/tarmac/trees/src/launchpad.net/goyaml
bzr pull --overwrite
cd /home/tarmac/trees/src/launchpad.net/goyaml
bzr tags
cd /home/tarmac/trees/src/launchpad.net/goyaml
bzr update -r revno:-1
cd /home/tarmac/trees/src/launchpad.net/gnuflag
bzr pull --overwrite
cd /home/tarmac/trees/src/launchpad.net/gnuflag
bzr tags
cd /home/tarmac/trees/src/launchpad.net/gnuflag
bzr update -r revno:-1
cd /home/tarmac/trees/src/code.google.com/p/go.net
hg pull
cd /home/tarmac/trees/src/code.google.com/p/go.net
hg tags
cd /home/tarmac/trees/src/code.google.com/p/go.net
hg branches
cd /home/tarmac/trees/src/code.google.com/p/go.net
hg update default
cd /home/tarmac/trees/src/launchpad.net/goamz
bzr pull --overwrite
cd /home/tarmac/trees/src/launchpad.net/goamz
bzr tags
cd /home/tarmac/trees/src/launchpad.net/goamz
bzr update -r revno:-1
cd /home/tarmac/trees/src/launchpad.net/gomaasapi
bzr pull --overwrite
cd /home/tarmac/trees/src/launchpad.net/gomaasapi
bzr tags
cd /home/tarmac/trees/src/launchpad.net/gomaasapi
bzr update -r revno:-1
cd /home/tarmac/trees/src/launchpad.net/goose
bzr pull --overwrite
cd /home/tarmac/trees/src/launchpad.net/tomb
bzr pull --overwrite
cd /home/tarmac/trees/src/launchpad.net/tomb
bzr tags
cd /home/tarmac/trees/src/launchpad.net/tomb
bzr update -r revno:-1
cd /home/tarmac/trees/src/launchpad.net/lpad
bzr pull --overwrite
cd /home/tarmac/trees/src/launchpad.net/lpad
bzr tags
cd /home/tarmac/trees/src/launchpad.net/lpad
bzr update -r revno:-1
cd /home/tarmac/trees/src/launchpad.net/gocheck
bzr pull --overwrite
cd /home/tarmac/trees/src/launchpad.net/gocheck
bzr tags
cd /home/tarmac/trees/src/launchpad.net/gocheck
bzr update -r revno:-1

----------------------------------------------------------------------
PANIC: local_test.go:39: localLiveSuite.SetUpSuite

Using identity service test double
... Panic: interface conversion: identityservice.IdentityService is *identityservice.KeyPair, not *identityservice.UserPass (PC=0x4112F8)

/build/buildd/golang-1/src/pkg/runtime/proc.c:1443
  in panic
/build/buildd/golang-1/src/pkg/runtime/iface.c:247
  in assertI2Tret
/build/buildd/golang-1/src/pkg/runtime/iface.c:227
  in assertI2T
/home/tarmac/trees/src/launchpad.net/goose/testservices/openstackservice/openstack.go:52
  in New
local_test.go:54
  in localLiveSuite.SetUpSuite
OOPS: 16 passed, 4 skipped, 1 FIXTURE-PANICKED, 10 MISSED
--- FAIL: Test (0.01 seconds)
FAIL
FAIL launchpad.net/goose/client 0.025s
ok launchpad.net/goose/errors 0.007s
ok launchpad.net/goose/glance 0.012s
ok launchpad.net/goose/http 0.014s
ok launchpad.net/goose/identity 0.021s
ok launchpad.net/goose/nova 0.093s
ok launchpad.net/goose/swift 0.051s
ok launchpad.net/goose/sync 0.007s
ok launchpad.net/goose/testing/envsuite 0.010s
ok launchpad.net/goose/testing/httpsuite 0.014s
? launchpad.net/goose/testservices [no test files]
? launchpad.net/goose/testservices/cmd [no test files]
ok launchpad.net/goose/testservices/hook 0.008s
ok launchpad.net/goose/testservices/identityservice 0.022s
ok launchpad.net/goose/testservices/novaservice 0.063s
? launchpad.net/goose/testservices/openstackservice [no test files]
ok launchpad.net/goose/testservices/swiftservice 0.023s
ok launchpad.net/goose/tools/secgroup-delete-all 0.023s

Setting GOPATH to: /home/tarmac/trees
Running: go get -u -x launchpad.net/juju-core/...
# cd /home/tarmac/trees/src/launchpad.net/goose; bzr pull --overwrite
bzr: ERROR: No pull location known or specified.
package launchpad.net/juju-core/bzr
 imports launchpad.net/juju-core/cert
 imports launchpad.net/juju-core/charm
 imports labix.org/v2/mgo/bson
 imports launchpad.net/goyaml
 imports launchpad.net/juju-core/charm/hooks
 imports launchpad.net/juju-core/log
 imports launchpad.net/juju-core/schema
 imports launchpad.net/juju-core/cloudinit
 imports launchpad.net/juju-core/cmd
 imports launchpad.net/gnuflag
 imports launchpad.net/juju-core/version
 imports launchpad.net/juju-core/cmd/builddb
 imports code.google.com/p/go.net/websocket
 imports labix.org/v2/mgo
 imports labix.org/v2/mgo/txn
 imports launchpad.net/goamz/aws
 imports launchpad.net/goamz/ec2
 imports launchpad.net/goamz/s3
 imports launchpad.net/gomaasapi
 imports launchpad.net/goose: exit status 3
WARN: Failed to update launchpad.net/juju-core
Running: go fmt ./...
Running: go build ./...
Running: go test ./...
warning: building out-of-date packages:
 launchpad.net/goose
installing these packages with 'go test -i ./...' will speed future tests.

FAIL: failed running: go test ./...