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.

Sign up for GitHub

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

Jump to bottom

[Bug]: Fix show ip bgp nei command rw required issue #2011

Merged
merged 1 commit into from
Jan 17, 2022
Merged

[Bug]: Fix show ip bgp nei command rw required issue #2011

merged 1 commit into from
Jan 17, 2022

Conversation

xumia
Copy link
Collaborator

@xumia xumia commented Jan 14, 2022

What I did

Fix the ro command rw permission required issue

show ip bgp neighbors 

Make sure your account has RW permission to current device.
Otherwise sudo requests will be rejected.

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@xumia xumia added the Bug label Jan 14, 2022
@xumia xumia requested a review from qiluo-msft January 14, 2022 10:08
@xumia xumia mentioned this pull request Jan 14, 2022
Merged
4 tasks
@ghooo
Copy link
Contributor

ghooo commented Jan 14, 2022

Build failure should be fixed by #2012

@qiluo-msft qiluo-msft mentioned this pull request Jan 14, 2022
Draft
@xumia
Copy link
Collaborator Author

xumia commented Jan 16, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xumia
Copy link
Collaborator Author

xumia commented Jan 16, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xumia
Copy link
Collaborator Author

xumia commented Jan 17, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xumia xumia merged commit 5398ee4 into sonic-net:master Jan 17, 2022
xumia added a commit that referenced this pull request Jan 17, 2022
@xumia
Fix show ip bgp nei command rw required issue (#2011)
585fd40 
Fix the ro command rw permission required issue
xumia added a commit that referenced this pull request Jan 17, 2022
@xumia
Fix show ip bgp nei command rw required issue (#2011)
ef47754 
Fix the ro command rw permission required issue
xumia added a commit that referenced this pull request Jan 17, 2022
@xumia
Fix show ip bgp nei command rw required issue (#2011)
6597c25 
Fix the ro command rw permission required issue
@SuvarnaMeenakshi
Copy link
Contributor

SuvarnaMeenakshi commented Mar 21, 2022 •
edited
Loading

@vperumal @anamehra @VenkatCisco fyi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qiluo-msft qiluo-msft qiluo-msft approved these changes

Assignees
No one assigned
Labels
Bug Included in 201911 Branch Included in 202106 Branch Included in 202111 Branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@xumia @ghooo @SuvarnaMeenakshi @qiluo-msft