Skip to content

[AS5835-54X] Support system-health and remove extra code #8137

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

Conversation

jostar-yang
Copy link
Contributor

@jostar-yang jostar-yang commented Jul 9, 2021

Signed-off-by: Jostar Yang [email protected]

Why I did it

Add to support show system-health

How I did it

Add system_health_monitoring_config.json and needed code to sonic_platform

How to verify it

Test show system-health cmd.

System services and devices monitor list

Name                      Status    Type
------------------------  --------  ----------
sonic                     OK        System
rsyslog                   OK        Process
root-overlay              OK        Filesystem
var-log                   OK        Filesystem
routeCheck                OK        Program
container_checker         OK        Program
telemetry|telemetry       OK        Program
telemetry|dialout_client  OK        Program
teamd|teamsyncd           OK        Program
teamd|teammgrd            OK        Program
syncd|syncd               OK        Program
syncd|dsserve             OK        Program
swss|orchagent            OK        Program
swss|portsyncd            OK        Program
swss|neighsyncd           OK        Program
swss|fdbsyncd             OK        Program
swss|vrfmgrd              OK        Program
swss|vlanmgrd             OK        Program
swss|intfmgrd             OK        Program
swss|portmgrd             OK        Program
swss|buffermgrd           OK        Program
swss|nbrmgrd              OK        Program
swss|vxlanmgrd            OK        Program
swss|coppmgrd             OK        Program
swss|tunnelmgrd           OK        Program
snmp|snmpd                OK        Program
snmp|snmp_subagent        OK        Program
sflow|sflowmgrd           OK        Program
lldp|lldpd_monitor        OK        Program
lldp|lldp_syncd           OK        Program
lldp|lldpmgrd             OK        Program
database|redis_server     OK        Program
bgp|zebra                 OK        Program
bgp|fpmsyncd              OK        Program
bgp|bgpd                  OK        Program
bgp|staticd               OK        Program
bgp|bgpcfgd               OK        Program
bgp|bgpmon                OK        Program
PSU-2 FAN-1               Not OK    Fan
PSU 2                     Not OK    PSU
FAN-1F                    OK        Fan
FAN-1R                    OK        Fan
FAN-2F                    OK        Fan
FAN-2R                    OK        Fan
FAN-3F                    OK        Fan
FAN-3R                    OK        Fan
FAN-4F                    OK        Fan
FAN-4R                    OK        Fan
FAN-5F                    OK        Fan
FAN-5R                    OK        Fan
PSU-1 FAN-1               OK        Fan
PSU 1                     OK        PSU

System services and devices ignore list

Name             Status    Type
---------------  --------  ------
psu.temperature  Ignored   Device
asic             Ignored   Device

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 9, 2021

This pull request introduces 1 alert when merging a546b99 into cb2ffa3 - view on LGTM.com

new alerts:

  • 1 for Unused local variable

Copy link
Collaborator

@lguohan lguohan left a comment

Choose a reason for hiding this comment

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

:shipit:

@lguohan lguohan merged commit 4eab151 into sonic-net:202012 Jul 25, 2021
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