Bug 918296

Summary: add cxoem command to ipmitool
Product: [Fedora] Fedora Reporter: Jeff Bastian <jbastian>
Component: ipmitoolAssignee: Ales Ledvinka <aledvink>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: 18CC: aledvink, blc, emcnabb, praveen_paladugu, tsmetana
Target Milestone: ---Keywords: Patch
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: ipmitool-1.8.12-16.fc20 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-11-05 16:23:01 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 919591, 919592    
Attachments:
Description Flags
patch to add cxoem command to ipmitool-1.8.12-9.fc18
none
patch to add cxoem command to ipmitool-1.8.12-9.fc18
none
cx6: additional patch with latest changes from upstream
none
cx6 single patch none

Description Jeff Bastian 2013-03-05 20:11:55 UTC 
Created attachment 705662 [details]
patch to add cxoem command to ipmitool-1.8.12-9.fc18

Description of problem:
Calxeda has added a 'cxoem' command to ipmitool to manage their servers.  For example:

$ ipmitool -I lanplus -H host.example.com -U user -P passwd cxoem info basic
Calxeda SoC (0x0096CD)
  Firmware Version: ECX-1000-v2.1.5
  SoC Version: v1.1.3
  Timestamp (1359478645): Tue Jan 29 10:57:25 2013


The patches are available in their git repo:
  http://sources.calxeda.com/gitweb/?p=ipmitool.git;a=summary

Their patches are based on ipmitool-1.8.11, but Fedora 18 is on 1.8.12, so I've adapted the patches to apply cleanly to the F18 sources.  See attached.

Please add the 'cxoem' command to ipmitool.


Version-Release number of selected component (if applicable):
ipmitool-1.8.12-9.fc18
Comment 1 Jeff Bastian 2013-03-05 21:20:19 UTC 
Koji scratch build of the packages:
  http://koji.fedoraproject.org/koji/taskinfo?taskID=5082831

And I've mirrored the build here:
  http://people.redhat.com/~jbastian/ipmitool-cxoem/
Comment 2 Ales Ledvinka 2013-03-06 11:48:25 UTC 
https://sourceforge.net/tracker/?func=detail&aid=3607033&group_id=95200&atid=610553
Comment 3 Jeff Bastian 2013-03-08 20:27:28 UTC 
Note: I've opened 3 new packages reviews, 2 of which depend on the cxoem patch for ipmitool:
  bug 919591: python-pyipmi
  bug 919592: cxmanage

The third is also needed by cxmanage, but it's not dependent on the cxoem patch:
  bug 919590: python-tftpy
Comment 4 Jeff Bastian 2013-04-18 22:51:51 UTC 
Created attachment 737460 [details]
patch to add cxoem command to ipmitool-1.8.12-9.fc18

Attached is a slimmer version of the patch to add the cxoem command.

43 commits from Calxeda were included in this version -- only those commits that touched ipmi_cxoem.c or ipmi_cxoem.h -- and 16 commits were excluded.  See the comments on the top of the patch for more details.
Comment 5 Jeff Bastian 2013-06-17 13:35:16 UTC 
Created attachment 762027 [details]
cx6: additional patch with latest changes from upstream

This patch builds on top of the patch from comment 4.  It contains the latest upstream changes from the past few months bringing the Fedora package to Calxeda version "cx6".  (The patch in comment 4 was cx3.)
Comment 6 Jeff Bastian 2013-06-17 13:38:29 UTC 
Created attachment 762028 [details]
cx6 single patch

Or, to simplify things, this attachment is a single patch that adds all Calxeda changes up through cx6 to Fedora 18 ipmitool-1.8.12-9
Comment 7 Jeff Bastian 2013-06-17 13:40:59 UTC 
(In reply to Jeff Bastian from comment #6)
> Or, to simplify things, this attachment is a single patch that adds all
> Calxeda changes up through cx6 to Fedora 18 ipmitool-1.8.12-9

Let me clarify: not *all* Calxeda changes.  Just those their commits that touch ipmi_cxoem.[ch] and related files.

There are some other commits that modify the SEL code and others; see the comments at the top of ipmitool-cxoem-cx3-slim.patch