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

Issue 88790043: HA: upgrade upstart and initiate replset

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 11 months ago by axw
Modified:
10 years, 11 months ago
Reviewers:
mp+216245, rog
Visibility:
Public.

Description

HA: upgrade upstart and initiate replset The machine agent will now upgrade the Mongo upstart script and initiate the replicaset when upgrading from a version prior to 1.19.0. Tested upgrading 1.18.1 to 1.19.1 with and without HA enabled; tested straight bootstrap to 1.19.1 with and without HA enabled. (Supersedes https://codereview.appspot.com/88350043/) https://code.launchpad.net/~axwalk/juju-core/lp1306902-ha-upgrade-take2/+merge/216245 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 4

Patch Set 2 : HA: upgrade upstart and initiate replset #

Patch Set 3 : HA: upgrade upstart and initiate replset #

Unified diffs Side-by-side diffs Delta from patch set Stats (+183 lines, -94 lines) Patch
A [revision details] View 1 2 1 chunk +2 lines, -0 lines 0 comments Download
M agent/format-1.18.go View 3 chunks +7 lines, -4 lines 0 comments Download
M agent/mongo/mongo.go View 1 2 1 chunk +3 lines, -6 lines 0 comments Download
M cmd/jujud/bootstrap.go View 1 2 2 chunks +4 lines, -4 lines 0 comments Download
M cmd/jujud/machine.go View 1 2 4 chunks +126 lines, -58 lines 0 comments Download
M cmd/jujud/machine_test.go View 1 2 10 chunks +18 lines, -8 lines 0 comments Download
M worker/peergrouper/desired_test.go View 1 2 1 chunk +0 lines, -6 lines 0 comments Download
M worker/peergrouper/initiate.go View 1 2 1 chunk +8 lines, -8 lines 0 comments Download
M worker/peergrouper/initiate_test.go View 1 chunk +1 line, -0 lines 0 comments Download
A worker/peergrouper/suite_test.go View 1 chunk +14 lines, -0 lines 0 comments Download

Messages

Total messages: 4
axw
Please take a look.
10 years, 11 months ago (2014-04-17 03:33:07 UTC) #1
rog
LGTM with two minor suggestions. https://codereview.appspot.com/88790043/diff/1/cmd/jujud/machine.go File cmd/jujud/machine.go (right): https://codereview.appspot.com/88790043/diff/1/cmd/jujud/machine.go#newcode438 cmd/jujud/machine.go:438: func (a *MachineAgent) ensureMongoServer(agentConfig ...
10 years, 11 months ago (2014-04-17 18:20:23 UTC) #2
axw
Please take a look. https://codereview.appspot.com/88790043/diff/1/cmd/jujud/machine.go File cmd/jujud/machine.go (right): https://codereview.appspot.com/88790043/diff/1/cmd/jujud/machine.go#newcode438 cmd/jujud/machine.go:438: func (a *MachineAgent) ensureMongoServer(agentConfig agent.Config) ...
10 years, 11 months ago (2014-04-18 04:26:23 UTC) #3
axw
10 years, 11 months ago (2014-04-18 04:44:47 UTC) #4
Please take a look.
Sign in to reply to this message.

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