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

[sudoers] add /usr/local/bin/storyteller to READ_ONLY_CMDS #13422

Merged
merged 2 commits into from
Jan 27, 2023

Conversation

zjswhhh
Copy link
Contributor

@zjswhhh zjswhhh commented Jan 18, 2023

Adding /usr/local/bin/storyteller to READ_ONLY_CMDS. So no write access or prompt for password is needed to run storyteller.

Tested on 202205 clusters, user who didn't request write access was able to grep log using storyteller.

sign-off: Jing Zhang zhangjing@microsoft.com

@zjswhhh
Copy link
Contributor Author

zjswhhh commented Jan 27, 2023

Hi @yxieca - please help merge. Thanks!

@yxieca yxieca merged commit dabb31c into sonic-net:master Jan 27, 2023
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Jan 27, 2023
…-net#13422)

Adding /usr/local/bin/storyteller to READ_ONLY_CMDS. So no write access or prompt for password is needed to run storyteller.

Tested on 202205 clusters, user who didn't request write access was able to grep log using storyteller.

sign-off: Jing Zhang zhangjing@microsoft.com
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202205: #13530

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Feb 5, 2023
…-net#13422)

Adding /usr/local/bin/storyteller to READ_ONLY_CMDS. So no write access or prompt for password is needed to run storyteller.

Tested on 202205 clusters, user who didn't request write access was able to grep log using storyteller.

sign-off: Jing Zhang zhangjing@microsoft.com
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202211: #13666

mssonicbld pushed a commit that referenced this pull request Feb 7, 2023
Adding /usr/local/bin/storyteller to READ_ONLY_CMDS. So no write access or prompt for password is needed to run storyteller.

Tested on 202205 clusters, user who didn't request write access was able to grep log using storyteller.

sign-off: Jing Zhang zhangjing@microsoft.com
maipbui added a commit to maipbui/sonic-buildimage that referenced this pull request Jul 18, 2024
maipbui added a commit to maipbui/sonic-buildimage that referenced this pull request Jul 18, 2024
maipbui added a commit to maipbui/sonic-buildimage that referenced this pull request Jul 18, 2024
maipbui added a commit to maipbui/sonic-buildimage that referenced this pull request Jul 18, 2024
maipbui added a commit to maipbui/sonic-buildimage that referenced this pull request Jul 18, 2024
yxieca pushed a commit that referenced this pull request Jul 18, 2024
yxieca pushed a commit that referenced this pull request Jul 18, 2024
yxieca pushed a commit that referenced this pull request Jul 18, 2024
yxieca pushed a commit that referenced this pull request Jul 18, 2024
yxieca pushed a commit that referenced this pull request Jul 18, 2024
arun1355492 pushed a commit to arun1355492/sonic-buildimage that referenced this pull request Jul 26, 2024
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Aug 1, 2024
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.

4 participants