Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CCI DNS Sync doesn't accept flags/order shown in help #184

Closed
briancline opened this issue Sep 11, 2013 · 2 comments
Closed

CCI DNS Sync doesn't accept flags/order shown in help #184

briancline opened this issue Sep 11, 2013 · 2 comments
Labels

Comments

@briancline
Copy link
Member

The CLI shows the following help, but it does not match up with this usage text:

usage: sl cci dns sync <identifier> [options]

DNS related actions for a CCI

Options:
  -a, -A        Sync only the A record
  --ptr, --PTR  Sync only the PTR record

Prompt Options:
  -y, --really  Confirm all prompt actions

Standard Options:
  --format=ARG           Output format. [Options: table, raw] [Default: table]
  -C FILE --config=FILE  Config file location. [Default: ~/.softlayer]
  --debug=LEVEL          Specifies the debug noise level
                           1=warn, 2=info, 3=debug
  --timings              Time each API call and display after results
  -h --help              Show this screen

Upon running with the shown usage format, I always receive an "Unknown argument(s)" error:

$ sl cci dns sync 2695793 -a --ptr -y
usage: sl cci dns sync <identifier> [options]

Unknown argument(s), use -h or --help for available options

If I omit -a and --ptr entirely, I receive the same.

Moving -y before the identifier seems to work:

$ sl cci dns sync -y 2695793

If I attempt to use the -a and/or --ptr flags before the identifier where -y works, even independently of one another, I receive another unknown arguments error:

$ sl cci dns sync -a -y 2695795
usage: sl cci dns sync <identifier> [options]

Unknown argument(s), use -h or --help for available options
@sudorandom
Copy link
Contributor

They're actually not mutually exclusive.

It looks like docopt doesn't like mixed cases of the exact same name. (-a/-A and --ptr/--PTR). It's also pretty useless to support both. I'm making a commit that removes the uppercase versions.

@briancline
Copy link
Member Author

17-minute turnaround... thanks, lightning-fingers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants