launchpad-reviewers team mailing list archive

[Peter Petrakis <peter.petrakis@xxxxxxxxxxxxx>]

Peter Petrakis has proposed merging lp:~vanhoof/maas/ipmi-state-fix_lp1086160 into lp:maas.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #1086160 in MAAS: "IPMI Power command does not work"
  https://bugs.launchpad.net/maas/+bug/1086160

For more details, see:
https://code.launchpad.net/~vanhoof/maas/ipmi-state-fix_lp1086160/+merge/159714

Make ipmipower work.
-- 
https://code.launchpad.net/~vanhoof/maas/ipmi-state-fix_lp1086160/+merge/159714
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~vanhoof/maas/ipmi-state-fix_lp1086160 into lp:maas.

=== modified file 'src/provisioningserver/power/templates/ipmi.template'
--- src/provisioningserver/power/templates/ipmi.template	2013-01-14 22:06:12 +0000
+++ src/provisioningserver/power/templates/ipmi.template	2013-04-18 20:05:37 +0000
@@ -57,7 +57,7 @@
 
 # Get the given system's power state: 'on' or 'off'.
 get_power_state() {
-    ipmi_state=$(issue_ipmi_command stat)
+    ipmi_state=$(issue_ipmi_command --stat)
     formulate_power_state ${ipmi_state}
 }