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

Change to use rvtysh when calling the show commands #1646

Merged
merged 1 commit into from
Jun 3, 2021

Conversation

xumia
Copy link
Collaborator

@xumia xumia commented May 31, 2021

What I did

Change to use rvtysh when calling the show commands
Merge code from master and fix code conflict, see #1572

How I did it

How to verify it

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

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

@xumia xumia requested a review from qiluo-msft May 31, 2021 07:09
@qiluo-msft
Copy link
Contributor

Can we enable the build checker and let this PR pass?

@xumia
Copy link
Collaborator Author

xumia commented May 31, 2021

Can we enable the build checker and let this PR pass?

I have sent PR #1649 to enable the branch 202012 PR checkers, but it has dependency on PR sonic-net/sonic-swss#1690 to merge the branch 202012.
How about the other branches?

@qiluo-msft qiluo-msft merged commit 43fff88 into sonic-net:202012 Jun 3, 2021
qiluo-msft added a commit that referenced this pull request Jun 10, 2021
qiluo-msft pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Jun 11, 2021
Advance submodule head for sonic-utilities

b894c5b5 Fix build test failure caused by error module name (sonic-net/sonic-utilities#1662)
5a7c06a0 [config]][tacacs+] Change tacacs+ minimum timeout value base on spec (sonic-net/sonic-utilities#1631)
080a689c [202012] [db_migrator] fix old 1911 feature config migration to a new one. (sonic-net/sonic-utilities#1636)
43fff88c Change to use rvtysh when calling the show commands (sonic-net/sonic-utilities#1646)
88a823f0 [db_migrator][Mellanox] Update Mellanox buffer migrator with 2km-cable supported (sonic-net/sonic-utilities#1564)
d096ff78 [config]Static routes to config_db (1534)
a68d8d09 route_check: Updates  (sonic-net/sonic-utilities#1645)
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.

2 participants