Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[metricbeat] change server_status_path default setting for nginx module #13806

Closed
TomonoriSoejima opened this issue Sep 26, 2019 · 2 comments · Fixed by #14099
Closed

[metricbeat] change server_status_path default setting for nginx module #13806

TomonoriSoejima opened this issue Sep 26, 2019 · 2 comments · Fixed by #14099
Labels
candidate Candidate to be added to the current iteration good first issue Indicates a good issue for first-time contributors Hacktoberfest Metricbeat Metricbeat Team:Integrations Label for the Integrations team

Comments

@TomonoriSoejima
Copy link

In nginx.xml, it makes server_status_path as server-status, but https://www.nginx.com/blog/monitoring-nginx/ makes nginx_status as an example and some people simply copy&paste that and they can run into HTTP error 404 in stubstaus: 404 Not Found" error

  # Path to server status. Default server-status
  #server_status_path: "server-status"

Maybe changing the default to nginx_status reduces the hassles?
@TomonoriSoejima TomonoriSoejima changed the title [metricbeat] change server_status_path default setting. [metricbeat] change server_status_path default setting for nginx module Sep 26, 2019
@kaiyan-sheng kaiyan-sheng added Team:Integrations Label for the Integrations team Metricbeat Metricbeat labels Sep 26, 2019
@andresrc andresrc added [zube]: Inbox good first issue Indicates a good issue for first-time contributors candidate Candidate to be added to the current iteration [zube]: Ready Hacktoberfest and removed [zube]: Inbox labels Oct 1, 2019
@goku321
Copy link
Contributor

goku321 commented Oct 4, 2019

Can I work on this?

@kaiyan-sheng
Copy link
Contributor

@goku321 Yes please do! Please assign this issue to yourself once you start working and link the PR with the issue as well. Thanks!

@goku321 goku321 mentioned this issue Oct 16, 2019
Merged
goku321 added a commit to goku321/beats that referenced this issue Oct 24, 2019
@goku321
Merge branch 'master' into elasticGH-13806
8382d8b 
Syncing with master
goku321 added a commit to goku321/beats that referenced this issue Nov 24, 2019
@goku321
Merge branch 'master' into elasticGH-13806
e20bf4c 
Syncing with master.
@zube zube bot added [zube]: Done and removed [zube]: Ready labels Nov 24, 2019
@kaiyan-sheng kaiyan-sheng mentioned this issue Nov 24, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
candidate Candidate to be added to the current iteration good first issue Indicates a good issue for first-time contributors Hacktoberfest Metricbeat Metricbeat Team:Integrations Label for the Integrations team
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Metricbeat] change server_status_path default setting for nginx module goku321/beats
4 participants
@andresrc @goku321 @kaiyan-sheng @TomonoriSoejima