Skip to content

Commit 785e43f

Browse files
committed
[SAI PTF][202205]Support sai ptf v2 Syncd-rpc
cherry-pick sonic-net#12761 Make syncd rpc docker which supports sai-ptf v2 Part of previous PR sonic-net#11610 local bulild the target 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 Signed-off-by: richardyu-ms <[email protected]>
1 parent 7775f15 commit 785e43f

File tree

4 files changed

+29
-4
lines changed

4 files changed

+29
-4
lines changed

platform/broadcom/docker-saiserver-brcm.mk

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
DOCKER_SAISERVER_BRCM = docker-saiserver$(SAITHRIFT_VER)-brcm.gz
44
$(DOCKER_SAISERVER_BRCM)_PATH = $(PLATFORM_PATH)/docker-saiserver-brcm
5-
$(DOCKER_SAISERVER_BRCM)_DEPENDS += $(SAISERVER)
5+
$(DOCKER_SAISERVER_BRCM)_DEPENDS += $(SAISERVER) $(SAISERVER_DBG)
66
$(DOCKER_SAISERVER_BRCM)_FILES += $(DSSERVE) $(BCMCMD)
77
$(DOCKER_SAISERVER_BRCM)_LOAD_DOCKERS += $(DOCKER_CONFIG_ENGINE_BULLSEYE)
88
SONIC_DOCKER_IMAGES += $(DOCKER_SAISERVER_BRCM)
+20-2
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,39 @@
11
# docker image for brcm syncd with rpc
22

3-
DOCKER_SYNCD_BRCM_RPC = docker-syncd-brcm-rpc.gz
3+
DOCKER_SYNCD_BRCM_RPC = docker-syncd-brcm-rpc$(SAITHRIFT_VER).gz
4+
DOCKER_SYNCD_BRCM_RPC_DBG = docker-syncd-brcm-rpc$(SAITHRIFT_VER)-$(DBG_IMAGE_MARK).gz
45
$(DOCKER_SYNCD_BRCM_RPC)_PATH = $(PLATFORM_PATH)/docker-syncd-brcm-rpc
6+
7+
8+
#Support two different versions of thrift
9+
ifeq ($(SAITHRIFT_V2),y)
10+
$(DOCKER_SYNCD_BRCM_RPC)_DEPENDS += $(SYNCD_RPC) $(LIBTHRIFT_0_14_1) $(LIBTHRIFT_0_14_1_DEV) $(PYTHON3_THRIFT_0_14_1) $(THRIFT_0_14_1_COMPILER) $(PTF)
11+
else
512
$(DOCKER_SYNCD_BRCM_RPC)_DEPENDS += $(SYNCD_RPC) $(LIBTHRIFT) $(PTF)
13+
endif
14+
615
ifeq ($(INSTALL_DEBUG_TOOLS), y)
716
$(DOCKER_SYNCD_BRCM_RPC)_DEPENDS += $(SYNCD_RPC_DBG) \
817
$(LIBSWSSCOMMON_DBG) \
918
$(LIBSAIMETADATA_DBG) \
1019
$(LIBSAIREDIS_DBG)
1120
endif
1221
$(DOCKER_SYNCD_BRCM_RPC)_FILES += $(DSSERVE) $(BCMCMD) $(SUPERVISOR_PROC_EXIT_LISTENER_SCRIPT)
22+
1323
$(DOCKER_SYNCD_BRCM_RPC)_LOAD_DOCKERS += $(DOCKER_SYNCD_BASE)
24+
$(DOCKER_SYNCD_BRCM_RPC)_DBG_DEPENDS += $($(DOCKER_CONFIG_ENGINE_BULLSEYE)_DBG_DEPENDS)
25+
$(DOCKER_SYNCD_BRCM_RPC)_DBG_IMAGE_PACKAGES = $($(DOCKER_CONFIG_ENGINE_BULLSEYE)_DBG_IMAGE_PACKAGES)
26+
1427
SONIC_DOCKER_IMAGES += $(DOCKER_SYNCD_BRCM_RPC)
1528
ifeq ($(ENABLE_SYNCD_RPC),y)
1629
SONIC_INSTALL_DOCKER_IMAGES += $(DOCKER_SYNCD_BRCM_RPC)
1730
endif
1831

32+
SONIC_DOCKER_DBG_IMAGES += $(DOCKER_SYNCD_BRCM_RPC_DBG)
33+
ifneq ($(ENABLE_SYNCD_RPC),y)
34+
SONIC_INSTALL_DOCKER_DBG_IMAGES += $(DOCKER_SYNCD_BRCM_RPC_DBG)
35+
endif
36+
1937
$(DOCKER_SYNCD_BRCM_RPC)_CONTAINER_NAME = syncd
2038
$(DOCKER_SYNCD_BRCM_RPC)_VERSION = 1.0.0+rpc
2139
$(DOCKER_SYNCD_BRCM_RPC)_PACKAGE_NAME = syncd
@@ -24,9 +42,9 @@ $(DOCKER_SYNCD_BRCM_RPC)_RUN_OPT += -v /host/machine.conf:/etc/machine.conf
2442
$(DOCKER_SYNCD_BRCM_RPC)_RUN_OPT += -v /host/warmboot:/var/warmboot
2543
$(DOCKER_SYNCD_BRCM_RPC)_RUN_OPT += -v /var/run/docker-syncd:/var/run/sswsyncd
2644
$(DOCKER_SYNCD_BRCM_RPC)_RUN_OPT += -v /etc/sonic:/etc/sonic:ro
27-
2845
$(DOCKER_SYNCD_BRCM_RPC)_BASE_IMAGE_FILES += bcmcmd:/usr/bin/bcmcmd
2946
$(DOCKER_SYNCD_BRCM_RPC)_BASE_IMAGE_FILES += bcmsh:/usr/bin/bcmsh
3047
$(DOCKER_SYNCD_BRCM_RPC)_MACHINE = broadcom
3148

3249
SONIC_BULLSEYE_DOCKERS += $(DOCKER_SYNCD_BRCM_RPC)
50+
SONIC_BULLSEYE_DBG_DOCKERS += $(DOCKER_SYNCD_BRCM_RPC_DBG)

platform/broadcom/docker-syncd-brcm-rpc/Dockerfile.j2

+1
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ RUN apt-get update \
2525
cmake \
2626
libqt5core5a \
2727
libqt5network5 \
28+
gdb \
2829
libboost-atomic1.74.0
2930

3031
RUN dpkg_apt() { [ -f $1 ] && { dpkg -i $1 || apt-get -y install -f; } || return 1; } ; \

rules/syncd.mk

+7-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,13 @@ $(SYNCD_RPC)_RDEPENDS += $(LIBSAIREDIS) $(LIBSAIMETADATA)
1717
$(eval $(call add_derived_package,$(SYNCD),$(SYNCD_RPC)))
1818

1919
# Inject libthrift build dependency for RPC build
20-
$(SYNCD)_DEPENDS += $(LIBSWSSCOMMON_DEV) $(LIBTHRIFT_DEV)
20+
# Support two different versions of thrift
21+
ifeq ($(SAITHRIFT_V2),y)
22+
$(SYNCD)_DEPENDS += $(LIBTHRIFT_0_14_1_DEV)
23+
else
24+
$(SYNCD)_DEPENDS += $(LIBTHRIFT_DEV)
25+
endif
26+
$(SYNCD)_DEPENDS += $(LIBSWSSCOMMON_DEV)
2127
$(SYNCD)_DPKG_TARGET = binary-syncd-rpc
2228
endif
2329

0 commit comments

Comments
 (0)