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

[consutil] Add brief option to show line command #1176

Merged
merged 1 commit into from
Oct 20, 2020

Conversation

Blueve
Copy link
Contributor

@Blueve Blueve commented Oct 15, 2020

- What I did

Add --brief and -b option to show line command.
By default, show line command will render a table which includes all console lines in CONSOLE_PORT table and all ttyUSB* devices under \dev. We display ttyUSB* because we want to provide convenience to user when adding console line settings from scratch.
We add this optional option is for reduce noise for some user because the user are not allowed to connect a console line which is not present in CONFIG_DB.

- How I did it

Stop merge console line result into the result table when user specified --brief or -b option.

- How to verify it

Build python-wheel package and tested on physical DUT.

- 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)

admin@sonic:~$ sudo show line -b
  Line    Baud    PID    Start Time    Device
------  ------  -----  ------------  --------
     1  115000                        switch1
     2    9600                              -

@Blueve Blueve merged commit 510d0ad into sonic-net:master Oct 20, 2020
@Blueve Blueve deleted the dev/jika/show_brief branch October 20, 2020 00:16
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
[aclorch] Enable IN_PORTS ACL qualifier on MIRROR tables (sonic-net#1176)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants