Skip to content

fix: ILO 5 metadata issue#3092

Open
digrajpu wants to merge 4 commits into
developfrom
fix/hpe_ilo5
Open

fix: ILO 5 metadata issue#3092
digrajpu wants to merge 4 commits into
developfrom
fix/hpe_ilo5

Conversation

@digrajpu

@digrajpu digrajpu commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@Kawron Kawron left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you provide little more context. What has changed, where did you found those new logs. Are the previous one wrong? Or is this update for new version of ILO?

Comment thread tests/test_hpe_ilo.py

mt = env.from_string(
"{{ mark }} {{ iso }}Z {{ host }} #ILO4 - - - Browser login: Administrator - 10.0.0.0(host.domain.local)\n"
"{{ mark }} {{ iso }}Z {{ host }} DriveArray: Smart Array - Drive is failed...\n"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I would also need more contest, but also, what happened to the old test here? Are we not supporting this other log format anymore?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The old test was removed while I was investigating the iLO 5 format from the documentation. Looking at it again, the existing tests cover a different format where the PROGRAM field is #ILO4, which will not match with updated parser. I'll restore the old test and update the parser to maintain backward compatibility.

@digrajpu

digrajpu commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator Author

Can you provide little more context. What has changed, where did you found those new logs. Are the previous one wrong? Or is this update for new version of ILO?

@Kawron @sbylica-splunk here we have official doc for remote syslog
https://support.hpe.com/hpesc/public/docDisplay?docId=a00105236en_us&docLocale=en_US&page=GUID-D7147C7F-2016-0901-06D0-00000000129A.html&utm_source=chatgpt.com

Comment thread package/etc/conf.d/conflib/syslog/app-syslog-hpe_ilo.conf Outdated
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.

3 participants