juju: add NewConnFromState

juju: add NewConnFromState

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

https://code.launchpad.net/~rogpeppe/juju-core/229-NewConnFromState/+merge/150538

(do not edit description out of merge proposal)

[rog, 2013-02-26 11:55:38]

Please take a look.

[TheMue, 2013-02-26 12:06:04]

LGTM

[dimitern, 2013-02-26 12:16:02]

LGTM

[dave_cheney.net, 2013-02-27 05:45:47]

On 2013/02/26 12:16:02, dimitern wrote:
> LGTM

What do you plan to use this for ?

[rog, 2013-02-27 10:34:04]

On 27 February 2013 05:45,  <dave@cheney.net> wrote:
> On 2013/02/26 12:16:02, dimitern wrote:
>>
>> LGTM
>
>
> What do you plan to use this for ?
>
> https://codereview.appspot.com/7391051/

It's to be used by the API server - it has access to the state, but for
some operations (e.g. deploy, status) it wants a juju.Conn.

Do you see a problem with it?

[dave_cheney.net, 2013-03-02 03:56:43]

Nope. LGTM.

On Wed, Feb 27, 2013 at 9:34 PM, roger peppe <rogpeppe@gmail.com> wrote:
> On 27 February 2013 05:45,  <dave@cheney.net> wrote:
>> On 2013/02/26 12:16:02, dimitern wrote:
>>>
>>> LGTM
>>
>>
>> What do you plan to use this for ?
>>
>> https://codereview.appspot.com/7391051/
>
> It's to be used by the API server - it has access to the state, but for
> some operations (e.g. deploy, status) it wants a juju.Conn.
>
> Do you see a problem with it?

[rog, 2013-03-04 16:11:13]

*** Submitted:

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