Skip to content

[202311][dhcp_server] Add rsyslog support for dhcp_server (#19303)#19410

Merged
yxieca merged 3 commits intosonic-net:202311from
yaqiangz:202311_dhcp_server_log
Jul 1, 2024
Merged

[202311][dhcp_server] Add rsyslog support for dhcp_server (#19303)#19410
yxieca merged 3 commits intosonic-net:202311from
yaqiangz:202311_dhcp_server_log

Conversation

@yaqiangz
Copy link
Contributor

@yaqiangz yaqiangz commented Jun 27, 2024

Why I did it

Manually backport and resolve conflicts of this pr: #19303
Fix issue that dhcp_server doesn't create syslog into host #18472

Work item tracking
  • Microsoft ADO (number only): 28396389

How I did it

Modify rsyslog config template

How to verify it

UTs passed
Buildimage and install in testbed to verify

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)

Description for the changelog

Link to config_db schema for YANG module changes

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

yaqiangz added 3 commits June 27, 2024 10:29
…19303)

Why I did it
Fix issue that dhcp_server doesn't create syslog into host sonic-net#18472

Work item tracking
Microsoft ADO (number only): 28396389
How I did it
Modify rsyslog config template

How to verify it
UTs passed
Buildimage and install in testbed to verify
@yaqiangz yaqiangz force-pushed the 202311_dhcp_server_log branch from 5d376a9 to a46f231 Compare June 28, 2024 09:46
@yaqiangz yaqiangz marked this pull request as ready for review June 28, 2024 14:34
@yaqiangz yaqiangz requested a review from lguohan as a code owner June 28, 2024 14:34
@yaqiangz
Copy link
Contributor Author

yaqiangz commented Jul 1, 2024

Hi @yxieca could you please help to merge this PR?
It's manually backport for #19303

@yxieca yxieca merged commit 93ed2d3 into sonic-net:202311 Jul 1, 2024
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