Merge lp://staging/~dave-cheney/pyjuju/go-state-add-environment into lp://staging/pyjuju/go

Proposed by Dave Cheney
Status: Merged
Approved by: Gustavo Niemeyer
Approved revision: 191
Merged at revision: 199
Proposed branch: lp://staging/~dave-cheney/pyjuju/go-state-add-environment
Merge into: lp://staging/pyjuju/go
Diff against target: 15 lines (+5/-0)
1 file modified
state/state.go (+5/-0)
To merge this branch: bzr merge lp://staging/~dave-cheney/pyjuju/go-state-add-environment
Reviewer Review Type Date Requested Status
Juju Engineering Pending
Review via email: mp+108109@code.staging.launchpad.net

Description of the change

state: add state.Environment

This func provides direct access to the current environment
configuration as a *ConfigNode. This is used buy jujut and
is needed for testing the provisioning agent as
state.readConfigNode() is not exported.

https://codereview.appspot.com/6261055/

To post a comment you must log in.
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

LGTM, but can you please add a trivial test before submitting?

https://codereview.appspot.com/6261055/

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: