Skip to content

[device]Add configuration parameters for as5835_54x#6243

Closed
dereksun01 wants to merge 2 commits intosonic-net:masterfrom
dereksun01:new3branch2020
Closed

[device]Add configuration parameters for as5835_54x#6243
dereksun01 wants to merge 2 commits intosonic-net:masterfrom
dereksun01:new3branch2020

Conversation

@dereksun01
Copy link
Contributor

@dereksun01 dereksun01 commented Dec 18, 2020

Signed-off-by: derek_sun derek_sun@edge-core

- Why I did it
Add configuration parameters for as5835_54x.

- How I did it
Modify device configuration file(mv2-as5835-48x10G+6x100G.config.bcm).

- How to verify it
-Running this configuration file on device(as5835_54x).

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

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog

Add configuration parameters for as5835_54x

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

derek_sun added 2 commits December 18, 2020 09:06
Signed-off-by: derek_sun <derek_sun@edge-core>
Signed-off-by: derek_sun <derek_sun@edge-core>
@dereksun01
Copy link
Contributor Author

Retest vsimage please

@dereksun01
Copy link
Contributor Author

Dear Manager,

Could you merge this PR? or need to improve it ? Thanks.

@dereksun01
Copy link
Contributor Author

Dear Manager,

Could you merge this PR to 202012 branch? Thanks.

@Lewis-Kang
Copy link

@lguohan could you please review and merge or let us know your concern? Thanks.

@dereksun01
Copy link
Contributor Author

dereksun01 commented Jan 21, 2021

Dear Manager,

Could you merge this PR to 202012 branch?
We have verify this PR on 202012 branch and it's works fine, Thanks.
as5835_54x_202012branch.log

@dereksun01
Copy link
Contributor Author

Dear Manager,

Could you merge this PR into 202012 branch too?
We have test this PR on 202012 branch and test log can refer the as5835_54x_202012branch.log, thanks.

@dereksun01 dereksun01 requested a review from jleveque as a code owner February 6, 2021 20:29
@dereksun01
Copy link
Contributor Author

-Close by derek_sun
-Reference: PR-6970
#6970

@dereksun01 dereksun01 closed this Mar 8, 2021
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.

2 participants