search

saltstack-formulas / beats-formula

SaltStack formula to manage Elastic Beats platform
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
MIT License
9 stars 19 forks source link

Add Heartbeat #14

Closed syphernl closed 5 years ago

syphernl commented 5 years ago

This PR adds the Heartbeat (shipper for uptime monitoring).

https://www.elastic.co/products/beats/heartbeat

aboe76 commented 5 years ago

@psmiraglia are you satisfied ? then we can merge it?

psmiraglia commented 5 years ago

Hey @aboe76 and @syphernl, sorry for the delay by I was a little bit busy in the last days. The changes look good to me so we can merge!

psmiraglia commented 5 years ago

@aboe76 @syphernl I just realised that we made a mistake with the semantic versioning. We added Heartbeat support a.k.a. a new feature. According to semver rules, the new version should be 0.2.0 (backward compatible new feature) instead of 0.1.2 (backward compatible hotfix). If you agree, I can fix it.

aboe76 commented 5 years ago

@psmiraglia If t's not too much to ask please do, we are in the process of adding semver to saltstack formulas, see the https://github.com/saltstack-formulas/template-formula for the progress