[nvidia-bluefield] Add dependencies required for Nvidia-bluefield image compilation.#18649
Conversation
|
Related to #18466 |
| pandoc \ | ||
| valgrind \ | ||
| udev \ | ||
| inkscape \ |
There was a problem hiding this comment.
Why does the SDK require valgrind and inkscape for compilation?
There was a problem hiding this comment.
valgrind is required for OFED compilation
inkscape is required for DPDK compilation
There was a problem hiding this comment.
DPDK uses inkscape for the memory reports.
There was a problem hiding this comment.
OFED uses valgrind to provide built-in debugging capabilities.
sonic-slave-bookworm/Dockerfile.j2
Outdated
| python3-pyelftools \ | ||
| hwdata \ | ||
| libzip-dev \ | ||
| libzip4 |
There was a problem hiding this comment.
Does lipzip4 need to be listed out separately here? libzip-dev should bring in the library automatically.
There was a problem hiding this comment.
You are right. I removed lipzip4
oleksandrivantsiv
force-pushed
the
nvidia-bluefield-dependencies
branch
from
d50f45f to
a8eb96a
Compare
|
Please restart the pipeline. Mellanox build should pass now |
|
@oleksandrivantsiv is there any dependency to avoid the checker failures? if not, can you please review them? |
|
/azpw run |
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@saiarcot895 can you approve? |
5 participants
Why I did it
Add dependencies required for Nvidia-bluefield image compilation.
Work item tracking
How I did it
Extend a list of the DEB packages installed in the Bookworm slave docker container.
How to verify it
Compile Bookworm slave docker container.
Which release branch to backport (provide reason below if selected)
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)