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.

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

[202205] Add special rsyslog filter for MSN2700 platform (#16684) #17020

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Oct 26, 2023

Why I did it

backport #16684

Mellanox MSN2700 platforms have a non-functional error log: "ERR pmon#sensord: Error getting sensor data: dps460/#10: Can't read". This error is because of a firmware issue with some PSU, we are not able to upgrade the FW online. Since there is no functional impact, this error log can be ignored safely.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Add a new rsyslog rule to the rsyslog-container.conf.j2, if the docker name is pmon and the platform name matches, the new rule will be inserted into the docker rsyslogd.conf

How to verify it

run regression on the MSN2700 platform to make the error log will not be printed to the syslog.

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)

@keboliu
Copy link
Collaborator Author

keboliu commented Oct 26, 2023

@yxieca please help to approve and merge when the checker pass, this is to backport #16684

@yxieca yxieca merged commit cb840c1 into sonic-net:202205 Oct 26, 2023
16 checks passed
@keboliu keboliu deleted the 202205_filter_for_error_log branch October 27, 2023 02:53
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