Skip to content
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

400G-ZR : Time window based PM statistics collection #390

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

jaganbal-a
Copy link

Description

window based PM statistics collection. Please refer to the HLD for more details sonic-net/SONiC#1258

Motivation and Context

As per the above HLD, PM stats history is not present/maintained for 400G-ZR modules as of today.
This PR has the changes to collect PM stats and store in time window of different granularity in state-DB as described in the HLD. The stored PM stats for each window of a port then can be retrieved by CLI defined in the HLD.

How Has This Been Tested?

The changes are tested with inphi and acacia 400G-ZR modules on Cisco LC88-36FH-M-O pid.

Refer to the HLD for problem description and design.
HLD: sonic-net/SONiC#1258
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jul 21, 2023

CLA Not Signed

@jaganbal-a jaganbal-a changed the title Signed-off-by: Jaganathan Anbalagan <[email protected]> 400G-ZR : Time window based PM statistics collection Jul 21, 2023
PM statistics is collected through scheduler
@jaganbal-a
Copy link
Author

PM_Stats_UT.txt

Jaganathan Anbalagan and others added 3 commits August 18, 2023 13:01
Made changes to control vdm freeze and unfreeze pm_mgr.
60sec window will not be filled if the platform uses 120sec pm_interval, removed  white space
removed the _TABLE from the TRANSCEIVER_PM_WINDOW_STATS table name.
sonic-xcvrd/xcvrd/pm_mgr.py Outdated Show resolved Hide resolved
sonic-xcvrd/xcvrd/pm_mgr.py Show resolved Hide resolved
@abdosi
Copy link
Contributor

abdosi commented Apr 22, 2024

@longhuan-cisco : Can you please provide when you you will resolve the conflict ?

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