search

ianunruh / shelly-exporter

Prometheus exporter for Shelly Cloud devices
Apache License 2.0
17 stars 2 forks source link

Add a few useful metrics for Shelly TRV #3

Closed huhndev closed 1 year ago

huhndev commented 1 year ago

Moin!

I just added some metrics to your shelly_exporter for my own monitoring for Shelly's thermostats (TRV). Works as expected. Feel free to change the names and descriptions of the metrics if you think of something better.

ianunruh commented 1 year ago

Hi @huhndev, thanks for the contribution. Do you know if Shelly supports multiple thermostats? If so, the index should be included in the metric to disambiguate.

huhndev commented 1 year ago

Moin @ianunruh! As far as I know, each device (Shelly TRV) has only one thermostat that can be addressed via the API.

See for more details: https://shelly-api-docs.shelly.cloud/gen1/#shelly-trv

On Tue, May 09, 2023 at 08:36:44PM -0700, Ian Unruh wrote:

Hi @huhndev, thanks for the contribution. Do you know if Shelly supports multiple thermostats? If so, the index should be included in the metric to disambiguate.

-- Reply to this email directly or view it on GitHub: https://github.com/ianunruh/shelly-exporter/pull/3#issuecomment-1541300796 You are receiving this because you were mentioned.

Message ID: @.***>

ianunruh commented 1 year ago

@huhndev Sounds good. Just one minor thing, can you also bump the Golang base image in the Dockerfile?