Skip to content

[Mellanox] Add FW dump with new SAI implementation and remove mst dump#1298

Merged
abdosi merged 1 commit intosonic-net:201911from
shlomibitton:shlomi_fw_dump_techsupport
Dec 16, 2020
Merged

[Mellanox] Add FW dump with new SAI implementation and remove mst dump#1298
abdosi merged 1 commit intosonic-net:201911from
shlomibitton:shlomi_fw_dump_techsupport

Conversation

@shlomibitton
Copy link
Contributor

@shlomibitton shlomibitton commented Dec 9, 2020

Signed-off-by: Shlomi Bitton shlomibi@nvidia.com

- What I did
Mellanox SAI implementation of the sai dump is now extracting more information which previously provided by direct call to a Mellanox FW tool which is not longer required.

The new files created will be taken along with the new dump.
Remove MST dump as it is not required anymore.

- How I did it
Add commands to copy FW dump output into techsupport output file, inside "sai_sdk_dump" directory.
Remove commands for MST dump.

- How to verify it
Run "show techsupport" with MLNX SAI 1.18.0

- Previous command output (if the output of a command-line utility has changed)

├── mstdump
│   ├── mstdump1.gz
│   ├── mstdump2.gz
│   └── mstdump3.gz

- New command output (if the output of a command-line utility has changed)

└── sai_sdk_dump
├── sai_sdk_dump_12_09_2020_03_17_PM.gz
├── sdkdump_ext_cr_001-09_12_2020-15_17_57.udmp.gz
├── sdkdump_ext_cr_001-09_12_2020-15_17_58.udmp.gz
├── sdkdump_ext_cr_001-09_12_2020-15_18_00.udmp.gz
├── sdkdump_ext_meta_001-09_12_2020-15_17_57.gz
├── sdkdump_ext_meta_001-09_12_2020-15_17_58.gz
└── sdkdump_ext_meta_001-09_12_2020-15_18_00.gz

@liat-grozovik
Copy link
Collaborator

@abdosi please approve so we can take to 201911 and update the sub moduel.

Remove mst dump

Signed-off-by: Shlomi Bitton <shlomibi@nvidia.com>
@abdosi
Copy link
Contributor

abdosi commented Dec 15, 2020

@shlomibitton please resolve the conflict.

@abdosi abdosi merged commit 12f03b1 into sonic-net:201911 Dec 16, 2020
deran1980 added a commit to deran1980/sonic-utilities that referenced this pull request Feb 3, 2021
deran1980 added a commit to deran1980/sonic-utilities that referenced this pull request Feb 7, 2021
liat-grozovik pushed a commit that referenced this pull request Feb 7, 2021
This reverts commit 12f03b1.

**- What I did**
revert changes to call mstdump and replace with old call to sdkdump

**- How I did it**
reverting a previous commit [Mellanox] Add FW dump with new SAI implementation and remove mst dump Azure#1298

**- How to verify it**
run techsupport
@shlomibitton shlomibitton deleted the shlomi_fw_dump_techsupport branch March 24, 2021 20:41
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
fd3e017 (HEAD -> 201911, origin/201911) [Multi-asic] Enhanced Feature Table configuration for multi-asic platforms (sonic-net#1152)
12f03b1 Add FW dump with new SAI implementation (sonic-net#1298)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
603ac53 (HEAD -> 201911, origin/201911) Advertise ipv6 link local address (sonic-net#1402)
9b0680c [acl_loader] Fix default DENY rule for V6 dataplane ACLs (sonic-net#1281)
25e64ce [show] Fix `show ip bgp sum` (sonic-net#1194)
4709da0 Revert "Add FW dump with new SAI implementation (sonic-net#1298)" (sonic-net#1408)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
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