Merge lp://staging/~rogpeppe/juju-core/upgrader into lp://staging/~juju/juju-core/trunk

Proposed by Roger Peppe
Status: Work in progress
Proposed branch: lp://staging/~rogpeppe/juju-core/upgrader
Merge into: lp://staging/~juju/juju-core/trunk
Diff against target: 1018 lines (+974/-2)
7 files modified
cmd/upgrader/testprog.go (+71/-0)
cmd/upgrader/upgrader.go (+250/-0)
cmd/upgrader/upgrader_test.go (+333/-0)
state/internal_test.go (+2/-2)
upgrade/msg.go (+125/-0)
upgrade/upgrade.go (+89/-0)
upgrade/upgrade_test.go (+104/-0)
To merge this branch: bzr merge lp://staging/~rogpeppe/juju-core/upgrader
Reviewer Review Type Date Requested Status
The Go Language Gophers Pending
Review via email: mp+109325@code.staging.launchpad.net

Description of the change

upgrader: new command and upgrade package

https://codereview.appspot.com/6307061/

To post a comment you must log in.
215. By Roger Peppe

upgrade: tests for reading and writing messages

216. By Roger Peppe

upgrade: add tests for message parsing

217. By Roger Peppe

cmd/upgrader: towards testing

218. By Roger Peppe

upgrader: tests WIP

219. By Roger Peppe

upgrader: finish implementation of tests

220. By Roger Peppe

go fmt

Unmerged revisions

220. By Roger Peppe

go fmt

219. By Roger Peppe

upgrader: finish implementation of tests

218. By Roger Peppe

upgrader: tests WIP

217. By Roger Peppe

cmd/upgrader: towards testing

216. By Roger Peppe

upgrade: add tests for message parsing

215. By Roger Peppe

upgrade: tests for reading and writing messages

214. By Roger Peppe

upgrader: new package and command

213. By Roger Peppe

state: store units inside their respective services

Making this change simplifies a lot of code - no need
to keep a separate sequence numbering system for units,
for example.

(original proposal at https://codereview.appspot.com/6247066/)

R=
CC=
https://codereview.appspot.com/6300060

212. By Gustavo Niemeyer

.lbox: update to juju-core

211. By Gustavo Niemeyer

Translated all paths to use the juju-core project in Launchpad.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches