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

[SAIServer]Upgrade SAI server init script #13175

Merged
merged 4 commits into from
Jan 1, 2023

Conversation

richardyu-ms
Copy link
Contributor

@richardyu-ms richardyu-ms commented Dec 26, 2022

Why I did it

why
In order to apply different config across different platform, and use the code with a unified format, reuse syncd init script to init saiserver.

How I did it

how
Reuse syncd init script

How to verify it

Test
Test in DUT s6000 and dx010 with sonic 202205

ptf --test-dir ptf saisanity.L2SanityTest --relax --xunit --xunit-dir "/tmp/sai_qualify/test_results_tmp" --interface '0-0@eth0' --interface '0-1@eth1' --interface '0-2@eth2' --interface '0-3@eth3' --interface '0-4@eth4' --interface '0-5@eth5' --interface '0-6@eth6' --interface '0-7@eth7' --interface '0-8@eth8' --interface '0-9@eth9' --interface '0-10@eth10' --interface '0-11@eth11' --interface '0-12@eth12' --interface '0-13@eth13' --interface '0-14@eth14' --interface '0-15@eth15' --interface '0-16@eth16' --interface '0-17@eth17' --interface '0-18@eth18' --interface '0-19@eth19' --interface '0-20@eth20' --interface '0-21@eth21' --interface '0-22@eth22' --interface '0-23@eth23' --interface '0-24@eth24' --interface '0-25@eth25' --interface '0-26@eth26' --interface '0-27@eth27' --interface '0-28@eth28' --interface '0-29@eth29' --interface '0-30@eth30' --interface '0-31@eth31' --relax "--test-params=thrift_server='$DUTIP';port_config_ini='/tmp/sai_qualify/sai_test/resources/port_config.ini'"

ptf --test-dir test/sai_test sai_sanity_test.SaiSanityTest --relax --xunit --xunit-dir "/tmp/sai_qualify/test_results_tmp" --interface '0-0@eth0' --interface '0-1@eth1' --interface '0-2@eth2' --interface '0-3@eth3' --interface '0-4@eth4' --interface '0-5@eth5' --interface '0-6@eth6' --interface '0-7@eth7' --interface '0-8@eth8' --interface '0-9@eth9' --interface '0-10@eth10' --interface '0-11@eth11' --interface '0-12@eth12' --interface '0-13@eth13' --interface '0-14@eth14' --interface '0-15@eth15' --interface '0-16@eth16' --interface '0-17@eth17' --interface '0-18@eth18' --interface '0-19@eth19' --interface '0-20@eth20' --interface '0-21@eth21' --interface '0-22@eth22' --interface '0-23@eth23' --interface '0-24@eth24' --interface '0-25@eth25' --interface '0-26@eth26' --interface '0-27@eth27' --interface '0-28@eth28' --interface '0-29@eth29' --interface '0-30@eth30' --interface '0-31@eth31' --relax "--test-params=thrift_server='$DUTIP';port_config_ini='/tmp/sai_qualify/sai_test/resources/port_config.ini'"

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

Todo:
Copy and change code automatically from syncd submodules

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

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. 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)

@richardyu-ms richardyu-ms marked this pull request as ready for review December 26, 2022 07:31
why
In order to apply different config across different platform, and use the code with a unified format, reuse syncd init script to init saiserver.

how
Reuse syncd init script

Test
Test in DUT
@Gfrom2016 Gfrom2016 added the Request for 202111 Branch For PRs being requested for 202111 branch label Dec 28, 2022
why
In order to apply different config across different platform, and use the code with a unified format, reuse syncd init script to init saiserver.

how
Reuse syncd init script

Test
Test in DUT
@richardyu-ms richardyu-ms force-pushed the update_saiserver_start_script branch 3 times, most recently from 3c935b5 to e4fc582 Compare December 29, 2022 11:04
@richardyu-ms richardyu-ms reopened this Dec 30, 2022
@xumia xumia merged commit bca955f into sonic-net:202205 Jan 1, 2023
richardyu-ms added a commit to richardyu-ms/sonic-buildimage that referenced this pull request Jan 2, 2023
Why I did it
why
In order to apply different config across different platform, and use the code with a unified format, reuse syncd init script to init saiserver.

How I did it
how
Reuse syncd init script

How to verify it
Test
Test in DUT s6000 and dx010 with sonic 202205
richardyu-ms added a commit to richardyu-ms/sonic-buildimage that referenced this pull request Jan 2, 2023
Why I did it
why
In order to apply different config across different platform, and use the code with a unified format, reuse syncd init script to init saiserver.

How I did it
how
Reuse syncd init script

How to verify it
Test
Test in DUT s6000 and dx010 with sonic 202205
xumia pushed a commit that referenced this pull request Jan 3, 2023
Why I did it
why
In order to apply different config across different platform, and use the code with a unified format, reuse syncd init script to init saiserver.

How I did it
how
Reuse syncd init script

How to verify it
Test
Test in DUT s6000 and dx010 with sonic 202205
xumia pushed a commit that referenced this pull request Jan 3, 2023
Why I did it
why
In order to apply different config across different platform, and use the code with a unified format, reuse syncd init script to init saiserver.

How I did it
how
Reuse syncd init script

How to verify it
Test
Test in DUT s6000 and dx010 with sonic 202205
richardyu-ms added a commit to richardyu-ms/sonic-buildimage that referenced this pull request Jan 3, 2023
…t#13227)

Why I did it
why
In order to apply different config across different platform, and use the code with a unified format, reuse syncd init script to init saiserver.

How I did it
how
Reuse syncd init script

How to verify it
Test
Test in DUT s6000 and dx010 with sonic 202205
xumia pushed a commit that referenced this pull request Jan 6, 2023
Why I did it
why
In order to apply different config across different platform, and use the code with a unified format, reuse syncd init script to init saiserver.

How I did it
how
Reuse syncd init script

How to verify it
Test
Test in DUT s6000 and dx010 with sonic 202205
@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 2, 2023

202012 branch PR merged: #13226

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.

5 participants