Skip to content

[Nokia][Device] Add device files for fabric monitor feature and set linkscan interval for Ramon#19144

Merged
rlhui merged 2 commits intosonic-net:masterfrom
saksarav-nokia:saksarav-nokia-fabric-monitor
May 31, 2024
Merged

[Nokia][Device] Add device files for fabric monitor feature and set linkscan interval for Ramon#19144
rlhui merged 2 commits intosonic-net:masterfrom
saksarav-nokia:saksarav-nokia-fabric-monitor

Conversation

@saksarav-nokia
Copy link
Copy Markdown
Contributor

Why I did it

The device files for fabric monitor feature was not present for Nokia platforms. Also the linkscan interval needs to be enabled for later BCM SDK versions

Work item tracking
  • Microsoft ADO (number only):

How I did it

1)Added device files for fabric monitor feature for Nokia IMM and SUP platforms
2) Set the linkscan interval for Ramons via sai_postinit_cmd.soc since setting via .bcm config is broken is not supported in SAI 10.X (CS00012345930)

How to verify it

1)Verified that the load_mingraph loads the fabric monitor device files and saved in the CONFIG_DB.
2)linkscan value is set correctly in BCM for Ramons

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

…inkscan interval for Ramon

Signed-off-by: saksarav <sakthivadivu.saravanaraj@nokia.com>
@saksarav-nokia
Copy link
Copy Markdown
Contributor Author

@judyjoseph pls review

Signed-off-by: saksarav <sakthivadivu.saravanaraj@nokia.com>
@saksarav-nokia
Copy link
Copy Markdown
Contributor Author

@arlakshm for viz

@rlhui rlhui merged commit 6a7aa29 into sonic-net:master May 31, 2024
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