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.

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

NGINX+ Prometheus Metrics #1330

Closed
mpstefan opened this issue Dec 4, 2023 · 1 comment · Fixed by #1394
Closed

NGINX+ Prometheus Metrics #1330

mpstefan opened this issue Dec 4, 2023 · 1 comment · Fixed by #1394
Assignees
Labels
metrics Additions or changes to metrics from NKG refined Requirements are refined and the issue is ready to be implemented. size/small Estimated to be completed within ~2 days
Milestone

Comments

@mpstefan
Copy link
Collaborator

mpstefan commented Dec 4, 2023

As a user of NGF
I want to utilize the expanded metrics of NGF
So that I can have greater visibility into how NGINX and my applications are performing.

Acceptance

  • When a NGF image with NGINX+ is running, the Prometheus endpoint is updated with all NGINX+ metrics.
  • Enhance the current documentation on metrics to include NGINX+ metrics.
@mpstefan mpstefan mentioned this issue Dec 4, 2023
@mpstefan mpstefan added the metrics Additions or changes to metrics from NKG label Dec 4, 2023
@mpstefan mpstefan added this to the v1.2.0 milestone Dec 4, 2023
@lucacome
Copy link
Contributor

lucacome commented Dec 4, 2023

Do we need nginx/nginx-prometheus-exporter#484 ? cc @ciarams87

@mpstefan mpstefan added refined Requirements are refined and the issue is ready to be implemented. size/small Estimated to be completed within ~2 days labels Dec 11, 2023
@ciarams87 ciarams87 self-assigned this Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metrics Additions or changes to metrics from NKG refined Requirements are refined and the issue is ready to be implemented. size/small Estimated to be completed within ~2 days
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants