Skip to content

Conversation

@Marketen
Copy link
Contributor

The CPU temperature query is now
avg(node_hwmon_temp_celsius{chip=~".*coretemp.*|.*18_3$|.*k10temp.*"}).

It returns the average temperature reported by lm-sensors from hardware monitoring chips typically associated with the CPU.

  • contains "coretemp": intel CPU
  • ends in 18_3: AMD CPU
  • contains k10temp: AMD CPU

This filters out unrelated temperature sensors like NVMe drives, GPU, or ACPI zones, ensuring that only CPU-related readings are averaged.

@Marketen Marketen self-assigned this May 28, 2025
@Marketen Marketen requested a review from a team as a code owner May 28, 2025 09:24
@pablomendezroyo pablomendezroyo merged commit a7fbd04 into pablo/add-notifications May 28, 2025
2 checks passed
@pablomendezroyo pablomendezroyo deleted the marc/improvecpunotification branch May 28, 2025 10:02
pablomendezroyo added a commit that referenced this pull request Jun 2, 2025
* Add notifications

* update notifications

* edit conditions

* add temperature and host out of disk space metrics

* fix typo

* use average of cpu cores only

* update definition

* add prio

* add correlationid and banner

* set disk and temperature as banner

* consistency with evaluation

* use category hardware

* use single line

* dont use white space

* update notifications yaml

* Update copies

* improve cpu query (#84)

* fix cpu usange and temp

---------

Co-authored-by: Pablo Mendez <pablo@dappnode.io>
Co-authored-by: mateumiralles <mateumiralles714@gmail.com>
Co-authored-by: Marc Font <36164126+Marketen@users.noreply.github.com>
Co-authored-by: Marketen <marcfont12@gmail.com>
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.

3 participants