Merge lp:~rogpeppe/juju-core/050-juju-provisioning-service into lp:~juju/juju-core/trunk

Proposed by Roger Peppe
Status: Work in progress
Proposed branch: lp:~rogpeppe/juju-core/050-juju-provisioning-service
Merge into: lp:~juju/juju-core/trunk
Prerequisite: lp:~rogpeppe/juju-core/051-authorize-internal-traffic
To merge this branch: bzr merge lp:~rogpeppe/juju-core/050-juju-provisioning-service
Reviewer Review Type Date Requested Status
The Go Language Gophers Pending
Review via email: mp+123502@code.launchpad.net

Description of the change

WIP: add "agent" services and run the provisioner as one.

This is extremely controversial, but seems to me like a very
cheap and arguably elegant way to provide a state
entity for any agent that requires one (for instance the PA).

Doesn't currently work because container is broken.

https://codereview.appspot.com/6493101/

To post a comment you must log in.

Unmerged revisions

465. By Roger Peppe

container: fix test

464. By Roger Peppe

cmd/jujud: remove agentName argument to NewUpgrader

463. By Roger Peppe

merge 051-authorize-internal-traffic

462. By Roger Peppe

state: add AddAgentService; cmd/jujud: add provisioning agent setup

461. By Roger Peppe

state: add IsAgent etc

460. By Roger Peppe

cmd/jujud: make provisioner use unit

459. By Roger Peppe

container: test for juju_* service

458. By Roger Peppe

container: recognise juju- service

457. By Gustavo Niemeyer

mstate/presence: foundation for presence handling

R=rog, TheMue
CC=
https://codereview.appspot.com/6495079

456. By Roger Peppe

juju: always connect to State.

All uses of juju.Conn other than Bootstrap and Destroy require a state
connection, so make State an exported field of Conn and connect to it
when creating the Conn.

The few places that call Bootstrap can easily call Environ.Bootstrap
instead.

Also add environs.NewFromName to make it easier to open an
environment, rename juju.NewConn to juju.NewConnFromName, and add
juju.NewConnFromEnviron to join the dots.

R=TheMue, niemeyer
CC=
https://codereview.appspot.com/6488077

Subscribers

People subscribed via source and target branches