Skip to content

Add SONiC HLD for Forwarding Path Online Diagnostics#2442

Draft
sridhartalari wants to merge 1 commit into
sonic-net:masterfrom
sridhartalari:online-diagnostics-hld
Draft

Add SONiC HLD for Forwarding Path Online Diagnostics#2442
sridhartalari wants to merge 1 commit into
sonic-net:masterfrom
sridhartalari:online-diagnostics-hld

Conversation

@sridhartalari

@sridhartalari sridhartalari commented Jul 2, 2026

Copy link
Copy Markdown

Summary

  • Adds a community SONiC HLD for Forwarding Path Online Diagnostics, mapping the upstream SAI switch attributes and statistics to SONiC configuration, orchestration, counters, CLI, and telemetry.
  • Defines CONFIG/APPL/STATE DB schema, SwitchOrch integration, Flex Counter polling for aggregate TX/RX/loss/corruption stats, multi-ASIC behavior, warmboot impact, and a test plan.

Authors: Anand Mehra, Xiaohu Huang, Sridhar Talari

Motivation

Forwarding path online diagnostics provide switch-scoped active datapath health checks through periodic probe injection and validation. The upstream SAI Online Diagnostics proposal standardizes this capability across vendors. SONiC needs a community design for how forwarding path online diagnostics are configured, observed, and tested.

Test plan

  • Documentation review by SONiC community / platform WG
  • Align with SAI proposal merge timeline (SAI_SWITCH_ATTR_ONLINE_DIAG_*, SAI_SWITCH_STAT_ONLINE_DIAG_*)
  • Follow-up implementation PRs for SwitchOrch, CLI, and flex counters per this HLD

Related work

  • SAI proposal: Online Diagnostics switch attributes and statistics

@linux-foundation-easycla

linux-foundation-easycla Bot commented Jul 2, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: sridhartalari / name: stalarir (6b153b5)

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@sridhartalari sridhartalari force-pushed the online-diagnostics-hld branch from 2b76503 to 6b153b5 Compare July 2, 2026 21:40
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@sridhartalari sridhartalari changed the title Add SONiC HLD for SAI Online Diagnostics Add SONiC HLD for Forwarding Path Online Diagnostics Jul 2, 2026
@sridhartalari sridhartalari force-pushed the online-diagnostics-hld branch from 6b153b5 to 7959f37 Compare July 2, 2026 21:47
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@sridhartalari sridhartalari force-pushed the online-diagnostics-hld branch 2 times, most recently from 68b94fd to 0b7967f Compare July 2, 2026 22:45
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

1 similar comment
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

Define the community SONiC integration model for forwarding path
online diagnostics: CONFIG/APPL/STATE DB schema, SwitchOrch and flex-counter
design, CLI, telemetry, and testing requirements mapped to the upstream
SAI Online Diagnostics proposal.
@sridhartalari sridhartalari force-pushed the online-diagnostics-hld branch from 0b7967f to c77479f Compare July 3, 2026 00:01
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@sridhartalari sridhartalari marked this pull request as draft July 3, 2026 00:08
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