Skip to content

Write error message to syslog when add user failed or connect to TACACS server failed. #17081

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

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Nov 3, 2023

Write error message to syslog when add user failed or connect to TACACS server failed.

Why I did it

With these messages, we can downgrade TACACS server with issue to lower priority.

Work item tracking
  • Microsoft ADO: 24667696

How I did it

Write error message to syslog when add user failed or connect to TACACS server failed.

How to verify it

Pass all UT.
Manually verify error message generated.

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

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

Tested branch (Please provide the tested image version)

  • master-16240.344768-113309b9d
  • SONiC.202305-17081.401641-ec2aed854

Description for the changelog

Write error message to syslog when add user failed or connect to TACACS server failed.

Link to config_db schema for YANG module changes

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

…CS server failed. (sonic-net#16240)

Write error message to syslog when add user failed or connect to TACACS server failed.

With these messages, we can downgrade TACACS server with issue to lower priority.

- Microsoft ADO: 24667696

Write error message to syslog when add user failed or connect to TACACS server failed.

Pass all UT.
Manually verify error message generated.

Write error message to syslog when add user failed or connect to TACACS server failed.
Copy link
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS merged commit a11b33b into sonic-net:202305 Nov 6, 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.

2 participants