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

[broadcom][cel][udev] Update console usb devices permission #5840

Merged
merged 1 commit into from
Nov 9, 2020

Conversation

Blueve
Copy link
Contributor

@Blueve Blueve commented Nov 6, 2020

- Why I did it

As discuss in this PR #5438, we want to allow user to connect to remote device without root privilege need.

- How I did it

Update udev rule for console ttyUSB devices. Compare with add user to dialout group, this way minimalized the permission authorization.

- How to verify it

Tested new udev rule on physical SONiC DUT.

before

admin@sonic:~$ picocom /dev/C0-1 -b 9600
picocom v3.1

port is        : /dev/C0-1
flowcontrol    : none
baudrate is    : 9600
parity is      : none
databits are   : 8
stopbits are   : 1
escape is      : C-a
local echo is  : no
noinit is      : no
noreset is     : no
hangup is      : no
nolock is      : no
send_cmd is    : sz -vv
receive_cmd is : rz -vv -E
imap is        : 
omap is        : 
emap is        : crcrlf,delbs,
logfile is     : none
initstring     : none
exit_after is  : not set
exit is        : no


FATAL: cannot open /dev/C0-1: Permission denied

*after

admin@sonic:~$ picocom /dev/C0-2 -b 9600
picocom v3.1

port is        : /dev/C0-2
flowcontrol    : none
baudrate is    : 9600
parity is      : none
databits are   : 8
stopbits are   : 1
escape is      : C-a
local echo is  : no
noinit is      : no
noreset is     : no
hangup is      : no
nolock is      : no
send_cmd is    : sz -vv
receive_cmd is : rz -vv -E
imap is        : 
omap is        : 
emap is        : crcrlf,delbs,
logfile is     : none
initstring     : none
exit_after is  : not set
exit is        : no

Type [C-a] [C-h] to see available commands
Terminal ready

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@Blueve Blueve requested review from yxieca and lguohan November 6, 2020 06:55
@Blueve
Copy link
Contributor Author

Blueve commented Nov 8, 2020

retest vsimage please

@Blueve Blueve merged commit fb6af9d into sonic-net:master Nov 9, 2020
@Blueve Blueve deleted the dev/jika/udev branch November 9, 2020 00:54
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
…t#5840)

* Add MODE "0666" to all console usb tty

Signed-off-by: Jing Kan [email protected]
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