Bug 1242388 - [aaa-jdbc-tool] Fix argument parsing of jdbc-tool
Summary: [aaa-jdbc-tool] Fix argument parsing of jdbc-tool
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-extension-aaa-jdbc
Version: 3.6
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.6.0
Assignee: Martin Perina
QA Contact: Pavel Stehlik
URL:
Whiteboard: infra
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-07-13 09:08 UTC by Ondra Machacek
Modified: 2016-02-10 19:33 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-07-23 09:15:31 UTC
oVirt Team: Infra
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1261131 0 unspecified CLOSED [aaa] Parameters name and value should have required argument 2021-02-22 00:41:40 UTC

Internal Links: 1261131

Description Ondra Machacek 2015-07-13 09:08:12 UTC 
Description of problem:


Version-Release number of selected component (if applicable):
3.6.0-3

How reproducible:
always

Steps to Reproduce:
1. run:
$ bin/ovirt-aaa-jdbc-tool --db-config=etc/ovirt-engine/extensions.d/profile.properties --log-level=INFO settings set --name MAX_FAILURES_PER_INTERVAL --value 3


Actual results:
Argument(s) 'value' required

Expected results:
Args are parsed successfully and command is ran successfully.

Additional info:
This issue is solved in latest ArgumentParser in master in core, please sync with it.
Comment 1 Martin Perina 2015-07-23 09:15:31 UTC 
This is not a bug, value for long parameters (for example "--name") should be separated by equals character (for example "--name=admin"), value for short parameters (for example -d) should be separated by space character (for example "-d 3"). For details please take a look at http://www.gnu.org/software/libc/manual/html_node/Argument-Syntax.html#Argument-Syntax
Comment 2 Ondra Machacek 2015-09-08 16:41:31 UTC 
Since the argument parser of this tool support both '=' and whitespace, would be nice to fix, anyway the problem was fixed by bug 1261131
Note You need to log in before you can comment on or make changes to this bug.