-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add monitors.d directory + config to default configuration #9004
Conversation
cf4d49f
to
abc1b64
Compare
For any reviewers out there, how can we test this in the context of apt/deb to make sure those packages add the same files to the directory layout? |
For each package type you’ll need to apply a bit of customization to add your monitors.d directory to the package. Take a look at the magefile.go for filebeat/metricbeat/auditbeat for an example. After building the packages ( Hopefully this will get you started. I’m happy to chat after you’ve taken a look at this stuff. |
Pinging @elastic/uptime |
@andrewvc Anything I can help with to push this PR forward? |
@ruflin not quite ready for review yet. I just rebased it and added support in the magefile. Also fixed heartbeat.yml to point at a relative path |
OK, things seem to work and package tests are added. I actually haven't manually tested them yet. Can do tomorrow. There's a lot of opportunity to DRY things up in the package tests, but it feels premature to me for the effort. WDYT @ruflin / @andrewkroh ? |
abc1b64
to
95efd96
Compare
@andrewvc Code LGTM. Would be good if you could also test it locally and after merging keep an eye on the packaging tests. In case you rebase again on master or do an additonal push, could you also make Hound happy? CI failures are not related. |
95efd96
to
5aec00d
Compare
@ruflin I tested this on OSX/Ubuntu Server/Fedora VMs using .tar.gz/.deb/.rpm respectively . Everything worked great. On the linux box I set things up as a service using ES as a service. All I had to do to get a working monitor was remove the I've improved the PR somewhat adding separate sample files per protocol to |
jenkins, please retest this looks like weird transient yum/docker issues |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI failure do not seem to be related and the ones failing are "in the works" AFAIK.
cd58c39
to
e861cd7
Compare
@ruflin I've set |
e861cd7
to
f531df9
Compare
018bdfd
to
7b72ca4
Compare
This makes heartbeat much easier to use by default, and less confusing in terms of where/how to create this directory.
7b72ca4
to
edb26c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI failures are not related.
) This makes heartbeat much easier to use by default, and less confusing in terms of where/how to create this directory. (cherry picked from commit 2e92a04)
This makes heartbeat much easier to use by default, and less confusing in terms of where/how
to create this directory.