Project

General

Profile

Feature #24765

Add ipmitool sel elist to freenas-debug

Added by Ash Gokhale almost 4 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Nice to have
Assignee:
William Grzybowski
Category:
Middleware
Target version:
Estimated time:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

It reports threshold explanations in addition to the events.

[root@zoltan-a] /usr/local/libexec/freenas-debug/ipmi# diff u vipmi.sh ipmi.sh
--
vipmi.sh 2017-06-21 13:31:10.409621000 -0700
+++ ipmi.sh 2017-06-21 13:31:26.313901000 -0700
@ -36,8 +36,8 @
ipmitool lan print
section_footer

- section_header "ipmitool sel list"
- ipmitool sel list
+ section_header "ipmitool sel elist"
+ ipmitool sel elist
section_footer

Associated revisions

Revision adf4a17b (diff)
Added by William Grzybowski almost 4 years ago

feat(debug): use sel elist instead of list Submitted by: Ash Ticket: #24765

Revision 5d597b44 (diff)
Added by William Grzybowski almost 4 years ago

feat(debug): use sel elist instead of list Submitted by: Ash Ticket: #24765 (cherry picked from commit adf4a17b9c57098abc517e4717083032a3fd1fe0)

Revision d23144c5 (diff)
Added by William Grzybowski almost 4 years ago

feat(debug): use sel elist instead of list Submitted by: Ash Ticket: #24765

History

#1 Updated by Vaibhav Chauhan almost 4 years ago

  • Target version changed from 11.1 to 11.0-U1

#2 Updated by William Grzybowski almost 4 years ago

  • Status changed from Unscreened to Screened

#3 Updated by William Grzybowski almost 4 years ago

  • Status changed from Screened to Needs Developer Review
  • Assignee changed from William Grzybowski to Ash Gokhale

Ash please review the commit and assign back to me.

#4 Updated by Vaibhav Chauhan almost 4 years ago

  • Target version changed from 11.0-U1 to 11.0-U2

#5 Updated by Vaibhav Chauhan almost 4 years ago

  • Target version changed from 11.0-U2 to 11.0-U3

#6 Avatar?id=14398&size=24x24 Updated by Kris Moore over 3 years ago

  • Status changed from Needs Developer Review to Reviewed by Developer
  • Assignee changed from Ash Gokhale to William Grzybowski

Lgtm

#7 Updated by Vaibhav Chauhan over 3 years ago

please publish your changes in a PR which will be merged against stable branch, also please let us know here when you have PR ready.

#8 Updated by Vaibhav Chauhan over 3 years ago

  • Status changed from Reviewed by Developer to 47

#9 Updated by Dru Lavigne over 3 years ago

  • Subject changed from debug should capture ipmitool sel elist [patch] to Add ipmitool sel elist to freenas-debug

#10 Updated by Joe Maloney over 3 years ago

  • Status changed from 47 to Ready For Release
  • Needs QA changed from Yes to No
  • QA Status Test Passes added
  • QA Status deleted (Not Tested)

Seems to work as expected:


[jmaloney@joe-laptop] ~/Downloads/ixdiagnose% grep -R 'ipmitool sel elist' *
fndebug/IPMI/dump.txt:+                         ipmitool sel elist @1503946837                         +
fndebug/IPMI/dump.txt:debug finished in 5 seconds for ipmitool sel elist
fndebug/System/dump.txt:6484  -  R      0:00.00 ipmitool sel elist

#11 Updated by Vaibhav Chauhan over 3 years ago

  • Status changed from Ready For Release to Resolved

Also available in: Atom PDF