Skip to content

[201811] Improve sudo cat command for RO user. (#14428)#14438

Merged
lguohan merged 1 commit intosonic-net:201811from
liuh-80:dev/liuh/fix-cat-201811
Apr 12, 2023
Merged

[201811] Improve sudo cat command for RO user. (#14428)#14438
lguohan merged 1 commit intosonic-net:201811from
liuh-80:dev/liuh/fix-cat-201811

Conversation

@liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Mar 28, 2023

Improve sudo cat command for RO user.
Manually cherry-pick for #14428

Why I did it

RO user can use sudo command show none syslog files.

How I did it

Improve sudo cat command for RO user.

How to verify it

Pass all UT.
Manually check fixed code work correctly.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Improve sudo cat command for RO user.

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

Improve sudo cat command for RO user.

RO user can use sudo command show none syslog files.

Improve sudo cat command for RO user.

Pass all UT.
Manually check fixed code work correctly.

Improve sudo cat command for RO user.
@liuh-80 liuh-80 changed the title Improve sudo cat command for RO user. (#14428) [201811] Improve sudo cat command for RO user. (#14428) Mar 28, 2023
@liuh-80 liuh-80 requested a review from qiluo-msft March 28, 2023 00:36
@qiluo-msft qiluo-msft requested a review from yxieca March 28, 2023 01:01
@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 28, 2023

This PR need a force merge, currently 201811 branch build break because jessie package deprecated on debian repo:

W: Failed to fetch http://debian-archive.trafficmanager.net/debian/dists/jessie/main/source/Sources 404 Not Found

W: Failed to fetch http://debian-archive.trafficmanager.net/debian/dists/jessie/contrib/source/Sources 404 Not Found

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 31, 2023

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 14438 in repo sonic-net/sonic-buildimage

@liuh-80
Copy link
Contributor Author

liuh-80 commented Apr 7, 2023

Close and re-open to trigger validation again.

@liuh-80 liuh-80 closed this Apr 7, 2023
@liuh-80 liuh-80 reopened this Apr 7, 2023
@lguohan lguohan merged commit 273ca35 into sonic-net:201811 Apr 12, 2023
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.

3 participants