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

Prometheus metrics for the controller #705

Closed
4 tasks
hbagdi opened this issue May 22, 2020 · 4 comments
Closed
4 tasks

Prometheus metrics for the controller #705

hbagdi opened this issue May 22, 2020 · 4 comments
Assignees
Labels
area/feature New feature or request

Comments

@hbagdi
Copy link
Member

hbagdi commented May 22, 2020

Problem statement:

As a user I cannot monitor the performance and errors of individual operations performed by KIC.

Proposed solution:

Use the existing Prometheus endpoint exposed by KIC 2.0 (created by the controller-runtime library) to expose additional metrics:

  • [type: histogram] Duration of the last sync to Kong
    • with a metric field mode="deck-sync", mode="post-config" to provide distinction between these different operations
    • with a metric field result="success", result="error"
  • [type: counter] Parsing error count
  • [type: counter] "sync to Kong" count
    • with a metric field mode="deck-sync", mode="post-config" to provide distinction between these different operations
    • with a metric field result="success", result="error"

Acceptance criteria:

  • KIC 2.0 exposes the metrics defined above on the --metrics-bind-addr (which is already present in KIC 2.0)
  • Tests for the metrics above - guaranteeing that errors are accounted for appropriately and that the distribution metrics contain valid data
  • Documentation for the metrics - both in prometheus metrics metadata and in docs.konghq.com
  • The solution is forward-compatible with KIC sending configuration to multiple proxies.
@hbagdi hbagdi added the area/feature New feature or request label May 22, 2020
@hbagdi hbagdi added this to the 1.3.0 milestone Sep 11, 2020
@stale
Copy link

stale bot commented Mar 20, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Will be closed unless advocated for within 7 days label Mar 20, 2021
@stale stale bot closed this as completed Mar 27, 2021
@hbagdi hbagdi reopened this Mar 29, 2021
@stale stale bot removed the stale Will be closed unless advocated for within 7 days label Mar 29, 2021
@shaneutt
Copy link
Contributor

Double checked with Harry, this should be a post-2.0 item.

@shaneutt shaneutt modified the milestones: 1.3.0, POST KIC 2.0 Jun 28, 2021
@mflendrich mflendrich modified the milestones: POST KIC 2.0, Blockers for cutting KIC 2.0-beta.1 Jul 7, 2021
@ccfishk ccfishk self-assigned this Jul 8, 2021
@ccfishk ccfishk removed the onboarding label Jul 8, 2021
@ccfishk
Copy link
Contributor

ccfishk commented Jul 28, 2021

@mflendrich do you want to refer [type: distribution] as https://prometheus.io/docs/concepts/metric_types/#histogram ?
Kindly apply the prometheus term.

@mflendrich
Copy link
Contributor

@mflendrich do you want to refer [type: distribution] as https://prometheus.io/docs/concepts/metric_types/#histogram ?
Kindly apply the prometheus term.

Yes, when I said "distribution" I meant "histogram".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/feature New feature or request
Projects
None yet
4 participants