ipmitool driver doesn't support IPMI 1.5

Bug #1468378 reported by Lucas Alvares Gomes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
High
Lucas Alvares Gomes

Bug Description

I got access to 2 machines where commands with IPMI 2.0 (-I lanplus) doesn't work, e.g

$ ipmitool -I lanplus -H X.X.X.X -L ADMINISTRATOR -U *** -R 12 -N 5 -P *** power status
ipmitool: lanplus.c:2177: ipmi_lanplus_send_payload: Assertion `session->v2_data.session_state == LANPLUS_STATE_PRESESSION' failed.
Aborted (core dumped)

But IPMI 1.5 (-I lan) works fine:

$ ipmitool -I lan -H X.X.X.X -L ADMINISTRATOR -U *** -R 12 -N 5 -P *** power status
Chassis Power is on

The Ironic node for that machine couldn't sync the power state and enters maintenance mode after a while

$ ironic node-show 06f3c639-8d56-4f9d-b8dd-f5dc9accf8de
...

| target_power_state | None |
| last_error | During sync_power_state, max retries exceeded for node 06f3c639-8d56 |
| | -4f9d-b8dd-f5dc9accf8de, node state None does not match expected state |
| | 'None'. Updating DB state to 'None' Switching node to maintenance mode. |
| maintenance_reason | During sync_power_state, max retries exceeded for node 06f3c639-8d56 |
| | -4f9d-b8dd-f5dc9accf8de, node state None does not match expected state |
| | 'None'. Updating DB state to 'None' Switching node to maintenance mode. |
| target_provision_state | None |
| maintenance | True |
| power_state | None |
| driver | pxe_ipmitool |

I couldn't find any workaround for the problem apart from changing the code ipmitool driver code replacing "-I lan" instead of "-I lanplus".

My suggestion is to offer a configuration option per node (node's driver_info) to tell the ipmitool driver to use IPMI 1.5 instead of IPMI 2.0.

I don't foresee the driver automatically falling back to 1.5 if the command fails because ipmitool doesn't return a meaningful message for such failure

Tags: ipmi
Changed in ironic:
assignee: nobody → Lucas Alvares Gomes (lucasagomes)
importance: Undecided → Medium
description: updated
description: updated
description: updated
Revision history for this message
Dmitry Tantsur (divius) wrote :

Importance high, as it breaks deployment, and there's no work around.

Changed in ironic:
status: New → Triaged
tags: added: ipmi
Changed in ironic:
importance: Medium → High
Changed in ironic:
status: Triaged → In Progress
Changed in ironic:
status: In Progress → Fix Committed
Changed in ironic:
milestone: none → 4.0.0
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.