search

sensu / sensu-slack-handler

The Sensu Go Slack handler for notifying a channel.
https://sensu.io
MIT License
3 stars 26 forks source link

Remove OS's from Unsupported/Experimental etcd versions #24

Closed asachs01 closed 2 years ago

asachs01 commented 4 years ago

According to https://github.com/etcd-io/etcd/blob/master/Documentation/op-guide/supported-platform.md, etcd is only stable on Linux/amd64 and Linux/ppc64le. Since handlers are only executed on the backend, I don't believe it makes much sense to have builds available for systems where etcd is listed as "Unstable" or "Experiemental", as we won't be supporting unstable/experimental backends. Having these additional assets available seems like it would add unnecessary bloat to the asset cache.

trombik commented 4 years ago

one of reasons why Nagios/Icinga was popular is that they are portable. Unlike web developers, sysadmins manage lots of different OS/arch you never have used. we, sysadmins, usually do not have choices. you do not have to officially support platforms that you are not familiar with, just let users maintain their platforms, like #33

echlebek commented 2 years ago

sensu-backend does not need to run within the same constraints as etcd. We can be more liberal than etcd, especially when it comes to pieces like handlers and agents. I don't think it's logical to apply those constraints here, so I'm closing the PR. Please re-open if you feel strongly about it.