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

[ARISTA][7060] Adding media_settings.json for x86_64-arista_7060_cx32s #17652

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

Conversation

vdahiya12
Copy link
Contributor

@vdahiya12 vdahiya12 commented Jan 2, 2024

Why I did it

This PR adds a media_settings.json file to apply the parameters for arista x86_64-arista_7060_cx32s platform for
QSFP+ interface of 40G speed and LR4, SR4 types.
This change will go together with sonic-net/sonic-platform-daemons#417 which has the support for parsing QSFP+ 40G speed regex style keys in media_settings.json and apply the change to interface parameters in APPL DB
This is indented as a part of long term fix repair item to correct RX_ERR on links facing x86_64-arista_7060_cx32s with 40G optical cables.

Work item tracking

25941176

  • Microsoft ADO (number only):

How I did it

added the meda_settings.json file

How to verify it

tested by deploying the change and running it with xcvrd/SAI changes

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)

Signed-off-by: vaibhav-dahiya <vdahiya@microsoft.com>
Signed-off-by: vaibhav-dahiya <vdahiya@microsoft.com>
@vdahiya12 vdahiya12 requested a review from lguohan as a code owner January 3, 2024 00:41
@vdahiya12 vdahiya12 requested a review from prgeor January 3, 2024 18:43
Signed-off-by: vaibhav-dahiya <vdahiya@microsoft.com>
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.

1 participant