Code review comment for lp:~thumper/juju-core/move-to-juju-loggo

Revision history for this message
Tim Penhey (thumper) wrote :

Reviewers: mp+209530_code.launchpad.net,

Message:
Please take a look.

Description:
Move loggo import to be under github.com/juju

Just updates imports and dependency hash.

https://code.launchpad.net/~thumper/juju-core/move-to-juju-loggo/+merge/209530

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/70010053/

Affected files (+107, -105 lines):
   A [revision details]
   M agent/agent.go
   M agent/tools/tools.go
   M charm/testing/mockstore.go
   M cloudinit/sshinit/configure.go
   M cmd/juju/main.go
   M cmd/juju/synctools.go
   M cmd/juju/synctools_test.go
   M cmd/jujud/machine.go
   M cmd/jujud/main.go
   M cmd/jujud/run_test.go
   M cmd/jujud/unit.go
   M cmd/logging.go
   M cmd/logging_test.go
   M cmd/plugins/juju-metadata/metadata.go
   M cmd/plugins/juju-metadata/signmetadata.go
   M cmd/plugins/juju-metadata/signmetadata_test.go
   M cmd/plugins/juju-metadata/toolsmetadata.go
   M cmd/plugins/juju-metadata/toolsmetadata_test.go
   M cmd/plugins/juju-restore/restore.go
   M cmd/plugins/local/main.go
   M cmd/supercommand.go
   M container/kvm/kvm.go
   M container/kvm/kvm_test.go
   M container/kvm/live_test.go
   M container/lxc/lxc.go
   M container/lxc/lxc_test.go
   M container/userdata.go
   M dependencies.tsv
   M environs/bootstrap/bootstrap.go
   M environs/config.go
   M environs/config/config.go
   M environs/config/config_test.go
   M environs/config_test.go
   M environs/configstore/disk.go
   M environs/imagemetadata/upload.go
   M environs/manual/provisioner.go
   M environs/simplestreams/simplestreams.go
   M environs/sync/sync.go
   M environs/testing/bootstrap.go
   M environs/tools/tools.go
   M environs/tools/tools_test.go
   M juju/api.go
   M log/log.go
   M log/log_test.go
   M provider/azure/environprovider.go
   M provider/common/bootstrap.go
   M provider/common/bootstrap_test.go
   M provider/dummy/environs.go
   M provider/ec2/ec2.go
   M provider/joyent/provider.go
   M provider/local/environprovider.go
   M provider/local/environprovider_test.go
   M provider/maas/environprovider.go
   M provider/manual/environ.go
   M provider/openstack/config_test.go
   M provider/openstack/provider.go
   M rpc/jsoncodec/codec.go
   M rpc/jsoncodec/codec_test.go
   M rpc/server.go
   M state/apiserver/apiserver.go
   M state/apiserver/charmrevisionupdater/updater.go
   M state/apiserver/client/client.go
   M state/apiserver/keymanager/keymanager.go
   M state/apiserver/logger/logger.go
   M state/apiserver/login_test.go
   M state/apiserver/pinger_test.go
   M state/state.go
   M state/unit.go
   M state/watcher.go
   M testing/checkers/log.go
   M testing/checkers/log_test.go
   M testing/testbase/log.go
   M tools/tools.go
   M upgrades/lockdirectory_test.go
   M upgrades/upgrade.go
   M utils/apt.go
   M utils/exec/exec.go
   M utils/fslock/fslock.go
   M utils/network.go
   M utils/ssh/authorisedkeys.go
   M worker/authenticationworker/worker.go
   M worker/deployer/deployer.go
   M worker/environ.go
   M worker/firewaller/firewaller.go
   M worker/instancepoller/observer_test.go
   M worker/instancepoller/updater.go
   M worker/localstorage/worker.go
   M worker/logger/logger.go
   M worker/logger/logger_test.go
   M worker/machineenvironmentworker/machineenvironmentworker.go
   M worker/machiner/machiner.go
   M worker/minunitsworker/minunitsworker.go
   M worker/minunitsworker/minunitsworker_test.go
   M worker/peergrouper/desired.go
   M worker/provisioner/kvm-broker.go
   M worker/provisioner/lxc-broker.go
   M worker/provisioner/provisioner.go
   M worker/rsyslog/worker.go
   M worker/uniter/context.go
   M worker/uniter/filter.go
   M worker/uniter/jujuc/juju-log.go
   M worker/uniter/jujuc/juju-log_test.go
   M worker/uniter/jujuc/server.go
   M worker/uniter/uniter.go
   M worker/upgrader/upgrader.go

« Back to merge proposal