lp:~dave-cheney/juju-core/098-fix-uniter-hook-failure-raring

Branch merges

Propose for merging

No branches dependent on this one.

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

The Go Language Gophers: Pending requested 2013-03-04

Diff: 17 lines (+2/-2)

1 file modified

worker/uniter/uniter_test.go (+2/-2)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1145015: worker/uniter: UniterSuite.TestUniterConfigChangedHook fails

High

Expired

Link a bug report

Related blueprints

956. By Dave Cheney on 2013-03-04

make tests less sensitive to git output specifics

955. By Dave Cheney on 2013-03-04

environs/ec2: try to get tests working on raring

Add fixtures for raring

R=jameinel, thumper
CC=
https://codereview.appspot.com/7380047

954. By Roger Peppe on 2013-03-04

juju: add NewConnFromState

We want to perform this operation to make
a Conn available to the API server.

R=TheMue, dimitern, dfc
CC=
https://codereview.appspot.com/7391051

953. By Dimiter Naydenov on 2013-03-01

state, uniter: Units now use charm URLs

Simplification of the state unit's interface
that will be needed in a follow-up CL on
the uniter.

Resubmit of the same branch already reviewed,
due to issues with lbox/cobzr. See:

R=
CC=
https://codereview.appspot.com/7450046

952. By Francesco Banconi on 2013-03-01

Implement the API unexpose command.

Moved the unexpose command logic to the
statecmd package. Added tests, verified
also running manual tests using a websocket
connection.

R=dimitern, rog
CC=
https://codereview.appspot.com/7421045

951. By William Reade on 2013-03-01

environs: drop InstanceIdAccessor hack

...and also ProviderType, which is already accessible in the environ config.

R=rog, dimitern
CC=
https://codereview.appspot.com/7375059

950. By Brad Crittenden on 2013-02-28

Add the 'expose' command to the API.

R=benji, dimitern, rog
CC=
https://codereview.appspot.com/7369058

949. By John A Meinel on 2013-02-28

revert only r943

William merged a change which reverted trunk back to 942
because rev 943 broke the test suite.

This just re-introduces all of the merges 944-947, so that
Gustavo and Ian don't have to re-propose and get approval
for their changes again.

I've run the test suite to make sure it passes as long as
just 943 has been removed.

In the future, you can just cherrypick-out specific revisions
with (eg):
 bzr merge . -r 943..942

And then run the test suite, and propose it. (Which is essentially
what this patch does, except it also reverts 948 which removed
all of those other revisions.)

R=wallyworld, fwereade
CC=
https://codereview.appspot.com/7424046

948. By William Reade on 2013-02-28

history: rewind

tests started failing in r943; reverting to r942.

R=
CC=
https://codereview.appspot.com/7422044

947. By Ian Booth on 2013-02-28

Better unauthorised errors

The state package defined a ErrUnauthorized variable to hold an error with a fixed
error string. Unfortunately, this results in important information being discarded
when there is an error. There are at least 3 possible causes of an unauth error and
the final error message should include this info. I've reworked the implementation
in a similar way to how state package's notFoundError is implemented, allowing the
root cause message to be retained with the error.

R=dfc, dimitern
CC=
https://codereview.appspot.com/7411045