Skip to content

feat(scl/cisco): add support for Cisco Nexus NXOS 9.3 syslog format #5410

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

smcclennon
Copy link

  • Add cisco-parser-nxos-timestamp-pattern for YYYY MMM DD HH:MM:SS timestamps
  • Extend date-parser with %Y %b %d %H:%M:%S format
  • Update sequence number handling to support both traditional "seqno: " and NXOS ": " prefixes
  • Add documentation and example for NXOS 9.3 format

The NXOS 9.3 format differs from traditional IOS by using different
timestamp and ": " prefix instead of "seqno: " after the priority field.

Example: <187>: 2025 Jun 25 11:27:28 GMT: %AUTHPRIV-3-SYSTEM_MSG: message

@kira-syslogng
Copy link
Contributor

Can one of the admins verify this patch?

1 similar comment
@kira-syslogng
Copy link
Contributor

Can one of the admins verify this patch?

@therandomstring
Copy link
Contributor

@smcclennon Thank you for your contribution!

Please be so kind as to rename your commit as to adhere to our guide. As a clarification to the link: the pipeline expects commits in a module: message format.

- Add cisco-parser-nxos-timestamp-pattern for YYYY MMM DD HH:MM:SS timestamps
- Extend date-parser with %Y %b %d %H:%M:%S format
- Update sequence number handling to support both traditional "seqno: " and NXOS ": " prefixes
- Add documentation and example for NXOS 9.3 format

The NXOS 9.3 format differs from traditional IOS by using different
timestamp and ": " prefix instead of "seqno: " after the priority field.

Example: <187>: 2025 Jun 25 11:27:28 GMT: %AUTHPRIV-3-SYSTEM_MSG: message

Signed-off-by: Shiraz McClennon <shiraz@smcclennon.com>
@smcclennon smcclennon force-pushed the feat/cisco-nxos-v9.3-support branch from 9698d10 to 65b6ce2 Compare July 2, 2025 02:05
@smcclennon
Copy link
Author

@therandomstring Thank you for the guidance, I've updated the commit message to follow the module: message format with the Signed-off-by line as requested. The changes have been pushed to the branch.

therandomstring
therandomstring previously approved these changes Jul 2, 2025
@therandomstring
Copy link
Contributor

@smcclennon Thank you for the changes! I've approved the PR. It is not mandatory, but if you could also write a news file for the changes, it would be very welcome.

Signed-off-by: Shiraz McClennon <shiraz@smcclennon.com>
@smcclennon smcclennon force-pushed the feat/cisco-nxos-v9.3-support branch from 72fca24 to 55c2938 Compare July 3, 2025 19:54
@smcclennon
Copy link
Author

@therandomstring Thank you for the approval and suggestion. I've added a news file entry with usage documentation and a summary for the changes

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