lp:~rogpeppe/juju-core/531-SetStateInstances

Created by Roger Peppe and last modified
Get this branch:
bzr branch lp:~rogpeppe/juju-core/531-SetStateInstances
Only Roger Peppe can upload to this branch. If you are Roger Peppe please log in for upload directions.

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
Roger Peppe
Project:
juju-core
Status:
Development

Recent revisions

2480. By Roger Peppe

wip

2479. By Roger Peppe

wip

2478. By Roger Peppe

wip

2477. By Roger Peppe

add SetStateInstances method and function

2476. By Roger Peppe

[r=rogpeppe] worker: factor out NewEnvironObserver

As intimated in the TODO, we want to use this in other
workers. The actual code remains the same.

https://codereview.appspot.com/76890049/

2475. By Andrew Wilkins

[r=axwalk],[bug=1295058] Update ComposeUserData to take cloudinit.Config

... and update provider/maas to use the apt
options instead of crafting apt-get commands.

Fixes lp:1295058

https://codereview.appspot.com/78640044/

2474. By Roger Peppe

[r=rogpeppe] state/api: further api client consolidation

This carries on the work started by https://codereview.appspot.com/78890043

https://codereview.appspot.com/79470043/

2473. By Dimiter Naydenov

[r=dimitern] state: Add Machine.Networks() and create them

This adds a new method to state.Machine: Networks(),
which returns any associated networks to include or
exclude for the machine.

Added a couple of extra fields to state.MachineTemplate:
IncludeNetworks and ExcludeNetworks. Each time a new
machine or container is created, if these are populated,
they will be also saved in state for the machine, as a
linked networksDoc.

A few drive-by fixes for naming consistency, including
changing Included/ExcludedNetworks across the codebase
to Include/ExcludeNetworks to better illustrate the
intent (i.e. we're saying what networks should be
included or excluded, not how the reality is yet - we
can do that post-provisioning only).

Added tests to ensure when the new networkDocs are
missing for a service/machine .Networks() still works
and returns nil error, as suggested in reviews.

https://codereview.appspot.com/79250044/

R=fwereade, gz

2472. By Tim Penhey

[r=thumper],[bug=1292275] Local provider destroy removes upstart scripts forcibly.

During the force destroy part, forcibly remove the
mongo and machine agent upstart scripts.

Tests are added to show that these are removed.
A drive by test was added to show that the containers
are destroyed too.

https://codereview.appspot.com/79690043/

2471. By Andrew Wilkins

[r=axwalk],[bug=1296739] provider/local: use "env" to propagate JUJU_HOME

Sudo may not allow environment variables to be
set directly, so we must use "env" to set JUJU_HOME
when re-executing juju as root.

Fixes lp:1296739

https://codereview.appspot.com/79620043/

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
Stacked on:
lp:~go-bot/juju-core/trunk
This branch contains Public information 
Everyone can see this information.

Subscribers