Skip to content

Commit c4af5ca

Browse files
patrickmacarthurVladimirKuk
authored andcommitted
[broadcom-dnx]: Install rdb-cli on syncd-dnx (sonic-net#20948)
This was missed on the original PR and caused show_techsupport sonic-mgmt tests to fail. Why I did it PR sonic-net#19268 replaced redis-rdb-tool with rdb-cli in the Broadcom XGS syncd container. However, it did not do the same for the DNX syncd container, leading to sonic-mgmt failures in show_techsupport tests on DNX platforms. How I did it I applied the same change to the Broadcom DNX syncd container. How to verify it Run sonic-mgmt show_techsupport tests on a Broadcom DNX DUT.
1 parent 49a999a commit c4af5ca

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

platform/broadcom/docker-syncd-brcm-dnx.mk

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ endif
2424

2525
$(DOCKER_SYNCD_DNX_BASE)_DEPENDS += $(SYNCD)
2626
$(DOCKER_SYNCD_DNX_BASE)_DEPENDS += $(BRCM_DNX_SAI)
27-
$(DOCKER_SYNCD_DNX_BASE)_FILES += $(DSSERVE) $(BCMCMD)
27+
$(DOCKER_SYNCD_DNX_BASE)_FILES += $(DSSERVE) $(BCMCMD) $(RDB-CLI)
2828

2929
ifeq ($(INSTALL_DEBUG_TOOLS), y)
3030
$(DOCKER_SYNCD_DNX_BASE)_DBG_DEPENDS += $(SYNCD_DBG) \

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

+2-2
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ RUN apt-get install -yf kmod
2323
## BRCM uses ethtool to set host interface speed
2424
RUN apt-get install -y ethtool
2525

26-
COPY ["files/dsserve", "files/bcmcmd", "start.sh", "start_led.sh", "bcmsh", "/usr/bin/"]
27-
RUN chmod +x /usr/bin/dsserve /usr/bin/bcmcmd
26+
COPY ["files/dsserve", "files/bcmcmd", "start.sh", "start_led.sh", "bcmsh", "files/rdb-cli", "/usr/bin/"]
27+
RUN chmod +x /usr/bin/dsserve /usr/bin/bcmcmd /usr/bin/rdb-cli
2828

2929
COPY ["supervisord.conf", "/etc/supervisor/conf.d/"]
3030
COPY ["files/supervisor-proc-exit-listener", "/usr/bin"]

0 commit comments

Comments
 (0)