ipmiextra: Allow explicit specification of the IPMI mgmt hostname

Mention openipmi as an optional dependency in the README.

diff --git a/Osstest/PDU/ipmiextra.pm b/Osstest/PDU/ipmiextra.pm
index d510e87414c017c1b9491c6612bfb066fd51191a..a10cd4d00a2a5f99ff953f411861c08fd2fac20e 100644 (file)
--- a/Osstest/PDU/ipmiextra.pm
+++ b/Osstest/PDU/ipmiextra.pm
@@ -35,9 +35,10 @@ BEGIN {
 }
 
 sub new {
-    my ($class, $ho, $methname, $when, $user, $pass, @opts) = @_;
+    my ($class, $ho, $methname, $when, $mgmt, $user, $pass, @opts) = @_;
     return bless { Host => $ho,
                   When => $when,
+                  Mgmt => $mgmt,
                   User => $user,
                   Pass => $pass,
                   Opts => \@opts }, $class;
@@ -48,7 +49,7 @@ sub pdu_power_state {
     my $onoff= $on ? "on" : "off";
 
     system_checked("ipmitool",
-                  "-H", "$mo->{Host}{Name}-mgmt",
+                  "-H", "$mo->{Mgmt}",
                   "-U", $mo->{User},
                   "-P", $mo->{Pass},
                   @{$mo->{Opts}})

diff --git a/README b/README
index e9aed52b017f0c282abbe98d8a3431942ebc38bb..821201a0d8597748c6687f09171c1be2aa72b96d 100644 (file)
--- a/README
+++ b/README
@@ -6,6 +6,8 @@ To run osstest in standalone mode:
      libdbi-perl libdbd-sqlite3-perl
      pax rsync
 
+ - Optional: ipmitool
+
  - Write a config file
     ~/.xen-osstest/config
    See below.