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

Issue 33870043: revert to 1.16.3

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 5 months ago by jameinel
Modified:
10 years, 5 months ago
Reviewers:
mp+196875, natefinch
Visibility:
Public.

Description

revert to 1.16.3 This just reverts all of the items that have landed since 1.16.3. (By doing "bzr revert -r tag:juju-1.16.3"). The actual changes that were originally planned for 1.16.4 have been pushed up to lp:~jameinel/juju-core/preparation-for-1.16.5 They have been prepared in such a way that when we have done a 1.16.4 release, we can just merge that branch, and we'll get back all the old changes with their revision history intact. (So bzr annotate, etc will all give the right details.) The first follow up to this branch is going to be merging lp:~juju/juju-core/1.16.2+update into 1.16 (since we actually want it to become 1.16.3 + updates.) https://code.launchpad.net/~jameinel/juju-core/rollback-to-1.16.3/+merge/196875 (do not edit description out of merge proposal)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+490 lines, -1058 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M cmd/juju/destroymachine.go View 2 chunks +3 lines, -76 lines 0 comments Download
M cmd/juju/destroymachine_test.go View 5 chunks +14 lines, -58 lines 0 comments Download
M cmd/juju/destroyunit.go View 2 chunks +8 lines, -3 lines 0 comments Download
M cmd/jujud/machine_test.go View 1 chunk +1 line, -13 lines 0 comments Download
M dependencies.tsv View 1 chunk +1 line, -1 line 0 comments Download
M rpc/client.go View 2 chunks +0 lines, -8 lines 0 comments Download
M rpc/reflect_test.go View 2 chunks +0 lines, -2 lines 0 comments Download
M scripts/win-installer/setup.iss View 1 chunk +1 line, -1 line 0 comments Download
M state/api/client.go View 2 chunks +1 line, -13 lines 0 comments Download
M state/api/params/params.go View 1 chunk +0 lines, -6 lines 0 comments Download
M state/apiserver/client/client.go View 4 chunks +1 line, -56 lines 0 comments Download
M state/apiserver/client/client_test.go View 1 chunk +0 lines, -139 lines 0 comments Download
M state/apiserver/client/perm_test.go View 1 chunk +1 line, -1 line 0 comments Download
D state/cleanup.go View 1 chunk +0 lines, -208 lines 0 comments Download
D state/cleanup_test.go View 1 chunk +0 lines, -229 lines 0 comments Download
M state/life.go View 1 chunk +1 line, -8 lines 0 comments Download
M state/life_test.go View 5 chunks +15 lines, -5 lines 0 comments Download
M state/machine.go View 2 chunks +3 lines, -20 lines 0 comments Download
M state/machine_test.go View 5 chunks +44 lines, -13 lines 0 comments Download
M state/relationunit_test.go View 26 chunks +87 lines, -88 lines 0 comments Download
M state/service.go View 1 chunk +3 lines, -7 lines 0 comments Download
M state/service_test.go View 2 chunks +3 lines, -3 lines 0 comments Download
M state/state.go View 2 chunks +157 lines, -0 lines 0 comments Download
M state/state_test.go View 1 chunk +33 lines, -0 lines 0 comments Download
A state/statecmd/destroyunit.go View 1 chunk +16 lines, -0 lines 0 comments Download
M state/unit.go View 1 chunk +0 lines, -22 lines 0 comments Download
M state/unit_test.go View 10 chunks +94 lines, -77 lines 0 comments Download
M version/version.go View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2
jameinel
Please take a look.
10 years, 5 months ago (2013-11-27 12:21:49 UTC) #1
natefinch
10 years, 5 months ago (2013-11-27 12:42:28 UTC) #2
LGTM
Sign in to reply to this message.

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