Menu

#113 delloem exec file won't handle more than one command

version-1.8.13
closed-fixed
None
5
2013-07-09
2012-09-25
No

ipmitool version: 1.8.12
operating system reproduced on: OpenEmbedded Linux

How to reproduce:
Create an exec file with the lines:
delloem powermonitor powerconsumption watt
delloem powermonitor powerconsumptionhistory watt

Execute the same two commands from the ipmitool command line.
Execute the exec file you created.

Note that the exec file always results in an error for the second command:
"Error getting power headroom status: Command not supported on this system"

Patches to ipmi_delloem.c and ipmi_shell.c seem to fix the issue and example patches are included as an attachment.

Discussion

  • Richard Detweiler

    example patches that seem to fix the problem.

     
  • Zdenek Styblik

    Zdenek Styblik - 2013-03-27

    Problem is in 'lib/ipmi_delloem.c', so there is no reason to pach anything else even though it fixes the issue. So NACK to the attached patch.
    I'm afraid fix will be, well, complicated.

     
  • Zdenek Styblik

    Zdenek Styblik - 2013-03-27
    • milestone: --> version-1.8.12
     
  • Zdenek Styblik

    Zdenek Styblik - 2013-07-08

    Actually, fix is quite easy, but it was pain to figure it out. Patch attached.

     
  • Zdenek Styblik

    Zdenek Styblik - 2013-07-09
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,4 +1,3 @@
    -
     ipmitool version: 1.8.12
     operating system reproduced on: OpenEmbedded Linux
    
    • status: open --> closed-fixed
    • assigned_to: Zdenek Styblik
     
  • Zdenek Styblik

    Zdenek Styblik - 2013-07-09

    Fix committed to CVS.

     

Log in to post a comment.

MongoDB Logo MongoDB