Skip to content

Commit 9cbaf42

Browse files
[Frr]: Frr bookworm upgrade (#18233)
Co-authored-by: Kalimuthu Velappan <[email protected]>
1 parent ab08b78 commit 9cbaf42

File tree

6 files changed

+16
-13
lines changed

6 files changed

+16
-13
lines changed

dockers/docker-fpm-frr/Dockerfile.j2

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{% from "dockers/dockerfile-macros.j2" import install_debian_packages, install_python_wheels, copy_files %}
2-
FROM docker-swss-layer-bullseye-{{DOCKER_USERNAME}}:{{DOCKER_USERTAG}}
2+
FROM docker-swss-layer-bookworm-{{DOCKER_USERNAME}}:{{DOCKER_USERTAG}}
33

44
ARG docker_container_name
55
ARG frr_user_uid

rules/docker-fpm-frr.mk

+5-5
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,13 @@ $(DOCKER_FPM_FRR)_PATH = $(DOCKERS_PATH)/$(DOCKER_FPM_FRR_STEM)
88
$(DOCKER_FPM_FRR)_PYTHON_WHEELS += $(SONIC_BGPCFGD) $(SONIC_FRR_MGMT_FRAMEWORK)
99

1010
$(DOCKER_FPM_FRR)_DEPENDS += $(FRR) $(FRR_SNMP) $(SWSS) $(LIBYANG2) $(SONIC_RSYSLOG_PLUGIN)
11-
$(DOCKER_FPM_FRR)_DBG_DEPENDS = $($(DOCKER_SWSS_LAYER_BULLSEYE)_DBG_DEPENDS)
11+
$(DOCKER_FPM_FRR)_DBG_DEPENDS = $($(DOCKER_SWSS_LAYER_BOOKWORM)_DBG_DEPENDS)
1212
$(DOCKER_FPM_FRR)_DBG_DEPENDS += $(SWSS_DBG) $(LIBSWSSCOMMON_DBG) \
1313
$(FRR_DBG) $(FRR_SNMP_DBG) $(LIBYANG2_DBG) $(SONIC_RSYSLOG_PLUGIN)
1414

15-
$(DOCKER_FPM_FRR)_DBG_IMAGE_PACKAGES = $($(DOCKER_SWSS_LAYER_BULLSEYE)_DBG_IMAGE_PACKAGES)
15+
$(DOCKER_FPM_FRR)_DBG_IMAGE_PACKAGES = $($(DOCKER_SWSS_LAYER_BOOKWORM)_DBG_IMAGE_PACKAGES)
1616

17-
$(DOCKER_FPM_FRR)_LOAD_DOCKERS += $(DOCKER_SWSS_LAYER_BULLSEYE)
17+
$(DOCKER_FPM_FRR)_LOAD_DOCKERS += $(DOCKER_SWSS_LAYER_BOOKWORM)
1818

1919
$(DOCKER_FPM_FRR)_VERSION = 1.0.0
2020
$(DOCKER_FPM_FRR)_PACKAGE_NAME = fpm-frr
@@ -42,5 +42,5 @@ $(DOCKER_FPM_FRR)_BASE_IMAGE_FILES += TSC:/usr/bin/TSC
4242
$(DOCKER_FPM_FRR)_BASE_IMAGE_FILES += TS:/usr/bin/TS
4343
$(DOCKER_FPM_FRR)_BASE_IMAGE_FILES += idf_isolation:/usr/bin/idf_isolation
4444

45-
SONIC_BULLSEYE_DOCKERS += $(DOCKER_FPM_FRR)
46-
SONIC_BULLSEYE_DBG_DOCKERS += $(DOCKER_FPM_FRR_DBG)
45+
SONIC_BOOKWORM_DOCKERS += $(DOCKER_FPM_FRR)
46+
SONIC_BOOKWORM_DBG_DOCKERS += $(DOCKER_FPM_FRR_DBG)

rules/libyang.mk

-3
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,7 @@ LIBYANG = libyang_$(LIBYANG_VERSION)_$(CONFIGURED_ARCH).deb
1212
$(LIBYANG)_SRC_PATH = $(SRC_PATH)/libyang
1313
# introduce artifical dependency between LIBYANG and FRR
1414
# make sure LIBYANG is compile after FRR
15-
# TODO: Remove once snmp build has been updated
16-
ifeq ($(BLDENV),bullseye)
1715
$(LIBYANG)_AFTER = $(FRR)
18-
endif
1916
SONIC_MAKE_DEBS += $(LIBYANG)
2017

2118
LIBYANG_DEV = libyang-dev_$(LIBYANG_VERSION)_$(CONFIGURED_ARCH).deb

src/sonic-bgpcfgd/tests/test_bbr.py

+3-2
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,10 @@
55
from copy import deepcopy
66
from . import swsscommon_test
77

8+
import sys
9+
sys.modules["swsscommon"] = swsscommon_test
810

9-
with patch.dict("sys.modules", swsscommon=swsscommon_test):
10-
from bgpcfgd.managers_bbr import BBRMgr
11+
from bgpcfgd.managers_bbr import BBRMgr
1112

1213
global_constants = {
1314
"bgp": {

src/sonic-bgpcfgd/tests/test_setsrc.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,10 @@
77
from . import swsscommon_test
88
from swsscommon import swsscommon
99

10-
with patch.dict("sys.modules", swsscommon=swsscommon_test):
11-
from bgpcfgd.managers_setsrc import ZebraSetSrc
10+
import sys
11+
sys.modules["swsscommon"] = swsscommon_test
12+
13+
from bgpcfgd.managers_setsrc import ZebraSetSrc
1214

1315
TEMPLATE_PATH = os.path.abspath('../../dockers/docker-fpm-frr/frr')
1416

src/sonic-frr/Makefile

+3
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ DERIVED_TARGET = $(FRR_PYTHONTOOLS) $(FRR_DBG) $(FRR_SNMP) $(FRR_SNMP_DBG)
77
SUFFIX = $(shell date +%Y%m%d\.%H%M%S)
88
STG_BRANCH = stg_temp.$(SUFFIX)
99

10+
# DEBEMAIL required by gpb dch
11+
export DEBEMAIL := [email protected]
12+
1013
$(addprefix $(DEST)/, $(MAIN_TARGET)): $(DEST)/% :
1114
# Build the package
1215
pushd ./frr

0 commit comments

Comments
 (0)