Skip to content

Commit

Permalink
[generate_dump] Revert generate_dump optimization PR's #2599", add fi…
Browse files Browse the repository at this point in the history
…xes for empty /dump forder and symbolic links (#2710)

- What I did
3442815 Revert "Revert "Optimize the execution time of the 'show techsupport' script to 5-10%, (#2504)""
c3bd01f Revert "Revert "[generate_dump] Optimize the execution time of 'show techsupport' CLI by parallel function execution (#2512)""

Add a fix for the empty /dump folder inside the final tar archive generated by the show techsupport CLI command.
Add a fix to not follow the symbolic links to avoid duplicate files inside the final tar archive generated by the show techsupport CLI command.

- How I did it
Modify the scripts/generate_dump script.

- How to verify it
1. Manual verification
do the show techsupport CLI command and save output original.tar.gz (with original generate_dump script)
do the show techsupport CLI command and save output fixes.tar.gz (with the generate_dump script modified by this PR)
unpack both archives original.tar.gz and fixes.tar.gz
compare both directories with ncdu & diff --brief --recursive original fixes Linux utilities
2. Run the community tests
sonic-mgmt/tests/show_techsupport

Signed-off-by: vadymhlushko-mlnx <vadymh@nvidia.com>
Co-authored-by: StormLiangMS <89824293+StormLiangMS@users.noreply.github.com>
  • Loading branch information
vadymhlushko-mlnx and StormLiangMS authored Mar 16, 2023
1 parent 8c7ddf5 commit 325775d
Showing 1 changed file with 145 additions and 132 deletions.
Loading

0 comments on commit 325775d

Please sign in to comment.