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] Add mlxtrace to techsupport #15961

Conversation

vadymhlushko-mlnx
Copy link
Contributor

Why I did it

Added the fwtrace config files in order to be able to call the mlxstrace utility during the show techsupport dump.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Added fwtrace config files. Added path to these files to sai.profile for each mlnx device.

How to verify it

Execute the show techsupport command and check if mlxstrace output is in system dump.

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

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

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)

@vadymhlushko-mlnx
Copy link
Contributor Author

@vadymhlushko-mlnx vadymhlushko-mlnx changed the title [mlxtrace] Add fwtrace config files. Add path to these files to sai.profile for each mlnx device. [mlxtrace] Add mft-fwtrace-cfg.deb which contains fwtrace_cfg files. Add path to these files to sai.profile for each mlnx device. Jul 25, 2023
@vadymhlushko-mlnx
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@vadymhlushko-mlnx
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

No commit pushedDate could be found for PR 15961 in repo sonic-net/sonic-buildimage

…or the mlxtrace utility

Signed-off-by: vadymhlushko-mlnx <vadymh@nvidia.com>
@vadymhlushko-mlnx vadymhlushko-mlnx force-pushed the master-mft-fwtrace-config-files branch from db6fcb9 to cfe9cfa Compare July 26, 2023 12:50
@vadymhlushko-mlnx
Copy link
Contributor Author

@qiluo-msft could you please help to review and merge this PR and cherry-picks?

@vadymhlushko-mlnx
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@vadymhlushko-mlnx
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@liat-grozovik liat-grozovik changed the title [mlxtrace] Add mft-fwtrace-cfg.deb which contains fwtrace_cfg files. Add path to these files to sai.profile for each mlnx device. [Mellanox] Add mlxtrace to techsupport Aug 2, 2023
Copy link
Collaborator

@liat-grozovik liat-grozovik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please wait for my approval for this PR and the related ones avaialble for other branches

@@ -1 +1,2 @@
SAI_INIT_CONFIG_FILE=/usr/share/sonic/hwsku/sai_3800_2x10g_100x50g_12x100g.xml
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vadymhlushko-mlnx why dont we have on this SKU the other dump configuration? is it be design?
please compare with other SN3800 SKUs where it is.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most probably, someone forgets to add.
Because those variables are important, for the auto-techsupport feature

SAI_DUMP_STORE_PATH=/var/log/mellanox/sdk-dumps
SAI_DUMP_STORE_AMOUNT=10

I will add

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Aug 20, 2023
- Why I did it
Added the fwtrace config files in order to be able to call the mlxstrace utility during the show techsupport dump.

Work item tracking
Microsoft ADO (number only):

- How I did it
Added fwtrace config files. Added path to these files to sai.profile for each mlnx device.

- How to verify it
Execute the show techsupport command and check if mlxstrace output is in system dump.

Signed-off-by: vadymhlushko-mlnx <vadymh@nvidia.com>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202305: #16215

StormLiangMS pushed a commit that referenced this pull request Aug 20, 2023
…or the mlxtrace utility (#15960)

Backport of #15961

Why I did it
Added the fwtrace config files in order to be able to call mlxstrace utility during show techsupport dump.

Work item tracking
Microsoft ADO (number only):
How I did it
Added fwtrace config files. Added path to these files to sai.profile for each mlnx device.

How to verify it
Execute the show techsupport command and check if mlxstrace output is in system dump.
sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this pull request Sep 20, 2023
- Why I did it
Added the fwtrace config files in order to be able to call the mlxstrace utility during the show techsupport dump.

Work item tracking
Microsoft ADO (number only):

- How I did it
Added fwtrace config files. Added path to these files to sai.profile for each mlnx device.

- How to verify it
Execute the show techsupport command and check if mlxstrace output is in system dump.

Signed-off-by: vadymhlushko-mlnx <vadymh@nvidia.com>
volodymyrsamotiy added a commit to volodymyrsamotiy/sonic-buildimage that referenced this pull request Sep 21, 2023
yxieca pushed a commit that referenced this pull request Dec 18, 2023
* [mlxtrace] Add mft-fwtrace-cfg.deb which contains fwtrace_cfg files for the mlxtrace utility

Signed-off-by: vadymhlushko-mlnx <vadymh@nvidia.com>

* [mlxtrace] Remove mlxtrace support for SPC4

Signed-off-by: vadymhlushko-mlnx <vadymh@nvidia.com>

---------

Signed-off-by: vadymhlushko-mlnx <vadymh@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.

6 participants