srl-labs / containerlab

container-based networking labs
https://containerlab.dev
BSD 3-Clause "New" or "Revised" License
1.54k stars 263 forks source link

[enhancement] containerlab version requires right elevation #339

Closed azyablov closed 3 years ago

azyablov commented 3 years ago

Minor issue, but issue sudo to check version installed could be a potential security concern for some environments.

[admin@dct0 clab]$ containerlab version
containerlab requires sudo privileges to run!
hellt commented 3 years ago

Hey @azyablov, thanks for taking time raising an issue

version (and a few other cmds) will not require sudo in the next release, but deploy/destroy/inspect commands will require sudo privileges still.