Code review comment for lp://staging/~fwereade/juju-core/environs-tools-final-replacement

Revision history for this message
William Reade (fwereade) wrote :

Reviewers: mp+158830_code.launchpad.net,

Message:
Please take a look.

Description:
cmd/juju: sync-tools fixes

sync-tools now requires the --dev flag before copying development
versions,
and no longer copies into a public bucket when the contents of that
bucket
are not available to the target environment.

This branch removes the last of the fuzzy multi-bucket tools code.

https://code.launchpad.net/~fwereade/juju-core/environs-tools-final-replacement/+merge/158830

Requires:
https://code.launchpad.net/~fwereade/juju-core/environs-tools-provisioning-integration/+merge/158805

(do not edit description out of merge proposal)

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

Affected files:
   A [revision details]
   M cmd/juju/synctools.go
   M cmd/juju/synctools_test.go
   M environs/tools.go
   M environs/tools_test.go

« Back to merge proposal