Skip to content

[nvidia-bluefield] Differentiate cache based on compilation and download#129

Closed
vivekrnv wants to merge 1 commit intomasterfrom
fix_cache_nasa
Closed

[nvidia-bluefield] Differentiate cache based on compilation and download#129
vivekrnv wants to merge 1 commit intomasterfrom
fix_cache_nasa

Conversation

@vivekrnv
Copy link
Copy Markdown
Owner

@vivekrnv vivekrnv commented Jun 10, 2025

Why I did it

Image compilation fails when using the same cache tar for build from source and download from artifact.

Eg:
When building from sources, mlnx-tools is a derived package of mlnx-ofed-kernel-utils and the cache tar has both the packages saved

$ tar -tvf  mlnx-tools_25.01-0.2501060_arm64.deb-adc83b19e793491b1c6ea0f-ba387afe3cf420ba021052c.tgz
-rw-r--r-- vkarri/dip    69712 2025-06-11 02:24 target/debs/bookworm/mlnx-tools_25.01-0.2501060_arm64.deb
-rw-r--r-- vkarri/dip    27816 2025-06-11 02:25 target/debs/bookworm/mlnx-ofed-kernel-utils_25.01.OFED.25.01.0.6.0.1-1_arm64.deb
-rw-r--r-- vkarri/dip  2445612 2025-06-11 02:25 target/debs/bookworm/mlnx-ofed-kernel-dkms_25.01.OFED.25.01.0.6.0.1-1_all.deb
-rw-r--r-- vkarri/dip    73036 2025-06-11 02:25 target/debs/bookworm/mlnx-tools_25.01-0.2501060_arm64.deb.cached.log

When downloading from source and enabling caching would cause the cache only to have mlnx-tools package

$ tar -tvf mlnx-tools_25.01-0.2501060_arm64.deb-adc83b19e793491b1c6ea0f-27f1acbefcfdfd078f9632c.tgz
-rw-r--r-- vkarri/dip    69712 2025-06-11 02:11 target/debs/bookworm/mlnx-tools_25.01-0.2501060_arm64.deb
-rw-r--r-- vkarri/dip      997 2025-06-11 02:11 target/debs/bookworm/mlnx-tools_25.01-0.2501060_arm64.deb.cached.log

Now imagine the following steps

  1. Enable cache, Fetch from artifactory and build a sonic image
  2. Use the same cache and build from sonic image using source compilation strategy(mlnx-tools cache in the previous step is picked)
05:44:06  [ building ] [ target/debs/bookworm/mlnx-ofed-kernel-utils_25.01.OFED.25.01.0.6.0.1-1_arm64.deb ] 
05:44:06  [ finished ] [ target/debs/bookworm/mlnx-ofed-kernel-utils_25.01.OFED.25.01.0.6.0.1-1_arm64.deb ] 
05:44:10  [ building ] [ target/debs/bookworm/mlnx-ofed-kernel-utils_25.01.OFED.25.01.0.6.0.1-1_arm64.deb-install ] 
05:44:10  [ finished ] [ target/debs/bookworm/mlnx-ofed-kernel-utils_25.01.OFED.25.01.0.6.0.1-1_arm64.deb-install ] 
05:44:10  [ FAIL LOG START ] [ target/debs/bookworm/mlnx-ofed-kernel-utils_25.01.OFED.25.01.0.6.0.1-1_arm64.deb-install ]
05:44:10  Build start time: Tue Jun 10 02:44:09 UTC 2025
05:44:10  target/debs/bookworm/mlnx-ofed-kernel-utils_25.01.OFED.25.01.0.6.0.1-1_arm64.deb does not exist
05:44:10  [  FAIL LOG END  ] [ target/debs/bookworm/mlnx-ofed-kernel-utils_25.01.OFED.25.01.0.6.0.1-1_arm64.deb-install ]
05:44:10  make: *** [slave.mk:869: target/debs/bookworm/mlnx-ofed-kernel-utils_25.01.OFED.25.01.0.6.0.1-1_arm64.deb-install] Error 1
05:44:10  make: *** Waiting for unfinished jobs...

How I did it

Differentiate cache based on the compilation strategy

How to verify it

Verify if the above flow succeeds

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

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

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)

@vivekrnv vivekrnv closed this Jun 11, 2025
@vivekrnv vivekrnv deleted the fix_cache_nasa branch June 12, 2025 15:41
vivekrnv pushed a commit that referenced this pull request Nov 12, 2025
…ically (sonic-net#24435)

#### Why I did it
src/sonic-dash-ha
```
* 149774b - (HEAD -> master, origin/master, origin/HEAD) Add a PR template (#129) (3 days ago) [yue-fred-gao]
* 711a0be - Sort the output of show actor command (#128) (3 days ago) [yue-fred-gao]
* 15fbd5c - Fix format for protobuf fields in show command. (#127) (3 days ago) [dypet]
* f2d47b9 - Add wait for loopback script. (#126) (3 days ago) [dypet]
* 570db05 - Fix issue #118: use hostname to build service path (#122) (3 days ago) [yue-fred-gao]
```
#### 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