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.

Sign up for GitHub

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

Jump to bottom

Test syncd with sai ptf v2 #12619

Closed
wants to merge 26 commits into from
Closed

Test syncd with sai ptf v2 #12619

wants to merge 26 commits into from

Conversation

richardyu-ms
Copy link
Contributor

Why I did it

How I did it

How to verify it

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 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
support sai-ptf-v2 in libsaithrift vs
3da8760
@richardyu-ms
add build target docker-ptf-sai syncd-rpcv2 and saiserverv2
4a3f166 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
add docker ptf sai
255f416 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
add build condition for broadcom
f05d082 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
add docker syncd dbg and add debug symbol to docker-saiserverv2
30d1a31 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
correct the build option
1fb3730
@richardyu-ms
change the azure pipeline build template
2f31a1b 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
change build option for docker-ptf-sai
9d37f8e
@richardyu-ms
enable ptf-sai docker build
14f7327
@richardyu-ms
remove the build for syncd-rpcv2
9d8a66c 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
fix issue in build tempalte
13dbc6a
@richardyu-ms
ignore useless package build when build sai-ptf
987d835 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
remove scapy version contraint
c7b3892 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
remove duplicated target docker-ptf
5278e7b 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
change template for testing the pipeline
549ddfc 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
remove duplicated target
fb999e7 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
fix error in make script
6b59e96 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
add shel to setup env
c624066 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
replace with certain platform name
a7c6a27 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
disable cache for syncd-rpcv2
2baab22 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
test without cache
3ef4a3d 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
disable cache
5628ecb 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
testing: disable the cache for build syncd-rpcv2
0d15eb4 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
add cache back and get the code ready for testing
84721fc 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
refactor code
c77e3fc 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms
add workaround for git files-ls issue
8b0aa2f 
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms richardyu-ms mentioned this pull request Nov 7, 2022
Merged
7 tasks
@richardyu-ms richardyu-ms reopened this Nov 11, 2022
richardyu-ms added a commit to richardyu-ms/sonic-buildimage that referenced this pull request Nov 19, 2022
@richardyu-ms
[SAI PTF]Support sai ptf v2 Syncd-rpc
ac33d6b 
Make syncd rpc docker which supports sai-ptf v2

local bulild the target

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=vs
NOSTRETCH=y NOJESSIE=y NOBULLSEYE=y SAITHRIFT_V2=y make target/docker-ptf-sai.gz

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=vs
NOSTRETCH=y NOJESSIE=y NOBULLSEYE=y make target/docker-ptf.gz

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=broadcom
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y SAITHRIFT_V2=y make target/docker-syncd-brcm-rpcv2.gz
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y SAITHRIFT_V2=y make target/docker-saiserverv2-brcm.gz

Test done:
sonic-net#12619
NOSTRETCH=y NOJESSIE=y make configure PLATFORM=broadcom
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y make target/docker-syncd-brcm-rpc.gz
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y make target/docker-saiserver-brcm.gz

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms richardyu-ms mentioned this pull request Nov 19, 2022
Merged
7 tasks
richardyu-ms added a commit that referenced this pull request Nov 21, 2022
@richardyu-ms
[SAI PTF]Support sai ptf v2 Syncd-rpc (#12761)
9734b42 
Make syncd rpc docker which supports sai-ptf v2

local bulild the target

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=vs
NOSTRETCH=y NOJESSIE=y NOBULLSEYE=y SAITHRIFT_V2=y make target/docker-ptf-sai.gz

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=vs
NOSTRETCH=y NOJESSIE=y NOBULLSEYE=y make target/docker-ptf.gz

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=broadcom
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y SAITHRIFT_V2=y make target/docker-syncd-brcm-rpcv2.gz
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y SAITHRIFT_V2=y make target/docker-saiserverv2-brcm.gz

Test done:
#12619
NOSTRETCH=y NOJESSIE=y make configure PLATFORM=broadcom
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y make target/docker-syncd-brcm-rpc.gz
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y make target/docker-saiserver-brcm.gz

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
StormLiangMS pushed a commit to StormLiangMS/sonic-buildimage that referenced this pull request Dec 8, 2022
@richardyu-ms @StormLiangMS
[SAI PTF]Support sai ptf v2 Syncd-rpc (sonic-net#12761)
a502692 
Make syncd rpc docker which supports sai-ptf v2

local bulild the target

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=vs
NOSTRETCH=y NOJESSIE=y NOBULLSEYE=y SAITHRIFT_V2=y make target/docker-ptf-sai.gz

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=vs
NOSTRETCH=y NOJESSIE=y NOBULLSEYE=y make target/docker-ptf.gz

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=broadcom
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y SAITHRIFT_V2=y make target/docker-syncd-brcm-rpcv2.gz
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y SAITHRIFT_V2=y make target/docker-saiserverv2-brcm.gz

Test done:
sonic-net#12619
NOSTRETCH=y NOJESSIE=y make configure PLATFORM=broadcom
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y make target/docker-syncd-brcm-rpc.gz
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y make target/docker-saiserver-brcm.gz

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
StormLiangMS pushed a commit that referenced this pull request Dec 10, 2022
@richardyu-ms @StormLiangMS
[SAI PTF]Support sai ptf v2 Syncd-rpc (#12761)
0eb82ab 
Make syncd rpc docker which supports sai-ptf v2

local bulild the target

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=vs
NOSTRETCH=y NOJESSIE=y NOBULLSEYE=y SAITHRIFT_V2=y make target/docker-ptf-sai.gz

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=vs
NOSTRETCH=y NOJESSIE=y NOBULLSEYE=y make target/docker-ptf.gz

NOSTRETCH=y NOJESSIE=y make configure PLATFORM=broadcom
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y SAITHRIFT_V2=y make target/docker-syncd-brcm-rpcv2.gz
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y SAITHRIFT_V2=y make target/docker-saiserverv2-brcm.gz

Test done:
#12619
NOSTRETCH=y NOJESSIE=y make configure PLATFORM=broadcom
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y make target/docker-syncd-brcm-rpc.gz
NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y make target/docker-saiserver-brcm.gz

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
@richardyu-ms richardyu-ms deleted the test_syncd_with_sai_ptf_v2 branch January 16, 2023 11:31
yaqiangz pushed a commit to yaqiangz/sonic-buildimage that referenced this pull request Feb 14, 2023
@neethajohn
Merged PR 7186175: Manual merge from github/master to internal
42cd9c0 
Resolved merge conflicts in the following files manually

device/arista/x86_64-arista_7050_qx32/Arista-7050-QX32-Flex/buffers_defaults_t0.j2
device/arista/x86_64-arista_7050_qx32/Arista-7050-QX32-Flex/buffers_defaults_t1.j2
device/arista/x86_64-arista_7050_qx32/Arista-7050-QX32/buffers_defaults_t0.j2
device/arista/x86_64-arista_7050_qx32/Arista-7050-QX32/buffers_defaults_t1.j2
device/arista/x86_64-arista_7050_qx32s/Arista-7050QX-32S-S4Q31/buffers_defaults_t0.j2
device/arista/x86_64-arista_7050_qx32s/Arista-7050QX-32S-S4Q31/buffers_defaults_t1.j2
device/dell/x86_64-dell_s6000_s1220-r0/Force10-S6000/buffers_defaults_t0.j2
device/dell/x86_64-dell_s6000_s1220-r0/Force10-S6000/buffers_defaults_t1.j2
src/sonic-config-engine/tests/test_j2files.py

Related work items: sonic-net#11864, sonic-net#12585, sonic-net#12607, sonic-net#12619, sonic-net#12626, sonic-net#12707, sonic-net#12714, sonic-net#12729, sonic-net#12733, sonic-net#12736, sonic-net#12750, sonic-net#12751, sonic-net#12755, sonic-net#12757, sonic-net#12758, sonic-net#12760, sonic-net#12761, sonic-net#12769, sonic-net#12783
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft qiluo-msft will be requested when the pull request is marked ready for review qiluo-msft is a code owner

@xumia xumia Awaiting requested review from xumia xumia will be requested when the pull request is marked ready for review xumia is a code owner

@lguohan lguohan Awaiting requested review from lguohan lguohan will be requested when the pull request is marked ready for review lguohan is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@richardyu-ms