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

Enable Prometheus metrics exporter for ingress-nginx controller. #297

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

achton
Copy link
Contributor

@achton achton commented May 2, 2024

What does this PR do?

To better track how the ingress-nginx controller is doing, we should export metrics from it to Prometheus/Grafana.

Any specific requests for how the PR should be reviewed?

This is not yet rolled out to the cluster, so eyeball it (and the docs) and do a Task-based rollout while monitoring the effects.

There are some dashboards available which could be imported for easy checking of metrics.

Available metrics are listed here.

@achton achton requested review from hypesystem and kasperg May 2, 2024 14:42
Copy link
Contributor

@hypesystem hypesystem left a comment

Choose a reason for hiding this comment

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

Nice, rul det gerne ud i platformen og let's get it merged!

@hypesystem
Copy link
Contributor

Du må gerne køre denne ud at your convenience - men sig lige til på Zulip så jeg kan reenable alerts til Zulip :-)

@hypesystem
Copy link
Contributor

hypesystem commented Jun 26, 2024

This PR was in a bit of an unclear state because we talked about it on Zulip. Basically we'd like to scale up prometheus and loki to be confident in their ability to handle more data ingress before we execute and run this. So it's still waiting a bit until we get over the hump of getting all the libraries live.

@ITViking
Copy link
Contributor

are we in a good enough place to get this in now? @hypesystem and @achton

@achton
Copy link
Contributor Author

achton commented Sep 19, 2024

My impression is "yes", but I think you guys are better suited to answer that question.
I've resolved conflicts for ya :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants