Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Mellanox] [202211] Replace iproute2 supplied by SDK to iproute2 downloaded from Debian repository (#14726) #14724

Merged

Conversation

liorghub
Copy link
Contributor

@liorghub liorghub commented Apr 19, 2023

Backport of #14726

Why I did it

Mellanox syncd container will be based on Debian iproute2 plus patches instead of Nvidia internal version of iproute2

How I did it

Download iproute2 from Debian repository, apply patches and compile to create a new target.
The target is then deployed in syncd container of Mellanox switches only.
The new target is called IPROUTE2_MLNX.

How to verify it

Compile and load on switch, verify interfaces network devices created successfully.
Verify LLDP shows connections to neighbors.
Verify ping between 2 hosts over 2 router ports is successful.

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

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

Description for 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

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

@liorghub liorghub requested a review from lguohan as a code owner April 19, 2023 14:50
@liorghub liorghub closed this Apr 19, 2023
@liorghub liorghub reopened this Apr 19, 2023
@liorghub liorghub changed the title [syncd docker] Replace iproute2 supplied by SDK to iproute2 downloaded from debian repository [202211 syncd docker] Replace iproute2 supplied by SDK to iproute2 downloaded from debian repository Apr 20, 2023
@dprital dprital requested a review from dgsudharsan April 20, 2023 06:38
@liorghub liorghub changed the title [202211 syncd docker] Replace iproute2 supplied by SDK to iproute2 downloaded from debian repository [202211 Nvidia] Replace iproute2 supplied by SDK to iproute2 downloaded from debian repository Apr 20, 2023
@liorghub liorghub changed the title [202211 Nvidia] Replace iproute2 supplied by SDK to iproute2 downloaded from debian repository [Nvidia] [202211] Replace iproute2 supplied by SDK to iproute2 downloaded from debian repository Apr 20, 2023
@liat-grozovik liat-grozovik changed the title [Nvidia] [202211] Replace iproute2 supplied by SDK to iproute2 downloaded from debian repository [Mellanox] [202211] Replace iproute2 supplied by SDK to iproute2 downloaded from Debian repository Apr 30, 2023
@liat-grozovik liat-grozovik changed the title [Mellanox] [202211] Replace iproute2 supplied by SDK to iproute2 downloaded from Debian repository [Mellanox] [202211] Replace iproute2 supplied by SDK to iproute2 downloaded from Debian repository (#14726) Apr 30, 2023
@liat-grozovik
Copy link
Collaborator

@StormLiangMS FYI. Please merge.

@liat-grozovik liat-grozovik merged commit d7d8d77 into sonic-net:202211 May 2, 2023
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.

3 participants