Skip to content

[202305][rsyslog]: Remote logging with the highest rule priority #21993

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 3 commits into from
May 6, 2025

Conversation

nazariig
Copy link
Collaborator

Why I did it

This fix resolves issue with missing certain types of messages while forwarding to the remote server.

Basically, this moves remote syslog forwarding rules to be executed before any other instructions take place,
so everything received by a local syslog instance will be forwarded to the remote machine.

The fix allows remote forwarding of messages defined as:
https://github.com/sonic-net/sonic-buildimage/blob/master/files/image_config/rsyslog/rsyslog.d/00-sonic.conf.j2

Work item tracking

  • N/A

How I did it

  • Moved rsyslog remote forwarding rules to be on top of other syslog instructions

How to verify it

  1. Enable remote syslog forwarding
  2. Check for message patterns defined as 00-sonic.conf.j2

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)

  • 202305

Description for the changelog

  • N/A

Link to config_db schema for YANG module changes

  • N/A

Details if related

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

      .---.        .-----------
     /     \  __  /    ------
    / /     \(  )/    -----
   //////   ' \/ `   ---
  //// / // :    : ---
 // /   /  /`    '--
//          //..\\
       ====UU====UU====
           '//||\\`
             ''``

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nazariig nazariig added the bugfix the PR is a bug fix PR label Mar 11, 2025
Signed-off-by: Nazarii Hnydyn <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nazariig nazariig marked this pull request as ready for review March 12, 2025 09:11
@nazariig nazariig requested a review from lguohan as a code owner March 12, 2025 09:11
@bingwang-ms
Copy link
Contributor

@StormLiangMS Can you please help merge this PR?

@nazariig
Copy link
Collaborator Author

@qiluo-msft can you please help to merge?

@nazariig
Copy link
Collaborator Author

#21984

@nazariig
Copy link
Collaborator Author

@qiluo-msft just a kind reminder: please help to merge

@StormLiangMS StormLiangMS merged commit e974ff3 into sonic-net:202305 May 6, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🐛 bugfix the PR is a bug fix PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants