Descriptioncmd/juju: status using the API
Add status cmd to api with state access fallback
Current progress on having `juju status` use the api. Moves the
existing code to a resurrected state/statecmd file. This is
exposed both through the api, and as a fallback to accessing
state directly.
The status tests now pass, but moving some testing down a layer
or two would probably be a good plan.
Is there anything else required for a status first pass?
dimitern: I pulled Martin's branch lp:~gz/juju-core/status_api,
merged trunk and resolved conflicts, also fixed a couple of
tests, but no other changes are made. All tests pass, and
I also did a successful live test on EC2. Original CL is
at: https://codereview.appspot.com/40350043/, and has already
2 LGTMs.
https://code.launchpad.net/~dimitern/juju-core/gz-status_api/+merge/199814
(do not edit description out of merge proposal)
Patch Set 1 #
Total comments: 2
Patch Set 2 : cmd/juju: status using the API #
MessagesTotal messages: 3
|