Webmin 1.953 (with authentic theme 19.52) appears to be showing almost all temperatures reported by the sensors command as "Core N". For example, my two core system shows these FIVE core temperatures:
I have not been able to locate the configuration settings to CHOOSE the temperatures I'd like to monitor on the dashboard, so I assume a piece of code is confused.
EDIT: I should add that I have webmin of six different servers, all with different hardware (but the same rev of Ubuntu 18.04.newest), and they all show more core temperatures than they have cores.
Last edit: Steve Valliere 2020-07-06
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
No, my version of webmin already HAS those exact lines (I found the file in /usr/share/webmin/proc, I hope that was the correct location.) Hmmm... "remove these lines'" just sunk in, but I do not know which lines you mean? The new lines 506/507 and revert to the original 506?
I think the problem is that the new code is including most of the 'temp' values (but not the GPU temp for some reason) and presenting them as if they are 'core' temps. At least in all of my systems, the 'temp' values are quite clearly NOT 'core' temperatures.
That said, the other temperatures would be good to include, but they should be labeled correctly as motherboard, GPU (or whatever) temperatures and not all labeled 'core1'
Last edit: Steve Valliere 2020-07-07
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
OK, I commented the two new lines (506/507) and restored the previous 506.
Then I asked the browser to refresh the dashboard screen and there was no change to the set of temperatures displayed.
Is there something more I must do in order to truly get the dashboard temperatures refreshed or was the browser refresh enough?
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
The authentic theme just updated to 19.53 and it appears to have fixed this issue for almost all of my systems. However, one system is still showing a PCI adapter temp as an additional Core 1 temp. Here is the output from sensors:
This is how the 'temp1' value is displayed as an additional 'Core 1' on my screen:
Perhaps the actual labels from the temperatures selected should be used instead of renaming any temperatures selected as "Core N"?
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Strange that the authentic update seems to have corrected 3 out of 5 (I manually corrected one following your instructions here to test.)
I did not apply the patch because (a) I'm not really comfortable trying to figure out the process; (b) I'd need to apply it to six computers; and (c) I wasn't sure how a manual patch would affect future updates (I been forced to remove/reinstall in order to get updates of things working again in the past -- not for webmin, but for other things.)
When the authentic update seemed to have corrected the display on a few dashboards, I thought the core temps on the dashboard must have been an authentic feature. Sorry for my mistake. I'll drop this completely now. Thanks for your help. :-)
EDIT: Just to test your patch, I applied it to the machine described here and the patch corrected the display to only the two Core temps. Thanks again!
Last edit: Steve Valliere 2020-07-13
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I'm still seeing strange core temps reported for one of my systems:
Here is the sensors output from that machine (captured through webmin's command shell function:
> sensors
adt7473-i2c-2-2e
Adapter: nvkm-0000:01:00.0-bus-0002
in1: +3.00 V (min = +0.00 V, max = +2.99 V)
+3.3V: +3.29 V (min = +2.96 V, max = +3.63 V)
fan1: 1129 RPM (min = 0 RPM)
fan2: 0 RPM (min = 0 RPM)
fan3: 0 RPM (min = 164 RPM) ALARM
fan4: 0 RPM (min = 0 RPM)
temp1: +47.2 C (low = +72.0 C, high = +92.0 C) ALARM
(crit = +191.0 C, hyst = +189.0 C)
Board Temp: +45.5 C (low = +20.0 C, high = +60.0 C)
(crit = +100.0 C, hyst = +96.0 C)
temp3: +47.2 C (low = +80.0 C, high = +105.0 C) ALARM
(crit = +100.0 C, hyst = +96.0 C)
coretemp-isa-0000
Adapter: ISA adapter
Core 0: +58.0 C (high = +84.0 C, crit = +100.0 C)
Core 1: +61.0 C (high = +84.0 C, crit = +100.0 C)
nouveau-pci-0100
Adapter: PCI adapter
GPU core: +1.30 V (min = +1.30 V, max = +1.40 V)
temp1: +48.0 C (high = +95.0 C, hyst = +3.0 C)
(crit = +110.0 C, hyst = +2.0 C)
(emerg = +135.0 C, hyst = +5.0 C)
dell_smm-virtual-0
Adapter: Virtual device
Processor Fan: 1025 RPM
Motherboard Fan: 1243 RPM
I think this still needs work, since it is confusing motherboard temperatures with CPU (core) temperatures.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I'm still seeing strange core temps reported for one of my systems:
Duh, just as odd as the sensors output itself!
Where is the CPU info in this output? Don't you think that you should rather file a bug to sensors project or even to your hardware manufacture. I don't know if we should wittingly work around apparently malformed command outputs?
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I'll try to find the sensors project, but it was pretty clear to me that the temps for "Core 0" and "Core 1" in the "coretemp" section were the temperatures that I would expect to see as "Core 1" and "Core 2" (or 0 and 1) on the webmin dashboard. But "temp1" and "temp3" from the motherboard section are also listed as core temps.
When I'm stuck in the middle with the sensors people saying you should handle it, and you saying they should fix it, I'll just give up because the world is messy and likely to stay thay way. Oh well. Thanks anyways.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
The patch works, only the actual core temperatures are showing now. I also posted my sensors output to the sensors project to see if they have any ideas about improving it. Thanks!
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
>sensors
w83627hf-isa-0290
Adapter: ISA adapter
in0: 1.23 V (min = +0.00 V, max = +4.08 V)
in1: 1.58 V (min = +0.00 V, max = +4.08 V)
in2: 3.34 V (min = +2.82 V, max = +3.79 V)
in3: 3.04 V (min = +1.78 V, max = +3.20 V)
in4: 3.23 V (min = +3.98 V, max = +4.05 V) ALARM
in5: 512.00 mV (min = +3.54 V, max = +4.02 V) ALARM
in6: 3.60 V (min = +4.08 V, max = +4.05 V) ALARM
in7: 3.41 V (min = +2.74 V, max = +3.06 V) ALARM
in8: 3.26 V (min = +2.00 V, max = +1.89 V) ALARM
fan1: 6026 RPM (min = 451 RPM, div = 16)
fan2: 1259 RPM (min = 598 RPM, div = 16)
fan3: 1339 RPM (min = 0 RPM, div = 16)
temp1: +32.0°C (high = +125.0°C, hyst = -2.0°C) sensor = thermistor
temp2: +24.5°C (high = +70.0°C, hyst = +65.0°C) sensor = CPU diode
temp3: +58.5°C (high = +70.0°C, hyst = +65.0°C) sensor = thermistor
cpu0_vid: +1.113 V
beep_enable: enabled
acpitz-acpi-0
Adapter: ACPI interface
temp1: +24.5°C (crit = +70.0°C)
coretemp-isa-0000
Adapter: ISA adapter
Core 0: +35.0°C (crit = +100.0°C)
I'm sorry but I can't add a dashboard image.
Webmin displays a CPU temperature of 41 ° C on the dashboard.
Webmin version is 1.981.
Is it possible to display the HDD temperature on the dashboard?
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Webmin 1.953 (with authentic theme 19.52) appears to be showing almost all temperatures reported by the sensors command as "Core N". For example, my two core system shows these FIVE core temperatures:
But here is what the secsors command shows:
I have not been able to locate the configuration settings to CHOOSE the temperatures I'd like to monitor on the dashboard, so I assume a piece of code is confused.
EDIT: I should add that I have webmin of six different servers, all with different hardware (but the same rev of Ubuntu 18.04.newest), and they all show more core temperatures than they have cores.
Last edit: Steve Valliere 2020-07-06
Hi,
Isn't that great to get an upgarde by Webmin? :--)
Anyway, if you remove these lines from mentioned file and click refresh button on the Dashboard, does it fix that?
No, my version of webmin already HAS those exact lines (I found the file in /usr/share/webmin/proc, I hope that was the correct location.) Hmmm... "remove these lines'" just sunk in, but I do not know which lines you mean? The new lines 506/507 and revert to the original 506?
I think the problem is that the new code is including most of the 'temp' values (but not the GPU temp for some reason) and presenting them as if they are 'core' temps. At least in all of my systems, the 'temp' values are quite clearly NOT 'core' temperatures.
That said, the other temperatures would be good to include, but they should be labeled correctly as motherboard, GPU (or whatever) temperatures and not all labeled 'core1'
Last edit: Steve Valliere 2020-07-07
OK, I commented the two new lines (506/507) and restored the previous 506.
Then I asked the browser to refresh the dashboard screen and there was no change to the set of temperatures displayed.
Is there something more I must do in order to truly get the dashboard temperatures refreshed or was the browser refresh enough?
The lines in yellow. Starting with 505 and upto next elsif statement.
Thank you, yes, removing those lines eliminated the non-core core temperatures from the dashboard.
In case any developers are reading this, here is another example that uses 'Ambient" instead of 'temp' or 'core' for some motherboard temps:
This system was showing ONE extra core temperature, so I'm guessing that the radeon temperature was used this time.
Last edit: Steve Valliere 2020-07-07
Fixed, thanks.
https://github.com/webmin/webmin/commit/3fa7141ae729dd0355f6050198b5a4771c7a28b3
The authentic theme just updated to 19.53 and it appears to have fixed this issue for almost all of my systems. However, one system is still showing a PCI adapter temp as an additional Core 1 temp. Here is the output from sensors:
This is how the 'temp1' value is displayed as an additional 'Core 1' on my screen:
Perhaps the actual labels from the temperatures selected should be used instead of renaming any temperatures selected as "Core N"?
Steve, did you apply this patch?
https://github.com/webmin/webmin/commit/3fa7141ae729dd0355f6050198b5a4771c7a28b3
It was not Authentic Theme issue, by the way.
Strange that the authentic update seems to have corrected 3 out of 5 (I manually corrected one following your instructions here to test.)
I did not apply the patch because (a) I'm not really comfortable trying to figure out the process; (b) I'd need to apply it to six computers; and (c) I wasn't sure how a manual patch would affect future updates (I been forced to remove/reinstall in order to get updates of things working again in the past -- not for webmin, but for other things.)
When the authentic update seemed to have corrected the display on a few dashboards, I thought the core temps on the dashboard must have been an authentic feature. Sorry for my mistake. I'll drop this completely now. Thanks for your help. :-)
EDIT: Just to test your patch, I applied it to the machine described here and the patch corrected the display to only the two Core temps. Thanks again!
Last edit: Steve Valliere 2020-07-13
I'm still seeing strange core temps reported for one of my systems:
Here is the sensors output from that machine (captured through webmin's command shell function:
I think this still needs work, since it is confusing motherboard temperatures with CPU (core) temperatures.
Duh, just as odd as the
sensorsoutput itself!Where is the CPU info in this output? Don't you think that you should rather file a bug to
sensorsproject or even to your hardware manufacture. I don't know if we should wittingly work around apparently malformed command outputs?I'll try to find the sensors project, but it was pretty clear to me that the temps for "Core 0" and "Core 1" in the "coretemp" section were the temperatures that I would expect to see as "Core 1" and "Core 2" (or 0 and 1) on the webmin dashboard. But "temp1" and "temp3" from the motherboard section are also listed as core temps.
When I'm stuck in the middle with the sensors people saying you should handle it, and you saying they should fix it, I'll just give up because the world is messy and likely to stay thay way. Oh well. Thanks anyways.
I will see what I can do. ಠ_ಠ⋋
Have a look, this patch should solve this issue.
The patch works, only the actual core temperatures are showing now. I also posted my sensors output to the sensors project to see if they have any ideas about improving it. Thanks!
If they reply, just let us know.
Hello
I have a problem with CPU temperature in the dashboard. Temperature is not correct.
This is my dashboard:
This is my sensors output:
Could you please help me?
Peter, I dont see any information, as screenshots are missing.
Besides, I would need sensor command output as text.
Any update on this?
Do you have latest Webmin vwrsion 1.981? What's the outout of
sensorscommand?Hello
Sensor command output as text:
I'm sorry but I can't add a dashboard image.
Webmin displays a CPU temperature of 41 ° C on the dashboard.
Webmin version is 1.981.
Is it possible to display the HDD temperature on the dashboard?
6000 rpm, really!? :D
I don't know. It wouldn't be difficult to make all of this work if
sensorscommand output would be consistent.It's already possible, you need
smart-statusmodule andsmartctlcommand installed (fromsmartmontoolspackage).