RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1286045 - fence_ipmilan missing diag as reboot method
Summary: fence_ipmilan missing diag as reboot method
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: fence-agents
Version: 7.1
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: rc
: ---
Assignee: Marek Grac
QA Contact: cluster-qe@redhat.com
URL:
Whiteboard:
: 1266599 (view as bug list)
Depends On:
Blocks: 1203710
TreeView+ depends on / blocked
 
Reported: 2015-11-27 10:14 UTC by Dominique Martinet
Modified: 2020-12-11 11:59 UTC (History)
6 users (show)

Fixed In Version: fence-agents-4.0.11-34.el7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-11-04 04:48:26 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
suggested patch (2.67 KB, patch)
2015-11-27 10:14 UTC, Dominique Martinet 		no flags Details | Diff
Upstream patch updated for RHEL7 (5.46 KB, patch)
2016-03-14 09:50 UTC, Oyvind Albrigtsen 		no flags Details | Diff
Show Obsolete (1) View All


Links
System ID Private Priority Status Summary Last Updated
Red Hat Knowledge Base (Solution) 1480813 0 None None None 2016-11-09 15:19:27 UTC
Red Hat Knowledge Base (Solution) 2317631 0 None None None 2016-11-09 15:18:19 UTC
Red Hat Product Errata RHBA-2016:2373 0 normal SHIPPED_LIVE fence-agents bug fix update 2016-11-03 13:51:30 UTC

Description Dominique Martinet 2015-11-27 10:14:02 UTC 
Created attachment 1099627 [details]
suggested patch

Description of problem:
The fence_ipmilan util got a 'diag' method added in el6 back in 2011, which got lost in the process of switching to a python version of the command for el7.

We feel that diag is still very useful and should be added back


Version-Release number of selected component (if applicable):
fence-agents-ipmilan-4.0.11-13.el7_1.1.x86_64
fence-agents-common-4.0.11-13.el7_1.1.x86_64

Steps to Reproduce:
1. Use fence_ipmilan with --method=diag
2.
3.

Actual results:
Option is refused and nothing is done

Expected results:
an ipmitool chassis power diag command is issued.
If your server is not configured to crash on this command (impitool chassis power diag), you will get something like this in dmesg (otherwise it'll take a kdump):
Uhhuh. NMI received for unknown reason 21 on CPU 0.
Do you have a strange power saving mode enabled?
Dazed and confused, but trying to continue


Additional info:
Patch suggestion from Bruno Travouillon (bruno.travouillon) is attached.
Comment 1 Marek Grac 2015-12-02 13:11:17 UTC 
A bit differnt tatch is now in upstream:

https://github.com/ClusterLabs/fence-agents/commit/7e65180162a54ff8c6d4c883be73376a3385dfe7

use with --action=diag

main reason for --action=diag instead of --method=diag is that --action was used in older versions of fence_ipmilan
Comment 2 Dominique Martinet 2015-12-14 16:56:57 UTC 
This patch does not work as expected, please see comment on github patch:
https://github.com/ClusterLabs/fence-agents/commit/7e65180162a54ff8c6d4c883be73376a3385dfe7#commitcomment-14964253

Quoting the interesting part here for posterity/notifications:
This does not behave as expected at all when I tried this?
I had to specify -m cycle to get the expected behavior, otherwise it does ipmitool chassis power off then waits till off then turns it back on (default -m offon)
e.g. 'fence_ipmilan -v -a IP -o diag -m cycle -P' worked but 'fence_ipmilan -v -a IP -o diag -P' did exactly the same as 'fence_ipmilan -v -a IP -o reboot -P'
Comment 3 Marek Grac 2015-12-16 09:45:08 UTC 
@Dominique:

you are right, fixed in upstream. Now, I have access to ipmi which allows diag, so it was tested properly.
Comment 4 Dominique Martinet 2015-12-17 18:08:44 UTC 
Fix works for me, thank you.
Comment 5 Oyvind Albrigtsen 2016-03-14 09:50:23 UTC 
Created attachment 1136105 [details]
Upstream patch updated for RHEL7
Comment 6 Mike McCune 2016-03-28 23:04:30 UTC 
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions
Comment 8 Oyvind Albrigtsen 2016-06-01 14:01:03 UTC 
*** Bug 1266599 has been marked as a duplicate of this bug. ***
Comment 11 errata-xmlrpc 2016-11-04 04:48:26 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2016-2373.html
Note You need to log in before you can comment on or make changes to this bug.