-
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 Nginx stub status to metricbeat #1642
Conversation
Add Nginx module with stub status metric set
Conflicts: metricbeat/etc/beat.yml metricbeat/include/list.go metricbeat/metricbeat.yml
Conflicts: metricbeat/docker-entrypoint.sh metricbeat/docs/fields.asciidoc metricbeat/etc/beat.yml metricbeat/etc/fields.yml metricbeat/etc/fields_base.yml metricbeat/metricbeat.template-es2x.json metricbeat/metricbeat.template.json metricbeat/metricbeat.yml metricbeat/module/nginx/*
|
||
=== Example Configuration | ||
|
||
The Apache module supports the standard configuration options which can be found |
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.
s/Apache/Nginx/
Looks good, I just had a few minor comments. Thanks for your contribution! |
@mrkschan Thanks a lot for the contribution and already integrating our most recent metricbeat changes. I'm happy to merge it as it is and do the changes found by @andrewkroh afterwards so we can help with it, or you can push them yourself. What do you prefer? |
fixed, please let me do if there's any other issue :) |
@mrkschan All good. Thanks a lot 👍 |
No description provided.