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

RC 25.1 - Thermal Sensors dashboard widget data issue #8240

Closed
2 tasks done
Foowy opened this issue Jan 23, 2025 · 6 comments
Closed
2 tasks done

RC 25.1 - Thermal Sensors dashboard widget data issue #8240

Foowy opened this issue Jan 23, 2025 · 6 comments
Assignees
Labels
cleanup Low impact changes
Milestone

Comments

@Foowy
Copy link

Foowy commented Jan 23, 2025

Important notices

Before you add a new report, we ask you kindly to acknowledge the following:

Describe the bug

Dashboard widget for Thermal Sensors is listing contradicting data for polled CPU (or just invalid data)

Example:

CPU 0 shows -

29c
100c
75c
22.5c

Widget also references a CPU 00, 01, and 02

Widget polled data correctly (or at least displayed correctly up to 24.7.12_2)

Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)

To Reproduce

Steps to reproduce the behavior:

  1. Go to Lobby > Dashboard > View thermal sensor widget

Expected behavior

Display either correct data or allow modification of widget to hide invalid data fields

Describe alternatives you considered

I can remove widget so my anxiety doesn't get the best of me

Screenshots

Image

Relevant log files

None

Additional context

None

Environment

Software version used and hardware type if relevant, e.g.:

OPNsense 25.1.r_6-amd64
Intel® Core™ I5-7200U 2.gGhz Dual Core

@Foowy
Copy link
Author

Foowy commented Jan 23, 2025

Relevant Thermal pulls:

root@fire:/ # /sbin/sysctl -n dev.cpu.0.temperature hw.acpi.thermal.tz0.temperature hw.temperature.CPU
25.0C
24.1C
sysctl: unknown oid 'hw.temperature.CPU'
root@fire:/ # /sbin/sysctl -n dev.cpu.1.temperature hw.acpi.thermal.tz0.temperature hw.temperature.CPU
24.0C
24.1C
sysctl: unknown oid 'hw.temperature.CPU'
root@fire:/ # /sbin/sysctl -n dev.cpu.2.temperature hw.acpi.thermal.tz0.temperature hw.temperature.CPU
23.0C
24.1C
sysctl: unknown oid 'hw.temperature.CPU'
root@fire:/ # /sbin/sysctl -n dev.cpu.3.temperature hw.acpi.thermal.tz0.temperature hw.temperature.CPU
23.0C
24.1C
sysctl: unknown oid 'hw.temperature.CPU'

@Foowy
Copy link
Author

Foowy commented Jan 23, 2025

Looking at possible pull #8062 as parsing of new field data that widget cant currently support/display (i am not smart enough to determine if it will correct or is related to issue i am seeing)

@borisneubert
Copy link

I have the same issue. Thermal sensors widgets shows sensors with unreasonable readings and there is not edit button in dashboard edit mode to select only the sensors of interest.

@fichtner
Copy link
Member

There is a patch in the forum. It’s hard to keep track of all the duplicate reports at the moment.

@fichtner
Copy link
Member

fichtner commented Jan 31, 2025

@fichtner fichtner self-assigned this Jan 31, 2025
@fichtner fichtner added the cleanup Low impact changes label Jan 31, 2025
@fichtner fichtner added this to the 25.7 milestone Jan 31, 2025
@borisneubert
Copy link

patch works for me; big thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Low impact changes
Development

No branches or pull requests

3 participants