Skip to content

[build]Fix FRR build cache issue#62

Closed
dgsudharsan wants to merge 1 commit intomasterfrom
frr_cache_fix
Closed

[build]Fix FRR build cache issue#62
dgsudharsan wants to merge 1 commit intomasterfrom
frr_cache_fix

Conversation

@dgsudharsan
Copy link
Owner

Why I did it

When FRR is built with Cache enabled, the build failed with the following error logs

[2023-09-20T15:17:00.273Z] fatal: Unable to hash src/sonic-frr/frr/tests/topotests/grpc_basic/lib
[2023-09-20T15:17:00.273Z] fatal: Unable to hash src/sonic-frr/frr/tests/topotests/ospfapi/lib
[2023-09-20T15:17:00.273Z] make: *** [Makefile.cache:528: target/debs/bullseye/frr_8.5.1-sonic-0_amd64.deb.smdep] Error 123
[2023-09-20T15:17:00.273Z] make: *** Waiting for unfinished jobs....
Work item tracking
  • Microsoft ADO (number only):

How I did it

Currently symlinks are excluded in hardcoded fashion. With FRR upgrades new symlinks might get introduced. To overcome it modified the way in which symlinks are excluded by finding symlinks using find command

How to verify it

Build FRR with cache enabled

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

dgsudharsan pushed a commit that referenced this pull request Jan 15, 2025
…ly (sonic-net#21230)

#### Why I did it
src/dhcprelay
```
* b242307 - (HEAD -> master, origin/master, origin/HEAD) Revert "Fail build when test fail (#53)" (#62) (19 hours ago) [Yaqiang Zhu]
* dbaee8b - Fail build when test fail (#53) (6 days ago) [Yaqiang Zhu]
* ef7b972 - [Build] Update to build bookworm debian package (#55) (8 days ago) [Yaqiang Zhu]
* 011066a - Add wait and check ip address check when dhcp6relay init (#52) (11 days ago) [Yaqiang Zhu]
```
#### How I did it
#### How to verify it
#### Description for the changelog
dgsudharsan pushed a commit that referenced this pull request Mar 17, 2025
… automatically (sonic-net#852)

#### Why I did it
src/sonic-platform-common
```
* c20c319 - (HEAD -> 202412, origin/202412) [code sync] Merge code from sonic-net/sonic-platform-common:202411 to 202412 (#62) (21 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
dgsudharsan pushed a commit that referenced this pull request Mar 17, 2025
…lly (sonic-net#907)

#### Why I did it
src/sonic-swss
```
* 6639e3a - (HEAD -> 202412, origin/202412) [code sync] Merge code from sonic-net/sonic-swss:202411 to 202412 (#62) (22 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
dgsudharsan pushed a commit that referenced this pull request Apr 30, 2025
…ly (sonic-net#22416)

#### Why I did it
src/sonic-stp
```
* a80676a - (HEAD -> master, origin/master, origin/HEAD) MSTP utility APIs (#62) (2 days ago) [Divya Kumaran Chandralekha]
* 2c3eccc - MSTP extern API declaration (#63) (2 days ago) [Divya Kumaran Chandralekha]
```
#### How I did it
#### How to verify it
#### Description for the changelog
dgsudharsan pushed a commit that referenced this pull request Jun 14, 2025
…ically (sonic-net#22863)

#### Why I did it
src/sonic-dash-ha
```
* 000fa6b - (HEAD -> master, origin/master, origin/HEAD) swbus-edge support of reconnecting to swbusd (#62) (10 hours ago) [yue-fred-gao]
```
#### How I did it
#### How to verify it
#### Description for the changelog
dgsudharsan pushed a commit that referenced this pull request Jul 1, 2025
…D automatically (sonic-net#1283)

#### Why I did it
src/sonic-sairedis
```
* 0ae8494 - (HEAD -> 202412, origin/HEAD, origin/202412) [unittest ]Conditionally enable TestServerSaiMockChannel (#67) (37 minutes ago) [mssonicbld]
* 12b21f3 - [syncd] Check whether stat_st and tam_telemetry is defined (sonic-net#1590) (#65) (5 days ago) [Ze Gan]
* 701a114 - [TAM]: Fix TAM notification and vslib implementation (sonic-net#1606) (#62) (5 days ago) [Ze Gan]
* 6b97f74 - [SAI]: Update SAI to support QUERY_STATS_ST_CAPABILITY and TAM (sonic-net#1573) (#63) (5 days ago) [Ze Gan]
```
#### How I did it
#### How to verify it
#### Description for the changelog
dgsudharsan pushed a commit that referenced this pull request Dec 16, 2025
…HEAD automatically (sonic-net#1599)

#### Why I did it
src/sonic-swss-common
```
* 66ac80e - (HEAD -> 202412, origin/HEAD, origin/202412) Add c-api/Rust wrappers for ConfigDBConnector and EventPublisher (#66) (13 hours ago) [mssonicbld]
* 9e5b094 - Add test case to verify zmq reconnect behaviour (#63) (17 hours ago) [mssonicbld]
* b4a3cc6 - Ignore nonexistent include in database_global.json (#64) (17 hours ago) [mssonicbld]
* 4f576db - Move SonicDbTable to sonic-dash-ha. (#65) (17 hours ago) [mssonicbld]
* fd28e9d - Move swss-common/swss-common-testing crates from https://github.com/sonic-net/sonic-dash-ha (#62) (17 hours ago) [mssonicbld]
* 726586c - SWSSDBConnector_new_keyed to c-api (#61) (19 hours ago) [mssonicbld]
* 16a2234 - c-api: README.md (#60) (19 hours ago) [mssonicbld]
* b501364 - Add swss::Logger to c-api (#59) (23 hours ago) [mssonicbld]
* c51425c - C API Exceptions (#58) (24 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
dgsudharsan pushed a commit that referenced this pull request Feb 25, 2026
…tically (sonic-net#25662)

#### Why I did it
src/sonic-dash-api
```
* 18a29c1 - (HEAD -> master, origin/master, origin/HEAD) Add step to compile Trixie in the CI builds (#64) (56 seconds ago) [Saikrishna Arcot]
* 99a835c - Move route rule priority to key (#62) (3 hours ago) [Lawrence Lee]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants