Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1784)

Issue 14031043: Remove legacy tools logic (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 7 months ago by wallyworld
Modified:
10 years, 6 months ago
Reviewers:
mp+187993, fwereade
Visibility:
Public.

Description

Remove legacy tools logic This branch removes all legacy tools logic. Simplestreams is the only way tools can be located. There were a few consequences of this change. Simplestreams does not allow fake series like "series" to be used. So the testing charm repository was changed to be based on "quantal" since no test machine will be running on quantal. A large number of tests were updated because of this, but the chanes were just changing "series" to "quantal". Another impact was on the upgrade tests. The legacy tools search logic used to look at the public bucket if no major version match was found. Now, if any tools are found in a location, seaching stops. So tools located in a private bucket hide tools in a public bucket and the tests needed to change to reflect this. https://code.launchpad.net/~wallyworld/juju-core/remove-legacy-tools/+merge/187993 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Remove legacy tools logic #

Total comments: 4

Patch Set 3 : Remove legacy tools logic #

Unified diffs Side-by-side diffs Delta from patch set Stats (+666 lines, -870 lines) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M charm/repo_test.go View 7 chunks +14 lines, -14 lines 0 comments Download
M cmd/juju/bootstrap_test.go View 6 chunks +22 lines, -21 lines 0 comments Download
M cmd/juju/destroymachine_test.go View 1 chunk +1 line, -1 line 0 comments Download
M cmd/juju/scp_test.go View 1 chunk +1 line, -1 line 0 comments Download
M cmd/juju/ssh_test.go View 2 chunks +2 lines, -2 lines 0 comments Download
M cmd/juju/status_test.go View 41 chunks +54 lines, -54 lines 0 comments Download
M cmd/juju/upgradejuju_test.go View 1 4 chunks +87 lines, -113 lines 0 comments Download
M cmd/jujud/agent_test.go View 4 chunks +8 lines, -4 lines 0 comments Download
M cmd/jujud/machine_test.go View 2 chunks +3 lines, -3 lines 0 comments Download
M cmd/plugins/juju-metadata/toolsmetadata_test.go View 1 chunk +15 lines, -23 lines 0 comments Download
M environs/bootstrap/bootstrap_test.go View 7 chunks +14 lines, -8 lines 0 comments Download
M environs/cloudinit/cloudinit_test.go View 5 chunks +9 lines, -9 lines 0 comments Download
M environs/cloudinit_test.go View 1 chunk +1 line, -1 line 0 comments Download
M environs/jujutest/livetests.go View 6 chunks +6 lines, -0 lines 0 comments Download
M environs/jujutest/tests.go View 3 chunks +3 lines, -0 lines 0 comments Download
M environs/manual/bootstrap.go View 1 chunk +0 lines, -4 lines 0 comments Download
M environs/manual/provisioner_test.go View 1 chunk +1 line, -1 line 0 comments Download
M environs/open_test.go View 1 1 chunk +1 line, -0 lines 0 comments Download
M environs/sync/sync.go View 5 chunks +5 lines, -11 lines 0 comments Download
M environs/sync/sync_test.go View 6 chunks +0 lines, -17 lines 0 comments Download
M environs/testing/tools.go View 7 chunks +51 lines, -65 lines 0 comments Download
D environs/tools/legacytools.go View 1 chunk +0 lines, -40 lines 0 comments Download
M environs/tools/storage.go View 2 chunks +2 lines, -27 lines 0 comments Download
M environs/tools/storage_test.go View 4 chunks +8 lines, -30 lines 0 comments Download
M environs/tools/tools.go View 4 chunks +34 lines, -58 lines 0 comments Download
M environs/tools/tools_test.go View 7 chunks +31 lines, -89 lines 0 comments Download
M environs/tools/urls_test.go View 2 chunks +6 lines, -2 lines 0 comments Download
M juju/apiconn_test.go View 1 chunk +2 lines, -0 lines 0 comments Download
M juju/conn_test.go View 15 chunks +16 lines, -10 lines 0 comments Download
M juju/testing/conn.go View 3 chunks +4 lines, -2 lines 0 comments Download
M provider/dummy/environs.go View 3 chunks +7 lines, -4 lines 0 comments Download
M provider/ec2/httpstorage/httpstorage_test.go View 1 chunk +2 lines, -2 lines 0 comments Download
M provider/ec2/live_test.go View 3 chunks +1 line, -8 lines 0 comments Download
M provider/ec2/local_test.go View 3 chunks +3 lines, -0 lines 0 comments Download
M provider/maas/environ_test.go View 4 chunks +3 lines, -4 lines 0 comments Download
M provider/openstack/live_test.go View 1 chunk +1 line, -1 line 0 comments Download
M provider/openstack/local_test.go View 2 chunks +2 lines, -2 lines 0 comments Download
M state/api/deployer/deployer_test.go View 1 chunk +1 line, -1 line 0 comments Download
M state/api/params/params_test.go View 2 chunks +2 lines, -2 lines 0 comments Download
M state/api/provisioner/provisioner_test.go View 8 chunks +10 lines, -10 lines 0 comments Download
M state/api/uniter/uniter_test.go View 1 chunk +1 line, -1 line 0 comments Download
M state/apiserver/agent/agent_test.go View 1 chunk +3 lines, -3 lines 0 comments Download
M state/apiserver/client/api_test.go View 2 chunks +2 lines, -2 lines 0 comments Download
M state/apiserver/client/client_test.go View 4 chunks +4 lines, -4 lines 0 comments Download
M state/apiserver/client/perm_test.go View 3 chunks +4 lines, -4 lines 0 comments Download
M state/apiserver/deployer/deployer_test.go View 1 chunk +2 lines, -2 lines 0 comments Download
M state/apiserver/logger/logger_test.go View 1 chunk +1 line, -1 line 0 comments Download
M state/apiserver/machine/common_test.go View 1 chunk +2 lines, -2 lines 0 comments Download
M state/apiserver/provisioner/provisioner_test.go View 3 chunks +3 lines, -3 lines 0 comments Download
M state/apiserver/server_test.go View 4 chunks +4 lines, -4 lines 0 comments Download
M state/apiserver/uniter/uniter_test.go View 2 chunks +4 lines, -4 lines 0 comments Download
M state/apiserver/upgrader/upgrader_test.go View 1 chunk +1 line, -1 line 0 comments Download
M state/assign_test.go View 38 chunks +47 lines, -47 lines 0 comments Download
M state/charm_test.go View 4 chunks +5 lines, -5 lines 0 comments Download
M state/conn_test.go View 1 chunk +2 lines, -2 lines 0 comments Download
M state/export_test.go View 1 chunk +1 line, -1 line 0 comments Download
M state/life_test.go View 1 chunk +1 line, -1 line 0 comments Download
M state/machine_test.go View 17 chunks +19 lines, -19 lines 0 comments Download
M state/megawatcher_internal_test.go View 23 chunks +33 lines, -33 lines 0 comments Download
M state/service_test.go View 3 chunks +4 lines, -4 lines 0 comments Download
M state/state_test.go View 27 chunks +41 lines, -41 lines 0 comments Download
M state/tools_test.go View 1 chunk +1 line, -1 line 0 comments Download
M state/unit_test.go View 6 chunks +6 lines, -6 lines 0 comments Download
M testing/charm.go View 2 chunks +2 lines, -2 lines 0 comments Download
M worker/firewaller/firewaller_test.go View 3 chunks +3 lines, -3 lines 0 comments Download
M worker/provisioner/provisioner_test.go View 1 chunk +1 line, -1 line 0 comments Download
M worker/uniter/charm/charm_test.go View 2 chunks +2 lines, -2 lines 0 comments Download
M worker/uniter/state_test.go View 1 chunk +1 line, -1 line 0 comments Download
M worker/uniter/uniter_test.go View 4 chunks +4 lines, -4 lines 0 comments Download
M worker/upgrader/upgrader_test.go View 6 chunks +27 lines, -22 lines 0 comments Download

Messages

Total messages: 4
wallyworld
Please take a look.
10 years, 7 months ago (2013-09-27 08:57:37 UTC) #1
wallyworld
Please take a look.
10 years, 7 months ago (2013-09-27 13:24:59 UTC) #2
wallyworld
Please take a look.
10 years, 7 months ago (2013-09-27 13:33:46 UTC) #3
fwereade
10 years, 7 months ago (2013-09-27 14:02:20 UTC) #4
I would quibble that you probably didn't need to s/series/quantal/ across the
board, but that -- and the other comments -- are as nothing before the sweeping
tide of win that is this CL. LGTM.

https://codereview.appspot.com/14031043/diff/3001/environs/manual/bootstrap.go
File environs/manual/bootstrap.go (left):

https://codereview.appspot.com/14031043/diff/3001/environs/manual/bootstrap.g...
environs/manual/bootstrap.go:108: restore :=
envtools.SetToolPrefix(envtools.NewToolPrefix)
Lovely to see these go ;).

https://codereview.appspot.com/14031043/diff/3001/environs/testing/tools.go
File environs/testing/tools.go (left):

https://codereview.appspot.com/14031043/diff/3001/environs/testing/tools.go#o...
environs/testing/tools.go:86: func RemoveFakeToolsMetadata(c *gc.C, stor
storage.Storage) {
This feels a bit more like "RemoveToolsMetadata" -- it's not like it stops if it
discovers they're real ;p.

https://codereview.appspot.com/14031043/diff/3001/environs/tools/tools_test.go
File environs/tools/tools_test.go (right):

https://codereview.appspot.com/14031043/diff/3001/environs/tools/tools_test.g...
environs/tools/tools_test.go:47: s.LoggingSuite.SetUpSuite(c)
nice

https://codereview.appspot.com/14031043/diff/3001/provider/dummy/environs.go
File provider/dummy/environs.go (left):

https://codereview.appspot.com/14031043/diff/3001/provider/dummy/environs.go#...
provider/dummy/environs.go:272: // TODO(fwereade): get rid of these.
ahhh, that's better :)
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b