Skip to content

Conversation

@Pterosaur
Copy link
Contributor

This pull request refines the documentation and sample code for transmitting IPFIX telemetry data over netlink messages. The changes clarify the netlink message structure, provide a visual breakdown, and update the kernel-side sample code to directly append IPFIX data as payload without extra netlink attributes.

Documentation improvements:

  • Expanded the explanation of netlink message composition, detailing the netlink and genetlink headers and describing how IPFIX recordings are included as payload. Added a mermaid diagram to visually illustrate the message structure.

Sample code updates:

  • Updated the kernel sample code to calculate the total IPFIX payload length, allocate the message buffer accordingly, and append IPFIX data directly to the payload, removing the use of netlink attributes and improving error handling.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@Pterosaur Pterosaur requested a review from r12f August 20, 2025 10:09
r12f
r12f previously approved these changes Aug 28, 2025
---
title: Netlink message structure
---
packet-beta

Choose a reason for hiding this comment

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

packet-data ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Here is 'packet-beta' that is a flag of mermaid. https://docs.mermaidchart.com/mermaid-oss/syntax/packet.html

Signed-off-by: Ze Gan <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

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.

5 participants