Skip to content

Feature/sel list human readable - #212

Merged
hthiery merged 2 commits into
masterfrom
feature/sel-list-human-readable
Oct 9, 2026
Merged

hthiery merged 2 commits into
masterfrom
feature/sel-list-human-readable

Conversation

@hthiery

@hthiery hthiery commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

hthiery and others added 2 commits October 9, 2026 14:23
A SEL entry only had its raw fields, so the sensor type, the event
offset, the timestamp and the generator ID had to be looked up in the
IPMI specification.

Add the names of the sensor types and the descriptions of the generic
and sensor-specific event offsets (IPMI v2.0 tables 42-2 and 42-3, PICMG
3.0 hot swap states), and SelEntry methods to convert the sensor type,
event, direction, timestamp and generator ID to strings. A timestamp of
an event before the SEL time was set is shown as time since the BMC
initialization (IPMI v2.0 section 37.1). threshold_event_values()
returns the trigger reading and threshold of a threshold event.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
'sel list' printed the raw fields of the entries. Print one line per
entry with the date and time, the sensor type, the event description
and the direction, and the trigger reading and threshold of threshold
events. The data of OEM records is printed as hex bytes.

With -s/--sdr the SDRs are read to print the sensor names and to convert
the threshold event values. With -d/--details all fields of an entry
are printed. The option is not named -v, the global -v already sets
the log level.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>
@hthiery
hthiery merged commit e761c4e into master Oct 9, 2026
28 of 29 checks passed
@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 93.872% (+0.04%) from 93.832% — feature/sel-list-human-readable into master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants