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

Revert "Add FW dump with new SAI implementation (#1338)" #4

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

deran1980
Copy link
Owner

This reverts commit b10622e.

- 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 sonic-net#1298

- How to verify it
run techsupport

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

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

yxieca and others added 4 commits February 3, 2021 12:46
…#1388)

- What I did

Add log path parameter to allow storyteller to work on logs downloaded from sonic devices.
Improve reboot regex to capture more warm reboot related logs.

- How to verify it
Use storyteller against a sonic dump.
ignore the message printed on stderr

sonic:$ show ip bgp sum
usermod: Permission denied.
usermod: cannot lock /etc/passwd; try again later.
usermod: Permission denied.
usermod: cannot lock /etc/passwd; try again later.

Signed-off-by: Guohan Lu <lguohan@gmail.com>
As part of dual-tor features, kernel tunnel interface (tun0) is being created. The routes over this is intended for kernel forwarding and not expected to be installed in ASIC.

Ref PR - sonic-net/sonic-swss#1615
@deran1980 deran1980 force-pushed the revert_mstdump_calls_master branch from 2e437de to a4083cb Compare February 4, 2021 11:26
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.

5 participants