Frr 8.2 upgrade #10691
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
|
|
||
| SPATH := $($(LIBYANG2)_SRC_PATH) | ||
| DEP_FILES := $(SONIC_COMMON_FILES_LIST) rules/libyang2.mk rules/libyang2.dep | ||
| DEP_FILES += $(SONIC_COMMON_BASE_FILES_LIST) | ||
| DEP_FILES += $(shell git ls-files $(SPATH)) | ||
|
|
||
| $(LIBYANG2)_CACHE_MODE := GIT_CONTENT_SHA | ||
| $(LIBYANG2)_DEP_FLAGS := $(SONIC_COMMON_FLAGS_LIST) | ||
| $(LIBYANG2)_DEP_FILES := $(DEP_FILES) | ||
|
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,43 @@ | ||
| # libyang2 | ||
|
|
||
| LIBYANG2_VERSION_BASE = 2.0 | ||
| LIBYANG2_VERSION = $(LIBYANG2_VERSION_BASE).112 | ||
| LIBYANG2_SUBVERSION = 6 | ||
| LIBYANG2_FULLVERSION = $(LIBYANG2_VERSION)-$(LIBYANG2_SUBVERSION) | ||
|
|
||
| export LIBYANG2_VERSION_BASE | ||
| export LIBYANG2_VERSION | ||
| export LIBYANG2_SUBVERSION | ||
| export LIBYANG2_FULLVERSION | ||
|
|
||
| LIBYANG2 = libyang2_$(LIBYANG2_FULLVERSION)_$(CONFIGURED_ARCH).deb | ||
| $(LIBYANG2)_SRC_PATH = $(SRC_PATH)/libyang2 | ||
| SONIC_MAKE_DEBS += $(LIBYANG2) | ||
|
|
||
| LIBYANG2_DEV = libyang2-dev_$(LIBYANG2_FULLVERSION)_$(CONFIGURED_ARCH).deb | ||
| $(eval $(call add_derived_package,$(LIBYANG2),$(LIBYANG2_DEV))) | ||
|
|
||
| LIBYANG2_DBG = libyang2-dbgsym_$(LIBYANG2_FULLVERSION)_$(CONFIGURED_ARCH).deb | ||
| $(eval $(call add_derived_package,$(LIBYANG2),$(LIBYANG2_DBG))) | ||
|
|
||
| LIBYANG2_CPP = libyang2-cpp1_$(LIBYANG2_FULLVERSION)_$(CONFIGURED_ARCH).deb | ||
| $(LIBYANG2_CPP)_DEPENDS += $(LIBYANG2) | ||
| $(eval $(call add_derived_package,$(LIBYANG2),$(LIBYANG2_CPP))) | ||
|
|
||
| LIBYANG2_CPP_DEV = libyang2-cpp-dev_$(LIBYANG2_FULLVERSION)_$(CONFIGURED_ARCH).deb | ||
| $(eval $(call add_derived_package,$(LIBYANG2),$(LIBYANG2_CPP_DEV))) | ||
|
|
||
| LIBYANG2_CPP_DBG = libyang2-cpp1-dbgsym_$(LIBYANG2_FULLVERSION)_$(CONFIGURED_ARCH).deb | ||
| $(eval $(call add_derived_package,$(LIBYANG2),$(LIBYANG2_CPP_DBG))) | ||
|
|
||
| YANG_TOOLS = yang-tools_$(LIBYANG2_FULLVERSION)_all.deb | ||
| $(YANG_TOOLS)_DEPENDS += $(LIBYANG2) | ||
| $(eval $(call add_derived_package,$(LIBYANG2),$(YANG_TOOLS))) | ||
|
|
||
| LIBYANG2_TOOLS = libyang2-tools_$(LIBYANG2_FULLVERSION)_$(CONFIGURED_ARCH).deb | ||
| $(eval $(call add_derived_package,$(LIBYANG2),$(LIBYANG2_TOOLS))) | ||
|
|
||
| LIBYANG2_TOOLS_DBG = libyang2-tools-dbgsym_$(LIBYANG2_FULLVERSION)_$(CONFIGURED_ARCH).deb | ||
| $(eval $(call add_derived_package,$(LIBYANG2),$(LIBYANG2_TOOLS_DBG))) | ||
|
|
||
| export LIBYANG2 LIBYANG2_DBG LIBYANG2_DEV LIBYANG2_CPP LIBYANG2_CPP_DEV LIBYANG2_CPP_DBG YANG_TOOLS LIBYANG2_TOOLS LIBYANG2_TOOLS_DBG | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -90,7 +90,7 @@ RUN apt-get update && apt-get install -y \ | |
| imagemagick \ | ||
| ghostscript \ | ||
| groff \ | ||
| libpcre2-dev \ | ||
|
There was a problem hiding this comment. libpcre2 is required by libyang2 when building buster. This dependency error is not seen in bullseye build, though. That's why the change only in buster. dpkg-source: info: extracting libyang2 in libyang2-2.0.112 There was a problem hiding this comment. Btw, libpcre3 is already present at L#150 in the same file. |
||
| gawk \ | ||
| chrpath \ | ||
| # For frr build | ||
|
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| * | ||
| !.gitignore | ||
| !Makefile |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
| .ONESHELL: | ||
| SHELL = /bin/bash | ||
| .SHELLFLAGS += -e | ||
|
|
||
| LIBYANG_URL = https://sonicstorage.blob.core.windows.net/debian/pool/main/liby/libyang | ||
|
|
||
| DSC_FILE = libyang2_$(LIBYANG2_FULLVERSION).dsc | ||
| ORIG_FILE = libyang2_$(LIBYANG2_VERSION).orig.tar.gz | ||
| DEBIAN_FILE = libyang2_$(LIBYANG2_FULLVERSION).debian.tar.xz | ||
|
|
||
| DSC_FILE_URL = $(LIBYANG_URL)/$(DSC_FILE) | ||
| ORIG_FILE_URL = $(LIBYANG_URL)/$(ORIG_FILE) | ||
| DEBIAN_FILE_URL = $(LIBYANG_URL)/$(DEBIAN_FILE) | ||
|
|
||
| MAIN_TARGET = $(LIBYANG2) | ||
| DERIVED_TARGETS = $(LIBYANG2_DEV) $(LIBYANG2_DBG) $(LIBYANG2_TOOLS) $(LIBYANG2_TOOLS_DBG) | ||
|
|
||
| $(addprefix $(DEST)/, $(MAIN_TARGET)): $(DEST)/% : | ||
| # Obtaining the libyang | ||
| rm -fr ./libyang2-$(LIBYANG2_VERSION) | ||
|
|
||
| # download debian libyang | ||
| wget -NO "$(DSC_FILE)" $(DSC_FILE_URL) | ||
| wget -NO "$(ORIG_FILE)" $(ORIG_FILE_URL) | ||
| wget -NO "$(DEBIAN_FILE)" $(DEBIAN_FILE_URL) | ||
| dpkg-source -x libyang2_$(LIBYANG2_FULLVERSION).dsc | ||
|
|
||
| pushd libyang2-$(LIBYANG2_VERSION) | ||
| #sed -i 's/set(LIBYANG_MAJOR_SOVERSION 1)/set(LIBYANG_MAJOR_SOVERSION 2)/' CMakeLists.txt | ||
| #sed -i 's/libyang2/libyang2/' debian/libyang2.install | ||
| # Enable large file support for 32-bit arch | ||
| echo 'add_definitions(-D_FILE_OFFSET_BITS=64)' >> CMakeLists.txt | ||
|
|
||
| dpkg-buildpackage -rfakeroot -b -us -uc -j$(SONIC_CONFIG_MAKE_JOBS) --admindir $(SONIC_DPKG_ADMINDIR) | ||
| popd | ||
|
|
||
| # Move the newly-built .deb packages to the destination directory | ||
| mv $* $(DERIVED_TARGETS) $(DEST)/ | ||
|
|
||
| $(addprefix $(DEST)/, $(DERIVED_TARGETS)): $(DEST)/% : $(DEST)/$(MAIN_TARGET) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,25 +1,25 @@ | ||
| From 15274a197e4d3b7ac0f11ca63a5274f1168739f5 Mon Sep 17 00:00:00 2001 | ||
| From: Pavel Shirshov <pavelsh@microsoft.com> | ||
| Date: Mon, 16 Nov 2020 18:11:47 -0800 | ||
| Subject: [PATCH 2/8] Reduce severity of 'Vty connected from' message | ||
|
|
||
| --- | ||
| lib/vty.c | 2 +- | ||
| 1 file changed, 1 insertion(+), 1 deletion(-) | ||
|
|
||
| diff --git a/lib/vty.c b/lib/vty.c | ||
| index 8eaf13619..7edeb5538 100644 | ||
| --- a/lib/vty.c | ||
| +++ b/lib/vty.c | ||
| @@ -1814,7 +1814,7 @@ static int vty_accept(struct thread *thread) | ||
| zlog_info("can't set sockopt to vty_sock : %s", | ||
| safe_strerror(errno)); | ||
|
|
||
| - zlog_info("Vty connection from %pSU", &su); | ||
| + zlog_debug("Vty connection from %pSU", &su); | ||
|
|
||
| vty_create(vty_sock, &su); | ||
|
|
||
| -- | ||
| 2.12.2 | ||
|
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,25 +1,25 @@ | ||
| From 39bb40dc4bad4462e4ae9c98580d75fa2c92e032 Mon Sep 17 00:00:00 2001 | ||
| From: Pavel Shirshov <pavelsh@microsoft.com> | ||
| Date: Mon, 16 Nov 2020 18:29:46 -0800 | ||
| Subject: [PATCH 3/8] Use vrf_id for vrf, not tabled_id | ||
|
|
||
| --- | ||
| zebra/zebra_fpm_netlink.c | 2 +- | ||
| 1 file changed, 1 insertion(+), 1 deletion(-) | ||
|
|
||
| diff --git a/zebra/zebra_fpm_netlink.c b/zebra/zebra_fpm_netlink.c | ||
| index ec22c5dd4..aad0156b3 100644 | ||
| --- a/zebra/zebra_fpm_netlink.c | ||
| +++ b/zebra/zebra_fpm_netlink.c | ||
| @@ -287,7 +287,7 @@ static int netlink_route_info_fill(struct netlink_route_info *ri, int cmd, | ||
| ri->nlmsg_pid = zvrf->zns->netlink_dplane_out.snl.nl_pid; | ||
|
|
||
| ri->nlmsg_type = cmd; | ||
| - ri->rtm_table = table_info->table_id; | ||
| + ri->rtm_table = zvrf_id(rib_dest_vrf(dest)); | ||
| ri->rtm_protocol = RTPROT_UNSPEC; | ||
|
|
||
| /* | ||
| -- | ||
| 2.12.2 | ||
|
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems this deb package didn't exist