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

healthd crash: ImportError: No module named sonic_platform.platform #5610

Open
vboykox opened this issue Oct 13, 2020 · 2 comments
Open

healthd crash: ImportError: No module named sonic_platform.platform #5610

vboykox opened this issue Oct 13, 2020 · 2 comments

Comments

@vboykox
Copy link
Member

vboykox commented Oct 13, 2020

Description
Receiving next log in /var/log/syslog

Traceback (most recent call last):
  File "/usr/local/bin/healthd", line 108, in <module>
    main()
  File "/usr/local/bin/healthd", line 104, in main
    health_monitor.run()
  File "/usr/local/bin/healthd", line 71, in run
    import sonic_platform.platform
ImportError: No module named sonic_platform.platform

Steps to reproduce the issue:

  1. Deploy the image after [system-health] Add support for monitoring system health #4835 merge
  2. see /var/log/syslog

Describe the results you received:
the heathd monitor does not crash

Describe the results you expected:
The daemon is started inside PMON container or the sonic_platform package is installed to the host fs.

Additional information you deem important (e.g. issue happens only occasionally):

SONiC Software Version: SONiC.master.5-dirty-20201013.025725
Distribution: Debian 10.6
Kernel: 4.19.0-9-2-amd64
Build commit: bba5df05
Build date: Tue Oct 13 07:09:25 UTC 2020
Built by: johnar@jenkins-worker-8

Introduced by:
#4835

@vboykox
Copy link
Member Author

vboykox commented Oct 13, 2020

@Junchao-Mellanox

@Junchao-Mellanox
Copy link
Collaborator

Junchao-Mellanox commented Oct 14, 2020

Hi, this feature requred to load platform API on host side, please see design document: https://github.com/Azure/SONiC/blob/master/doc/system_health_monitoring/system-health-HLD.md#4-platform-api-and-pmon-related-change-to-support-this-new-service

And during our test on Mellanox platform, it works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants