From 82b6b3d6916743fb6c5eaf9e4a686ffbd75b550c Mon Sep 17 00:00:00 2001
From: DavidZagury <32644413+DavidZagury@users.noreply.github.com>
Date: Tue, 6 Jul 2021 13:00:34 +0300
Subject: [PATCH] [Mellanox] Remove mstdump from  Mellanoxs collect dump script
 (#1706)

- What I did
Remove redundant mellanox dumps now that Mellanox SAI is taking care of that.

- How I did it
Deleted the part related to mstdumps from the generate_script

- How to verify it
Run 'show techsupport' see we no have mstdump folder but the actual dump available under the saisdk dump
---
 scripts/generate_dump | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/scripts/generate_dump b/scripts/generate_dump
index 474ee9920e..925c9bf632 100755
--- a/scripts/generate_dump
+++ b/scripts/generate_dump
@@ -772,13 +772,6 @@ collect_mellanox() {
 
     ${CMD_PREFIX}rm -rf $sai_dump_folder
     ${CMD_PREFIX}docker exec -it syncd rm -rf $sai_dump_folder
-
-    local mst_dump_filename="/tmp/mstdump"
-    local max_dump_count="3"
-    for i in $(seq 1 $max_dump_count); do
-        ${CMD_PREFIX}/usr/bin/mstdump /dev/mst/mt*conf0 > "${mst_dump_filename}${i}"
-        save_file "${mst_dump_filename}${i}" mstdump true
-    done
     
     # Save SDK error dumps
     local sdk_dump_path=`${CMD_PREFIX}docker exec syncd cat /tmp/sai.profile|grep "SAI_DUMP_STORE_PATH"|cut -d = -f2`