Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(47)

Issue 6430045: cmd/juju: status: add format and output support (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 8 months ago by dave
Modified:
12 years, 8 months ago
Reviewers:
mp+115630
Visibility:
Public.

Description

cmd/juju: status: add format and output support % juju status --help usage: juju status [options] purpose: Output status information about a deployment. options: --debug (= false) if set, log debugging messages -e, --environment (= "") juju environment to operate in --format (= yaml) specify output format (json|yaml) --log-file (= "") path to write log to -o, --output (= "") specify an output file -v, --verbose (= false) if set, log additional messages https://code.launchpad.net/~dave-cheney/juju-core/go-cmd-juju-status-output-again/+merge/115630 Requires: https://code.launchpad.net/~dave-cheney/juju-core/go-cmd-juju-status/+merge/115460 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : cmd/juju: status: add format and output support #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M cmd/juju/status.go View 1 2 chunks +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3
dave_cheney.net
Please take a look.
12 years, 8 months ago (2012-07-18 22:08:21 UTC) #1
niemeyer
LGTM. Well done on the move of formatter handling.
12 years, 8 months ago (2012-07-18 22:20:58 UTC) #2
dave_cheney.net
12 years, 8 months ago (2012-07-18 22:34:21 UTC) #3
*** Submitted:

cmd/juju: status: add format and output support

% juju status --help
usage: juju status [options]
purpose: Output status information about a deployment.

options:
--debug  (= false)
    if set, log debugging messages
-e, --environment (= "")
    juju environment to operate in
--format  (= yaml)
    specify output format (json|yaml)
--log-file (= "")
    path to write log to
-o, --output (= "")
    specify an output file
-v, --verbose  (= false)
    if set, log additional messages

R=niemeyer
CC=
https://codereview.appspot.com/6430045
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b