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.

Sign up for GitHub

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

Jump to bottom

Sensormon (Voltage and Current Sensors) support with PDDF #17032

Open
bmridul opened this issue Oct 27, 2023 · 3 comments
Open

Sensormon (Voltage and Current Sensors) support with PDDF #17032

bmridul opened this issue Oct 27, 2023 · 3 comments
Assignees
@FuzailBrcm
Labels
BRCM Triaged this issue has been triaged

Comments

@bmridul
Copy link
Contributor

bmridul commented Oct 27, 2023

Description

Support Sensormon intergration with PDDF. This will allow voltage and current sensor monitoring with PDDF support.

Additional information

This issue will track PDDF support for SensorMon. PDDF already provides data files based representation for thermal sensors. This can be extended for voltage and current sensors and their monitoring via Sensormon.
@FuzailBrcm
Copy link
Contributor

@bmridul
Can you provide some documentation/references on 'SensorMon'?
We can look into them and see how the PDDF can be enhanced for compliance.

@FuzailBrcm
Copy link
Contributor

@geans-pin , @adyeung
Adding them for reference.

@adyeung adyeung added Triaged this issue has been triaged BRCM labels Nov 8, 2023
@bmridul
Copy link
Contributor Author

bmridul commented Nov 9, 2023

@bmridul Can you provide some documentation/references on 'SensorMon'? We can look into them and see how the PDDF can be enhanced for compliance.

@FuzailBrcm , Sensormon HLD and code pointers can be found here
sonic-net/SONiC#1394

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FuzailBrcm FuzailBrcm

Labels
BRCM Triaged this issue has been triaged
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bmridul @adyeung @FuzailBrcm