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

[7050] define hwsku.json for Arista-7050QX-32S-S4Q31 to skip SFP checks for first 4 ports #8624

Merged
merged 2 commits into from
Sep 1, 2021

Conversation

yxieca
Copy link
Contributor

@yxieca yxieca commented Aug 28, 2021

Why I did it

The first 4 ports on this dut are breakout ports. They might not always be connected in lab. Mark them as 'RJ45' to skip the SFP check since they are by default disabled.

How to verify it

run platform test_reboot.py

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

@yxieca yxieca marked this pull request as ready for review August 30, 2021 03:03
@yxieca
Copy link
Contributor Author

yxieca commented Aug 30, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit 7735e8a into sonic-net:master Sep 1, 2021
@yxieca yxieca deleted the sfp branch September 1, 2021 00:15
qiluo-msft pushed a commit that referenced this pull request Sep 1, 2021
…ks for first 4 ports (#8624)

Why I did it
The first 4 ports on this dut are breakout ports. They might not always be connected in lab. Mark them as 'RJ45' to skip the SFP check since they are by default disabled.

How to verify it
run platform test_reboot.py

Signed-off-by: Ying Xie <[email protected]>
judyjoseph pushed a commit that referenced this pull request Sep 2, 2021
…ks for first 4 ports (#8624)

Why I did it
The first 4 ports on this dut are breakout ports. They might not always be connected in lab. Mark them as 'RJ45' to skip the SFP check since they are by default disabled.

How to verify it
run platform test_reboot.py

Signed-off-by: Ying Xie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants