Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

In ClusterStamp.json in saved searches section "traefik_entrypoint_requests_total" is used where as we are not using Traefik ingress #26

Closed
nitinkhandelwal26 opened this issue Jul 28, 2021 · 1 comment · Fixed by #34
Closed

In ClusterStamp.json in saved searches section "traefik_entrypoint_requests_total" is used where as we are not using Traefik ingress #26

nitinkhandelwal26 opened this issue Jul 28, 2021 · 1 comment · Fixed by #34
Assignees
@ckittel
Labels
bug Something isn't working

Comments

@nitinkhandelwal26
Copy link

Hello Team,

I just noticed, in clusterstamp.v2.json if I go to the saved searches section at line 888, in query "traefik_entrypoint_requests_total" filter is used.

Please check it's still relevant to this implementation? as we are using NGINX ingress in this.

Regards,
@ckittel ckittel self-assigned this Aug 5, 2021
@ckittel ckittel added the bug Something isn't working label Aug 5, 2021
@ckittel
Copy link
Member

ckittel commented Aug 5, 2021 •
edited
Loading

@nitinkhandelwal26 -- Thank you, that was indeed an oversight after the ingress change. I've captured this in our internal backlog (Ref 468514), and have scheduled that change for about two weeks from now (next sprint). I'll leave this issue open until it is resolved though. If you happen to figure out what the right Prometheus metric is and rewrite that query yourself, we'd be glad to take your contribution. No big deal though if you don't/can't.

@ckittel ckittel mentioned this issue Aug 23, 2021
Merged
ckittel added a commit that referenced this issue Aug 23, 2021
@ckittel
Add prometheous metric scraping for ingress (#34)
92faa64 
* Adds Prometheus metrics collection on the ingress controller
* Updates the Log Analytics query that used to be pulling Traefik metrics. This Fixes #26.
* Remove old osm-system namespace (from pre-addon days) from OMS agent config.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@ckittel ckittel

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ingress Controller: Add Metrics & Update Query mspnp/aks-baseline-regulated
2 participants
@ckittel @nitinkhandelwal26