Discussion:
[Ipmitool-devel] RFC - Re-design of PEF user interface
Zdenek Styblik
2014-02-07 04:46:06 UTC
Permalink
Hello,

here is my proposal to re-design and re-work of ipmitool's PEF user
interface. This is as a reaction to Jordan Hargrave's request to add
'pef setpolicy'.
This is all based on chapter 30. PEF and Alerting Commands, IPMIv2.0
p.375. Aim isn't to implement all these commands, but to outline user
interface, so these commands can be implemented eventually.
There are couple unresolved commands and feel free to chip in your
ideas how to solve them. :)

Let's set deadline for comments until 28th February 2014.

<Chapter> <Title>
30.0 Help
* 'pef help'

30.1 Get PEF Capabilities
* 'pef info' - it's ok, but should be accompanied/complemented with
'pef capabilities' or something

30.2 Arm PEF Postpone Timer
* 'pef timer get' - this implies user input equal to 0xFFh
* 'pef timer set <0x00-0xFFh>'

30.3 Set PEF Config Parameters
30.4 Get PEF Config Parameters
* 'pef policy list'
* 'pef policy enable <policy_id>'
* 'pef policy disable <policy_id>'
* 'pef policy create <policy_id> <params>'
* 'pef policy delete <policy_id>'
* and the same thing for 'pef filters'

30.6 Set Last Processed Event ID
* unresolved despite it pretty much is '<something> <SEL_id>', but to
figure out name of <something> :)

30.7 Get Last Processed Event ID
* 'pef status' -> I have no idea, but 'status' is misleading/wrong
given the context, although Oracle's documentation puts it in a way it
makes sense, quote: ``Print the current PEF status (the last SEL entry
processed by the BMC, etc).''

30.8 PET Acknowledge
* 'pef pet ack <params>'

30.7 Alert Immediate
* unresolved - I haven't figured out this one

Changes to current/standing commands:
* 'pef list' - it lists all entries in PEF
* 'pef policy' - replaced with 'pef policy list'

Best regards,
Z.

--
Zdenek Styblik
email: ***@gmail.com
jabber: ***@gmail.com
Zdenek Styblik
2014-05-24 07:03:54 UTC
Permalink
Post by Zdenek Styblik
Hello,
here is my proposal to re-design and re-work of ipmitool's PEF user
interface. This is as a reaction to Jordan Hargrave's request to add
'pef setpolicy'.
This is all based on chapter 30. PEF and Alerting Commands, IPMIv2.0
p.375. Aim isn't to implement all these commands, but to outline user
interface, so these commands can be implemented eventually.
There are couple unresolved commands and feel free to chip in your
ideas how to solve them. :)
Let's set deadline for comments until 28th February 2014.
<Chapter> <Title>
30.0 Help
* 'pef help'
30.1 Get PEF Capabilities
* 'pef info' - it's ok, but should be accompanied/complemented with
'pef capabilities' or something
30.2 Arm PEF Postpone Timer
* 'pef timer get' - this implies user input equal to 0xFFh
* 'pef timer set <0x00-0xFFh>'
30.3 Set PEF Config Parameters
30.4 Get PEF Config Parameters
* 'pef policy list'
* 'pef policy enable <policy_id>'
* 'pef policy disable <policy_id>'
* 'pef policy create <policy_id> <params>'
* 'pef policy delete <policy_id>'
* and the same thing for 'pef filters'
30.6 Set Last Processed Event ID
* unresolved despite it pretty much is '<something> <SEL_id>', but to
figure out name of <something> :)
30.7 Get Last Processed Event ID
* 'pef status' -> I have no idea, but 'status' is misleading/wrong
given the context, although Oracle's documentation puts it in a way it
makes sense, quote: ``Print the current PEF status (the last SEL entry
processed by the BMC, etc).''
30.8 PET Acknowledge
* 'pef pet ack <params>'
30.7 Alert Immediate
* unresolved - I haven't figured out this one
* 'pef list' - it lists all entries in PEF
* 'pef policy' - replaced with 'pef policy list'
Best regards,
Z.
Attached is a patch for proposition above. This should make way for
``Add support for enable/disable PEF policy entries'' -
https://sourceforge.net/p/ipmitool/bugs/323/
Comments and code reviews are welcome. After all, that's the way forward.

Regards,
Z.
Post by Zdenek Styblik
--
Zdenek Styblik
Loading...