Grafana Dashboard

[Stono]

This PR containers a Grafana dashboard for the metrics in #2726

It contains:

• Ability to filter by Namespace, Controller Class and Controller
• Visbility of Request Volume, connections, success rates, config reloads and configs out of sync.
• Network IO pressure, memory and CPU use
• Ingress P50, P95 and P99 percentile response times with IN/OUT throughput
• SSL certificate expiry
• Annotational overlays to show when config reloads happened

It looks like this:

[jpds]

Can you please make the datasource for this dashboard a variable? I have two prometheus servers as I run them in different AZs.

[codecov-io]

Codecov Report

Merging #2771 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2771   +/-   ##
=======================================
  Coverage   40.87%   40.87%           
=======================================
  Files          73       73           
  Lines        5108     5108           
=======================================
  Hits         2088     2088           
  Misses       2731     2731           
  Partials      289      289

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bdd2c5e...1cb52d2. Read the comment docs.

[jpds]

Also, on the default scrape interval of 60s, a lot of the charts don't display.

[jpds]

A metric I've found useful is:

sum(rate(nginx_ingress_controller_requests[2m])) by (ingress, status)

[Stono]

I can't really adjust it for everyone's use case. You're going to have to make tweaks for your environment when you import it.

…

On Wed, 11 Jul 2018, 3:00 pm Codecov, ***@***.***> wrote: Codecov <https://codecov.io/gh/kubernetes/ingress-nginx/pull/2771?src=pr&el=h1> Report Merging #2771 <https://codecov.io/gh/kubernetes/ingress-nginx/pull/2771?src=pr&el=desc> into master <https://codecov.io/gh/kubernetes/ingress-nginx/commit/bdd2c5e3be9c1d431ca5f743591f1520cca7d9ce?src=pr&el=desc> will *not change* coverage. The diff coverage is n/a. [image: Impacted file tree graph] <https://codecov.io/gh/kubernetes/ingress-nginx/pull/2771?src=pr&el=tree> @@ Coverage Diff @@## master #2771 +/- ## ======================================= Coverage 40.87% 40.87% ======================================= Files 73 73 Lines 5108 5108 ======================================= Hits 2088 2088 Misses 2731 2731 Partials 289 289 ------------------------------ Continue to review full report at Codecov <https://codecov.io/gh/kubernetes/ingress-nginx/pull/2771?src=pr&el=continue> . *Legend* - Click here to learn more <https://docs.codecov.io/docs/codecov-delta> Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov <https://codecov.io/gh/kubernetes/ingress-nginx/pull/2771?src=pr&el=footer>. Last update bdd2c5e...1cb52d2 <https://codecov.io/gh/kubernetes/ingress-nginx/pull/2771?src=pr&el=lastupdated>. Read the comment docs <https://docs.codecov.io/docs/pull-request-comments> . — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2771 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABaviaNGk9JuSw3zb2Z8t7QdgEaa5lyeks5uFgUNgaJpZM4VLJp1> .

[aledbf]

@Stono can you check the name of the metrics (s/seconds/milliseconds) and then we are ready to merge

[aledbf]

@jpds let's get this merge and then we can improve the dashboard (PRs are always welcomed)

[aledbf]

/approve
/lgtm

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aledbf, Stono

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [aledbf]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[erik-stephens]

Not sure if this is "new issue" worthy. Wanted to mention that exporting dashboards from grafana automatically provides a facility for making the data sources portable (at least with the grafana 5.x that we're using). I don't know how this was created to not have those templated. Will trip others up when trying to import it.

[jmreicha]

Getting a Datasoure named Prometheus was not found error in Grafana (5.1.3) when I try to import this dashboard.