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

Added a new nginx plus api endpoints #6508

Merged
merged 2 commits into from
Dec 18, 2019
Merged

Added a new nginx plus api endpoints #6508

merged 2 commits into from
Dec 18, 2019

Conversation

kzaitsev
Copy link
Contributor

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

New endpoints

  • Added http location zones measurements ( only for api version >= 5 )
  • Added resolver zones measurements ( only for api version >= 5 )

documentation

glinton
glinton previously requested changes Oct 10, 2019
View reviewed changes
Copy link
Contributor

@glinton glinton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

plugins/inputs/nginx_plus_api/README.md Outdated Show resolved Hide resolved
plugins/inputs/nginx_plus_api/README.md Outdated Show resolved Hide resolved
@glinton glinton added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Oct 10, 2019
@kzaitsev kzaitsev requested a review from glinton October 13, 2019 20:48
@psvmcc
Copy link

psvmcc commented Dec 17, 2019

Any news about this PR?

@danielnelson danielnelson added this to the 1.14.0 milestone Dec 18, 2019
@danielnelson danielnelson dismissed glinton’s stale review December 18, 2019 00:57

comments are addressed

@danielnelson danielnelson merged commit b380c65 into influxdata:master Dec 18, 2019
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
@kzaitsev
Add support for new nginx plus api endpoints (influxdata#6508)
282a1cf
idohalevi pushed a commit to idohalevi/telegraf that referenced this pull request Sep 29, 2020
@kzaitsev
Add support for new nginx plus api endpoints (influxdata#6508)
7a0afd5
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
@kzaitsev @danielnelson
Add support for new nginx plus api endpoints (influxdata#6508)
7f93e6d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielnelson danielnelson danielnelson approved these changes

@glinton glinton Awaiting requested review from glinton

Assignees
No one assigned
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin
Projects
None yet
Milestone
1.14.0
Development

Successfully merging this pull request may close these issues.
4 participants
@kzaitsev @psvmcc @danielnelson @glinton