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

Add support for Rock Pi GPU temp #14

Open
jorkzijlstra opened this issue Oct 19, 2020 · 2 comments
Open

Add support for Rock Pi GPU temp #14

jorkzijlstra opened this issue Oct 19, 2020 · 2 comments

Comments

@jorkzijlstra
Copy link

Hello,

Currently I'm using a Rock Pi 4B with your arm_exporter. Its working great for the CPU temperature.
Its only logging an exception for the gpu temperature:

time="2020-10-19T20:04:32Z" level=error msg="gpu collector failed after 0.006953s: exit status 255" source="collector.go:142"

The reason is because the files aren't there on the Ubuntu.

CPU temp:

x@x:/sys/class/thermal/thermal_zone0$ cat type 
soc-thermal
x@x:/sys/class/thermal/thermal_zone0$ cat temp 
46250

GPU temp

x@x:/sys/class/thermal/thermal_zone1$ cat type 
gpu-thermal
x@x:/sys/class/thermal/thermal_zone1$ cat temp 
42777

Would it be possible to include these in the gpu exporter.

@lukasmalkmus
Copy link
Owner

I could look into that. This would require some additional checks. Do you feel confident building from source so you can check it is working on the actual hardware if I bring up a PR?

Till then, maybe you can tinker around with this: https://gist.github.com/lukasmalkmus/410232035c3e831cb780347e77ae5382

@jorkzijlstra
Copy link
Author

@lukasmalkmus I'm a programmer but just not used to Go. I already had a look into the code and my plan was to return a specific collector based upon the existence of the files in the NewGPUCollector method but I haven't had the change to actually make a change let alone a PR.

So I'm fine building it from scratch.

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

No branches or pull requests

2 participants