Bug 498551 - The short form for fcoeadm switch display output properly while long displays help screen.
Summary: The short form for fcoeadm switch display output properly while long displays...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: fcoe-utils
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Jan Zeleny
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-05-01 01:48 UTC by Aaron Brown
Modified: 2009-05-14 16:22 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
: 503167 (view as bug list)
Environment:
Last Closed: 2009-05-14 16:22:13 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Patch fixes handling of long options given to fcoeadm (3.58 KB, patch)
2009-05-06 09:22 UTC, Jan Zeleny
no flags Details | Diff

Description Aaron Brown 2009-05-01 01:48:51 UTC
Description of problem:
when using the short form (fcoeadm -i) of fcoeadm switches, fcoeadm works as expected but when using the long form (fcoeadm --interface) they don't. The long form just displays the help menu instead of proper output.  The displayed help screen does show the long form as a valid command.

Version-Release number of selected component (if applicable):
fcoeadm v1.0.7

How reproducible:
enter any fcoeadm command in both short form and long form, observe differences.
  
Actual results:
Help is displayed for the long form rather then output expected for the flag.

Expected results:
The output should be the same for both short and long forms of the fcoeadm options.

Additional info:

Comment 1 Aaron Brown 2009-05-01 02:01:55 UTC
The fcoeadm --help, --stats and --version commands do function as expected, the rest give nothing but the help screen.

Comment 2 Jan Zeleny 2009-05-05 15:25:09 UTC
I'm working on patch. Could you please tell me, if you are experiencing problems with --lun parameter? It looks fine to me.

Comment 3 Aaron Brown 2009-05-05 21:57:00 UTC
Yes, the "fcoeadm --lun ..." form of the command does appear to work exactly as teh "fcoeadm -l ..." form.  I did see the help screen rather then LUN info when I was trying this earlier, but after examining my history buffer I realize I was using improper syntax.

Comment 4 Jan Zeleny 2009-05-06 09:22:14 UTC
Created attachment 342624 [details]
Patch fixes handling of long options given to fcoeadm

I've made attached patch. Before I commit it to rawhide, I want open-fcoe guys to take look at it and accept it to upstream version.

Comment 5 Robert Love 2009-05-06 20:38:15 UTC
Jan, could you please mail this patch to devel so the list can review it? That would also allow us to get a propper commit message and you can have your Signed-off line in the commit.

FYI: I took a quick glance at the change and it looks good. I have not tested it though.

Comment 6 Jan Zeleny 2009-05-14 16:22:13 UTC
For now, I've applied the patch, it should work fine. If there are any issues with it, please open new bugzilla. In the future I expect complete rework of option handling. Rebase of the package will probably take place after that.


Note You need to log in before you can comment on or make changes to this bug.