lp://staging/~axwalk/juju-core/null-provider-destroy-environment

Branch merges

Propose for merging

No branches dependent on this one.

On hold for merging into lp://staging/~go-bot/juju-core/trunk

Juju Engineering: Pending requested 2013-11-01

Diff: 179 lines (+124/-4)

3 files modified

provider/common/destroy.go (+111/-0)
provider/null/environ.go (+13/-0)
provider/null/environ_test.go (+0/-4)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

2010. By Andrew Wilkins on 2013-11-01

provider/null: first half of Environ.Destroy()

This is CL 1 of 2: destroy-environment will work
by first destroying all units, and non-environment
manager machines; then, the environment-manager
machine(s) will be destroyed by sshing to them
and killing jujud. jujud will catch the signal,
and clean itself up.

The next CL will implement the manager machine
killing bit.

2009. By John A Meinel on 2013-10-31

[r=jameinel] testing/checkers/log: Clearer failures

When Andrew changed the LogMatches function to only assert the
existence of messages he accidentally removed the work I had done to
give clearer messages when there was a problem.

This changes some types around and updates the params values like we
did before. So that instead of getting:

log_test.go:47:
    c.Check(log, jc.LogMatches, []string{"foo bar", "a2345"})
... obtained []loggo.TestLogValues = []loggo.TestLogValues{loggo.TestLogValues{Level:0x3, Module:"", Filename:"", Line:0, Timestamp:time.Time{sec:0, nsec:0, loc:(*time.Location)(nil)}, Message:"foo bar"}, loggo.TestLogValues{Level:0x3, Module:"", Filename:"", Line:0, Timestamp:time.Time{sec:0, nsec:0, loc:(*time.Location)(nil)}, Message:"12345"}}
... expected []string = []string{"foo bar", "a2345"}

We now get:

log_test.go:47:
    c.Check(log, jc.LogMatches, []string{"foo bar", "a2345"})
... obtained checkers.SimpleMessages = SimpleMessages{
INFO 12345
}
... expected checkers.SimpleMessages = SimpleMessages{
UNSPECIFIED a2345
}

https://codereview.appspot.com/19070043/

2008. By Ian Booth on 2013-10-30

[r=wallyworld] Add option to generate simplestreams mirror info

When generating tools metadata using sync-tools or the
tools plugin, add option is provided to optionally generate
mirrors metadata. The option is --public because it is used
when generating tools metadata for public clouds.

https://codereview.appspot.com/16550046/

2007. By Ian Booth on 2013-10-26

[r=wallyworld] Remove environs PublicStorage()

With the simplestreams tools infrastructure now in
place, there is no longer any need for environment
provided public storage. To ensure a clean environment
config for 2.0, the relevant config attributes have been
removed rather than deprecated. No one much uses them anyway
eg public-bucket.

https://codereview.appspot.com/16400043/

2006. By Ian Booth on 2013-10-26

[r=wallyworld] Get tools metadata from streams.canonical.com

This branch removes all of the hard coded cloud specific
tools metadata urls and instead uses streams.canonical.com
as the source for tools metadata. The deprecated public
bucket url config item is also removed.

https://codereview.appspot.com/15690052/

2005. By Ian Booth on 2013-10-25

[r=wallyworld] Use separate mirrors file

simplestreams now requires the mirrors index information
be in a separate mirrors file rather than the index file.

https://codereview.appspot.com/16670043/

2004. By Martin Packman on 2013-10-25

[r=gz],[bug=1230373] environs/simplestreams: Log update series errors

Rather than returning an error from updateSeriesVersions
and then ignoring it, log any issues from reading the
ubuntu.csv file.

https://codereview.appspot.com/16930043/

R=wallyworld

2003. By Ian Booth on 2013-10-24

[r=wallyworld] add robust simplestreams cloud matching

Allow auth urls to have trailing "/" (or not) and
simplestreams will still work as expected.

https://codereview.appspot.com/16600043/

2002. By Martin Packman on 2013-10-24

[r=gz] cloudinit: Use cloud archive key not short key id

Currently, when adding the cloud archive to apt_sources for
precise, we supply the short key id to obtain the signing
key. However, this requires access to the keyserver, which
not all deployments will have. By supplying the full text
of the public key instead, we can verify the download
without needing additional access.

There is a package in precise containing this key, alas we
can't just add that as the apt_sources step runs first,
and update with the cloud archive will fail unless the
key is approved before installing packages.

https://codereview.appspot.com/15060051/

R=axwalk, smoser

2001. By Marco Ceppi on 2013-10-24

[r=jameinel] names/relation.go: allow underscore in relation names

Let relation names use underscore as a separator like they use hyphen.