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

Added support for file system based sensors implementation. #438

Merged
merged 3 commits into from
Mar 20, 2024

Conversation

bmridul
Copy link
Collaborator

@bmridul bmridul commented Feb 21, 2024

Description

Added support for file systems based sensors implementation in Sensormond.
See HLD for reference.

Motivation and Context

Provides an easy way to add support for different vendor platforms in Sensormond.

How Has This Been Tested?

Unit Test and target test.

Additional Information (Optional)

@prgeor
Copy link
Collaborator

prgeor commented Feb 23, 2024

@bmridul please fix the build fialure

@bmridul
Copy link
Collaborator Author

bmridul commented Feb 23, 2024

@bmridul please fix the build fialure

Build failure is due to dependency on other PR - sonic-net/sonic-platform-common#426. We have to wait for that to get merged before this one..

@bmridul
Copy link
Collaborator Author

bmridul commented Mar 7, 2024

@prgeor , Pls check. Build issue is fixed now.

@prgeor prgeor merged commit d513eca into sonic-net:master Mar 20, 2024
5 checks passed
yuazhe pushed a commit to yuazhe/sonic-platform-daemons that referenced this pull request Jul 2, 2024
…t#438)

* Added support for file system based sensors implementation.

* Fixed UT

* Fixed UT coverage
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