Skip to content
This repository has been archived by the owner on Jul 26, 2022. It is now read-only.

feat: basic metrics #147

Merged
merged 2 commits into from
Sep 27, 2019
Merged

feat: basic metrics #147

merged 2 commits into from
Sep 27, 2019

Conversation

moolen
Copy link
Member

@moolen moolen commented Aug 11, 2019

What this PR does / why we need it:
This PR Introduces a HTTP Server component that serves Prometheus metrics. We want metrics for this application to gain insights into the app and its integration with 3rd party APIs:

  • do we successfully sync external secrets?
  • if not what secrets, namespaces or backends are affected

This PR adds the sync_calls metric with labels name|namespace|backend|status that provide visibility for the above criteria.

Copy link
Contributor

@silasbw silasbw left a comment

Choose a reason for hiding this comment

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

awesome 🌶️ 🌶️ 🌶️

minor comments

lib/metrics-server.js Outdated Show resolved Hide resolved
lib/metrics-server.js Outdated Show resolved Hide resolved
@silasbw silasbw requested a review from keweilu August 21, 2019 15:48
Signed-off-by: Moritz Johner <[email protected]>
@moolen
Copy link
Member Author

moolen commented Aug 21, 2019

thank you <3
I addressed the comments & rebased onto master

README.md Show resolved Hide resolved
@moolen
Copy link
Member Author

moolen commented Aug 27, 2019

@silasbw @keweilu ptal

Copy link
Contributor

@keweilu keweilu left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

@keweilu keweilu merged commit ded6d31 into external-secrets:master Sep 27, 2019
anarcher pushed a commit to anarcher/kubernetes-external-secrets that referenced this pull request Oct 30, 2019
* feat: basic metrics

Signed-off-by: Moritz Johner <[email protected]>

* chore: update chart docs & env vars
Flydiverny pushed a commit to Flydiverny/kubernetes-external-secrets that referenced this pull request Nov 3, 2019
* feat: basic metrics

Signed-off-by: Moritz Johner <[email protected]>

* chore: update chart docs & env vars
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants