Skip to content

[Mellanox] Upgrade syncd, syncd-rpc and saiserver to bookworm#18651

Merged
liat-grozovik merged 13 commits intosonic-net:masterfrom
vivekrnv:bookworm_docker
Apr 25, 2024
Merged

[Mellanox] Upgrade syncd, syncd-rpc and saiserver to bookworm#18651
liat-grozovik merged 13 commits intosonic-net:masterfrom
vivekrnv:bookworm_docker

Conversation

@vivekrnv
Copy link
Copy Markdown
Contributor

Why I did it

Upgrade Mellanox (syncd/syncd-rpc/saiserver) containers to Bookworm

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Compile succesfully and run sonic-mgmt tests

Manual Verification and Regression tests

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)

Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
@vivekrnv vivekrnv requested a review from lguohan as a code owner April 10, 2024 23:03
@vivekrnv vivekrnv requested review from saiarcot895 and removed request for lguohan April 10, 2024 23:03
@vivekrnv vivekrnv requested a review from dgsudharsan April 10, 2024 23:03
@liat-grozovik liat-grozovik changed the title Upgrade mlnx (syncd/syncd-rpc/saiserver) to bookworm [Mellanox] Upgrade syncd, syncd-rpc and saiserver to bookworm Apr 14, 2024
@vivekrnv
Copy link
Copy Markdown
Contributor Author

@liat-grozovik Can you merge this PR?

@liat-grozovik liat-grozovik merged commit bca7cbe into sonic-net:master Apr 25, 2024
lguohan pushed a commit that referenced this pull request Dec 16, 2024
…n with bookworm libasan (#21134)

syncd is linking to libasan v8 during build after the bookwork upgrade #18651 but libasan v6 is installed in the syncd container for the mellanox platform which is causing runtime errors.

Signed-off-by: Andriy Yurkiv <ayurkiv@nvidia.com>
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Dec 18, 2024
…n with bookworm libasan (sonic-net#21134)

syncd is linking to libasan v8 during build after the bookwork upgrade sonic-net#18651 but libasan v6 is installed in the syncd container for the mellanox platform which is causing runtime errors.

Signed-off-by: Andriy Yurkiv <ayurkiv@nvidia.com>
bingwang-ms pushed a commit that referenced this pull request Dec 27, 2024
…n with bookworm libasan (#21134) (#21213)

syncd is linking to libasan v8 during build after the bookwork upgrade #18651 but libasan v6 is installed in the syncd container for the mellanox platform which is causing runtime errors.

Signed-off-by: Andriy Yurkiv <ayurkiv@nvidia.com>
Co-authored-by: Andriy Yurkiv <70649192+ayurkiv-nvda@users.noreply.github.com>
mssonicbld added a commit to mssonicbld/sonic-buildimage that referenced this pull request Jan 13, 2025
…orm libasan

<!--
     Please make sure you've read and understood our contributing guidelines:
     https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

     ** Make sure all your commits include a signature generated with `git commit -s` **

     If this is a bug fix, make sure your description includes "fixes #xxxx", or
     "closes #xxxx" or "resolves #xxxx"

     Please provide the following information:
-->

#### Why I did it
**syncd** is linking to libasan v8 during build after the bookwork upgrade sonic-net#18651 but libasan v6 is installed in the syncd container for the mellanox platform which is causing runtime errors.

##### Work item tracking
- Microsoft ADO **(number only)**:

#### How I did it
Install libasan8 on docker-syncd-mlnx

#### How to verify it
Build sonic with ENABLE_ASAN=y and verify syncd starts successfully.

<!--
If PR needs to be backported, then the PR must be tested against the base branch and the earliest backport release branch and provide tested image version on these two branches. For example, if the PR is requested for master, 202211 and 202012, then the requester needs to provide test results on master and 202012.
-->

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

<!--
- Note we only backport fixes to a release branch, *not* features!
- Please also provide a reason for the backporting below.
- e.g.
- [x] 202006
-->

- [ ] 201811
- [ ] 201911
- [ ] 202006
- [ ] 202012
- [ ] 202106
- [ ] 202111
- [ ] 202205
- [ ] 202211
- [ ] 202305
- [x] 202405

#### Tested branch (Please provide the tested image version)

<!--
- Please provide tested image version
- e.g.
- [x] 20201231.100
-->

- [ ] <!-- image version 1 -->
- [ ] <!-- image version 2 -->

#### Description for the changelog
<!--
Write a short (one line) summary that describes the changes in this
pull request for inclusion in the changelog:
-->

<!--
 Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.
-->

#### Link to config_db schema for YANG module changes
<!--
Provide a link to config_db schema for the table for which YANG model
is defined
Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md
-->

#### A picture of a cute animal (not mandatory but encouraged)
mssonicbld added a commit that referenced this pull request Jan 13, 2025
…orm libasan (#21404)

<!--
 Please make sure you've read and understood our contributing guidelines:
 https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

 failure_prs.log skip_prs.log Make sure all your commits include a signature generated with `git commit -s` **

 If this is a bug fix, make sure your description includes "fixes #xxxx", or
 "closes #xxxx" or "resolves #xxxx"

 Please provide the following information:
-->

#### Why I did it
**syncd** is linking to libasan v8 during build after the bookwork upgrade #18651 but libasan v6 is installed in the syncd container for the mellanox platform which is causing runtime errors.

##### Work item tracking
- Microsoft ADO **(number only)**:

#### How I did it
Install libasan8 on docker-syncd-mlnx

#### How to verify it
Build sonic with ENABLE_ASAN=y and verify syncd starts successfully.

<!--
If PR needs to be backported, then the PR must be tested against the base branch and the earliest backport release branch and provide tested image version on these two branches. For example, if the PR is requested for master, 202211 and 202012, then the requester needs to provide test results on master and 202012.
-->

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

<!--
- Note we only backport fixes to a release branch, *not* features!
- Please also provide a reason for the backporting below.
- e.g.
- [x] 202006
-->

- [ ] 201811
- [ ] 201911
- [ ] 202006
- [ ] 202012
- [ ] 202106
- [ ] 202111
- [ ] 202205
- [ ] 202211
- [ ] 202305
- [x] 202405

#### Tested branch (Please provide the tested image version)

<!--
- Please provide tested image version
- e.g.
- [x] 20201231.100
-->

- [ ] <!-- image version 1 -->
- [ ] <!-- image version 2 -->

#### Description for the changelog
<!--
Write a short (one line) summary that describes the changes in this
pull request for inclusion in the changelog:
-->

<!--
 Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.
-->

#### Link to config_db schema for YANG module changes
<!--
Provide a link to config_db schema for the table for which YANG model
is defined
Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md
-->

#### A picture of a cute animal (not mandatory but encouraged)
VladimirKuk pushed a commit to Marvell-switching/sonic-buildimage that referenced this pull request Jan 21, 2025
…n with bookworm libasan (sonic-net#21134)

syncd is linking to libasan v8 during build after the bookwork upgrade sonic-net#18651 but libasan v6 is installed in the syncd container for the mellanox platform which is causing runtime errors.

Signed-off-by: Andriy Yurkiv <ayurkiv@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants