Bug 703841 - bkr workflow-simple --prettyxml should imply --debug
Summary: bkr workflow-simple --prettyxml should imply --debug
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: command line
Version: 0.6
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified vote
Target Milestone: ---
Assignee: Bill Peck
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-05-11 12:29 UTC by Michal Haško
Modified: 2019-05-22 13:38 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-06-03 05:31:05 UTC

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Michal Haško 2011-05-11 12:29:02 UTC 
Description of problem:
There is no output produced by 'bkr workflow-simple' when the --prettyxml is used without --debug. In other words, --prettyxml should imply --debug

Version-Release number of selected component (if applicable):
$ rpm -q beaker-client
beaker-client-0.6.8-5.git.47.219578f.el6.noarch

How reproducible:
100%

Steps to Reproduce:
$ bkr workflow-simple --family=RedHatEnterpriseLinux6 --task=/desktop/rhel6/spice-client/libs --dryrun --prettyxml
  
Actual results:
no output

Expected results:
should produce the same result as:
$ bkr workflow-simple --family=RedHatEnterpriseLinux6 --task=/desktop/rhel6/spice-client/libs --dryrun --prettyxml --debug
Comment 1 Marian Csontos 2011-05-11 12:56:15 UTC 
Not only in workflow-simple but basically everywhere where it does not produce XML output.

The help says:

  --prettyxml                                print the xml in pretty format
  --debug                                    print the jobxml that it would submit

It does not say --prettyxml requires --debug and IMO implicit requirement is better solution than introducing the requirement in help.
Comment 2 Bill Peck 2011-05-25 18:49:54 UTC 
Changes pushed to gerrit for review
Note You need to log in before you can comment on or make changes to this bug.