Bug 1583768

Summary: lsmcli volume-write-cache-policy-update --help typo
Product: Red Hat Enterprise Linux 7 Reporter: Jakub Krysl <jkrysl>
Component: libstoragemgmtAssignee: Tony Asleson <tasleson>
Status: CLOSED ERRATA QA Contact: Jakub Krysl <jkrysl>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 7.6CC: tasleson
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-08-06 12:40:24 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Jakub Krysl 2018-05-29 15:52:12 UTC
Description of problem:
There is a typo in help for $SUBJECT command, missing at least space between 'WT' and 'which':
# lsmcli volume-write-cache-policy-update --help
usage: lsmcli volume-write-cache-policy-update [-h] [-v] [-u <URI>] [-P] [-H]
                                               [-t <SEP>] [-e] [-f]
                                               [-w CHILD_WAIT] [--header] [-b]
                                               [-s] --vol <VOL_ID> --policy
                                               <POLICY>

optional arguments:
  -h, --help            show this help message and exit
  -v, --version         show program's version number and exit
  -u <URI>, --uri <URI>
                        Uniform resource identifier (env LSMCLI_URI)
  -P, --prompt          Prompt for password (env LSMCLI_PASSWORD)
  -H, --human           Print sizes in human readable format
                        (e.g., MiB, GiB, TiB)
  -t <SEP>, --terse <SEP>
                        Print output in terse form with "SEP" as a record separator
  -e, --enum            Display enumerated types as numbers instead of text
  -f, --force           Bypass confirmation prompt for data loss operations
  -w CHILD_WAIT, --wait CHILD_WAIT
                        Command timeout value in ms (default = 30s)
  --header              Include the header with terse
  -b                    Run the command async. Instead of waiting for completion.
                         Command will exit(7) and job id written to stdout.
  -s, --script          Displaying data in script friendly way with additional information(if exists)

cmd required arguments:
  --vol <VOL_ID>        Volume ID
  --policy <POLICY>     Write cache policys: WB, AUTO, WTwhich stand for "write back", "auto", "write through"


Version-Release number of selected component (if applicable):
libstoragemgmt-1.6.2-1

How reproducible:
100%

Steps to Reproduce:
1. lsmcli volume-write-cache-policy-update --help

Actual results:
  --policy <POLICY>     Write cache policys: WB, AUTO, WTwhich stand for "write back", "auto", "write through"

Expected results:
  --policy <POLICY>     Write cache policys: WB, AUTO, WT which stand for "write back", "auto", "write through"

Additional info:

Comment 4 Jakub Krysl 2019-03-28 11:30:09 UTC
libstoragemgmt-smis-plugin-1.7.3-2.el7.noarch
libstoragemgmt-1.7.3-2.el7.x86_64

# lsmcli volume-write-cache-policy-update --help
usage: lsmcli volume-write-cache-policy-update [-h] [-v] [-u <URI>] [-P] [-H]
                                               [-t <SEP>] [-e] [-f]
                                               [-w CHILD_WAIT] [--header] [-b]
                                               [-s] --vol <VOL_ID> --policy
                                               <POLICY>

optional arguments:
  -h, --help            show this help message and exit
  -v, --version         show program's version number and exit
  -u <URI>, --uri <URI>
                        Uniform resource identifier (env LSMCLI_URI)
  -P, --prompt          Prompt for password (env LSMCLI_PASSWORD)
  -H, --human           Print sizes in human readable format
                        (e.g., MiB, GiB, TiB)
  -t <SEP>, --terse <SEP>
                        Print output in terse form with "SEP" as a record separator
  -e, --enum            Display enumerated types as numbers instead of text
  -f, --force           Bypass confirmation prompt for data loss operations
  -w CHILD_WAIT, --wait CHILD_WAIT
                        Command timeout value in ms (default = 30s)
  --header              Include the header with terse
  -b                    Run the command async. Instead of waiting for completion.
                         Command will exit(7) and job id written to stdout.
  -s, --script          Displaying data in script friendly way with additional information(if exists)

cmd required arguments:
  --vol <VOL_ID>        Volume ID
  --policy <POLICY>     Write cache polices: WB, AUTO, WT which stand for "write back", "auto", "write through"

Comment 6 errata-xmlrpc 2019-08-06 12:40:24 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2019:2064