lp:~dave-cheney/juju-core/go-service-machiner

Branch merges

Propose for merging

No branches dependent on this one.

Rejected for merging into lp:~juju/juju-core/trunk

The Go Language Gophers: Pending requested 2012-07-03

Diff: 609 lines (+304/-258)

6 files modified

cmd/jujud/machine.go (+2/-7)
cmd/jujud/machiner.go (+0/-79)
cmd/jujud/machiner_test.go (+0/-172)
service/machiner/machiner.go (+80/-0)
service/machiner/machiner_test.go (+174/-0)
service/machiner/suite_test.go (+48/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

283. By Dave Cheney on 2012-07-03

move machiner to its own package

282. By Dave Cheney on 2012-07-03

move machiner to a service package

281. By Dave Cheney on 2012-07-03

merge from prereq

280. By Dave Cheney on 2012-07-03

service/provisioner: new package

This proposal moves the Provisioner to a new package so it can
be imported by commands (jujud etc) and other packages
(environs/jujutest).

note:

* MachinerAgent/Machiner will be handled in a following branch.

* suite_test.go is duplicated for the moment, once Williams test
refactor is completed, this duplication will be removed.

R=niemeyer
CC=
https://codereview.appspot.com/6344071

279. By Dave Cheney on 2012-07-03

cmd/jujud: bifurcate Provisioniner and Provisioner

This proposal prepares for the move of the Provisioner into
juju-core/services/provisioner by moving it into its own file.

R=niemeyer
CC=
https://codereview.appspot.com/6350064

278. By Dave Cheney on 2012-07-03

cmd/juju: enable deploy command

William tells me this was an inadvertent oversight.

R=niemeyer
CC=
https://codereview.appspot.com/6350049

277. By Dave Cheney on 2012-06-30

environs/ec2: return matching ami for the region

Using defaultInstanceConstraints always specified us-east-1 when
searching for the instance ami. This branch respects the region
setting from the configuration.

R=niemeyer
CC=
https://codereview.appspot.com/6362043

276. By Frank Mueller on 2012-06-29

watcher: renamed the fields Deleted to Removed

Several watcher return a struct containing two fields
for added and removed subnodes or other informations
inside a node content. The name of the field for
removed informations has been inconsistent. Now all
are changed into the identifier "Removed".

R=niemeyer
CC=
https://codereview.appspot.com/6351047

275. By Frank Mueller on 2012-06-28

state: added watcher for service units

The added watcher returns informations about the
assigned or unassigned units of a service.

R=fwereade, niemeyer
CC=
https://codereview.appspot.com/6325062

274. By Aram Hăvărneanu on 2012-06-28

mstate: basic service units

R=niemeyer
CC=
https://codereview.appspot.com/6346044