Skip to content
New issue

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

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

Already on GitHub? # to your account

[Mellanox] Install MFT packages on syncd container #7844

Merged
merged 1 commit into from
Jun 14, 2021

Conversation

DavidZagury
Copy link
Contributor

@DavidZagury DavidZagury commented Jun 10, 2021

Why I did it

To have access to MFT tools in the Syncd container on Mellanox switches due to SAI dump API implementation enhancements

How I did it

Add MFT to the dependencies of docker-syncd-mlnx

How to verify it

Verify that MFT tools are presents on Syncd in a Mellanox switch

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

@liat-grozovik liat-grozovik changed the title [Mellanox] Install MFT packages on Syncd container [Mellanox] Install MFT packages on syncd container Jun 13, 2021
@lguohan lguohan merged commit cedc211 into sonic-net:master Jun 14, 2021
qiluo-msft pushed a commit that referenced this pull request Jun 17, 2021
To have access to MFT tools in the Syncd container on Mellanox switches due to SAI dump API implementation enhancements
Junchao-Mellanox pushed a commit to Junchao-Mellanox/sonic-buildimage that referenced this pull request Jun 24, 2021
To have access to MFT tools in the Syncd container on Mellanox switches due to SAI dump API implementation enhancements
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
To have access to MFT tools in the Syncd container on Mellanox switches due to SAI dump API implementation enhancements
@DavidZagury DavidZagury deleted the mlxtrace branch December 8, 2021 09:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants