Merge lp:~andreserl/maas/maas_ipmi_lan2.0 into lp:maas/trunk

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: 1416
Merged at revision: 1416
Proposed branch: lp:~andreserl/maas/maas_ipmi_lan2.0
Merge into: lp:maas/trunk
Diff against target: 13 lines (+2/-2)
1 file modified
src/provisioningserver/power/templates/ipmi.template (+2/-2)
To merge this branch: bzr merge lp:~andreserl/maas/maas_ipmi_lan2.0
Reviewer Review Type Date Requested Status
Raphaël Badin (community) Approve
Review via email:

Commit message

Enforce IPMI 2.0 authentication (LP: #1086162)

To post a comment you must log in.
Revision history for this message
Raphaël Badin (rvb) wrote :

LGTM. I'd prefer the more verbose version of the param (i.e. --driver-type=LAN_2_0) but using -D is ok.

review: Approve
lp:~andreserl/maas/maas_ipmi_lan2.0 updated
1416. By Andres Rodriguez

Use verbose version of argument

Revision history for this message
Raphaël Badin (rvb) wrote :

I think this change broke all our automated tests in the lab:

In the lab, when I run:
ipmipower --driver-type=LAN_2_0 -h ip -u pw -p user --stat

I get: "ip: BMC error"

Should we change the IMPI settings to add a boolean parameter (LAN_v2) instead of making it the default?

Revision history for this message
Raphaël Badin (rvb) wrote :

Instead of a boolean, we could have a drop down choice with all the available driver types:
Excerpt from man ipmipower:
       -D IPMIDRIVER, --driver-type=IPMIDRIVER
              Specify the driver type to use instead of doing an auto selection. The currently available outofband drivers are LAN and LAN_2_0, which perform IPMI 1.5 and IPMI 2.0 respectively.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/provisioningserver/power/templates/ipmi.template'
2--- src/provisioningserver/power/templates/ipmi.template 2012-10-06 16:23:49 +0000
3+++ src/provisioningserver/power/templates/ipmi.template 2012-12-20 16:15:26 +0000
4@@ -42,9 +42,9 @@
5 issue_ipmi_command() {
6 # See for details of this workaround
7 echo workaround |\
8- ${ipmi_chassis_config} -h ${power_address} -u ${power_user} -p ${power_pass} --commit --filename ${config}
9+ ${ipmi_chassis_config} --driver-type=LAN_2_0 -h ${power_address} -u ${power_user} -p ${power_pass} --commit --filename ${config}
10 echo workaround |\
11- ${ipmipower} -h ${power_address} -u ${power_user} -p ${power_pass} "$@"
12+ ${ipmipower} --driver-type=LAN_2_0 -h ${power_address} -u ${power_user} -p ${power_pass} "$@"
13 }