martomi / chiadog

A watch dog providing a peace in mind that your Chia farm is running smoothly 24/7.
MIT License
460 stars 123 forks source link

Migrate configuration handling to the confuse lib, take 2 #364

Closed jinnatar closed 1 year ago

jinnatar commented 1 year ago

This enables:

This change touches all integrations and has minor potential for subtle breakage.

The following notifiers have been thoroughly tested:

Notifiers not tested:

Notifiers with known issues:

Notifiers I cannot test myself:

Groundwork for fixing #361

martomi commented 1 year ago

@Artanicus sorry, I might have merged while you’re still in WIP (just read the updated PR description). Feel free to finish the rest in a separate PR, as this one was already looking good and was quite large so I wanted to have it in master as a base for other changes :-)

jinnatar commented 1 year ago

No worries, all in all it's fairly solid. Slack I haven't tested yet (slightly more complex integration) but will, Grafana I know fails because it violates a few notifier conventions (have a WIP fix I'll PR separately since it's a bit opinionated), and Pushcut I can't test since they support such a limited set of platforms.