Skip to content

Upgrade libasan to version 6 in docker-syncd-mlnx to align with bullseye libasan#10886

Merged
saiarcot895 merged 1 commit intosonic-net:masterfrom
alexrallen:fix_sync_asan_version
May 27, 2022
Merged

Upgrade libasan to version 6 in docker-syncd-mlnx to align with bullseye libasan#10886
saiarcot895 merged 1 commit intosonic-net:masterfrom
alexrallen:fix_sync_asan_version

Conversation

@alexrallen
Copy link
Copy Markdown
Contributor

Why I did it

syncd is linking to libasan v6 during build after the bullseye upgrade (#10580) and libasan v5 is installed in the syncd container for the mellanox platform which is causing runtime errors.

How I did it

Install libasan6 on docker-syncd-mlnx

How to verify it

Build sonic with ENABLE_ASAN=y and verify syncd starts successfully.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Upgrade libasan to version 6 in docker-syncd-mlnx to align with bullseye libasan

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

1200

@alexrallen alexrallen requested a review from lguohan as a code owner May 19, 2022 18:29
@liat-grozovik
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@saiarcot895
Copy link
Copy Markdown
Contributor

Just to check, what's the behavior if libasan6 is not explicitly installed and ASAN is enabled? In theory, after the deb package gets installed, apt install -f should install libasan6 automatically, and so explicitly installing it here shouldn't be needed.

@alexrallen
Copy link
Copy Markdown
Contributor Author

I could check, I tend to agree with your assessment that it should install this dependency automatically based on the macros we are using,

@liat-grozovik
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan
Copy link
Copy Markdown
Collaborator

@saiarcot895 Can we merge this?

@saiarcot895 saiarcot895 merged commit 71c868f into sonic-net:master May 27, 2022
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Jun 20, 2022
…anch

Related work items: #52, #71, #73, #75, #77, sonic-net#1306, sonic-net#1588, sonic-net#1991, sonic-net#2031, sonic-net#2040, sonic-net#2053, sonic-net#2066, sonic-net#2069, sonic-net#2087, sonic-net#2107, sonic-net#2110, sonic-net#2112, sonic-net#2113, sonic-net#2117, sonic-net#2124, sonic-net#2125, sonic-net#2126, sonic-net#2128, sonic-net#2130, sonic-net#2131, sonic-net#2132, sonic-net#2133, sonic-net#2134, sonic-net#2135, sonic-net#2136, sonic-net#2137, sonic-net#2138, sonic-net#2139, sonic-net#2140, sonic-net#2143, sonic-net#2158, sonic-net#2161, sonic-net#2233, sonic-net#2243, sonic-net#2250, sonic-net#2254, sonic-net#2260, sonic-net#2261, sonic-net#2267, sonic-net#2278, sonic-net#2282, sonic-net#2285, sonic-net#2288, sonic-net#2289, sonic-net#2292, sonic-net#2294, sonic-net#8887, sonic-net#9279, sonic-net#9390, sonic-net#9511, sonic-net#9700, sonic-net#10025, sonic-net#10322, sonic-net#10479, sonic-net#10484, sonic-net#10493, sonic-net#10500, sonic-net#10580, sonic-net#10595, sonic-net#10628, sonic-net#10634, sonic-net#10635, sonic-net#10644, sonic-net#10670, sonic-net#10691, sonic-net#10716, sonic-net#10731, sonic-net#10750, sonic-net#10751, sonic-net#10752, sonic-net#10761, sonic-net#10769, sonic-net#10775, sonic-net#10776, sonic-net#10779, sonic-net#10786, sonic-net#10792, sonic-net#10793, sonic-net#10800, sonic-net#10806, sonic-net#10826, sonic-net#10839, sonic-net#10840, sonic-net#10842, sonic-net#10844, sonic-net#10847, sonic-net#10849, sonic-net#10852, sonic-net#10865, sonic-net#10872, sonic-net#10877, sonic-net#10886, sonic-net#10889, sonic-net#10903, sonic-net#10904, sonic-net#10905, sonic-net#10913, sonic-net#10914, sonic-net#10916, sonic-net#10919, sonic-net#10925, sonic-net#10926, sonic-net#10929, sonic-net#10933, sonic-net#10934, sonic-net#10937, sonic-net#10941, sonic-net#10947, sonic-net#10952, sonic-net#10953, sonic-net#10957, sonic-net#10959, sonic-net#10971, sonic-net#10972, sonic-net#10980
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.

4 participants