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] Fix content of platform.json for DCS-7050CX3-32S #12082

Merged
merged 5 commits into from
Oct 19, 2022

Conversation

andywongarista
Copy link
Contributor

Why I did it

Some tests under platform_tests/api were failing on the 7050CX3 due to outdated facts in platform.json

How I did it

Updated platform.json facts with appropriate values

How to verify it

Run tests under platform_tests/api to verify no failures

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

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

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

@prgeor
Copy link
Contributor

prgeor commented Sep 22, 2022

@andywongarista could you rebase with master once?

1 similar comment
@prgeor
Copy link
Contributor

prgeor commented Sep 24, 2022

@andywongarista could you rebase with master once?

@prgeor
Copy link
Contributor

prgeor commented Dec 8, 2022

@yxieca not sure if this is already cherry-picked into 202205 as commented by @andywongarista above

@mssonicbld
Copy link
Collaborator

@andywongarista PR conflicts with 202205 branch

@qiluo-msft
Copy link
Collaborator

This commit could not be cleanly cherry-picked to 202012. Please submit another PR.

@liuh-80
Copy link
Contributor

liuh-80 commented Feb 9, 2023

Cherry pick PR merged: #13659

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.

6 participants