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

instrumentation: add request duration metric #12

Merged
merged 2 commits into from
May 2, 2023

Conversation

ashwanthgoli
Copy link

adds gomemcache_request_duration_seconds to track the time spent in performing cache operations.
It's partitioned by the server addr to enable visibility at per-server level

@CLAassistant
Copy link

CLAassistant commented May 2, 2023

CLA assistant check
All committers have signed the CLA.

@ashwanthgoli ashwanthgoli marked this pull request as ready for review May 2, 2023 11:58
Copy link

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

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

LGTM!

Signed-off-by: Ashwanth Goli <iamashwanth@gmail.com>
Signed-off-by: Ashwanth Goli <iamashwanth@gmail.com>
@dannykopping dannykopping merged commit 851158f into master May 2, 2023
3 checks passed
@dannykopping dannykopping deleted the add-instrumentation branch May 2, 2023 12:20
56quarters added a commit that referenced this pull request Sep 13, 2023
The instrumentation adds a dependency on the Prometheus client
and didn't end up being used by Loki (or any other database).
Removing so this client can be used with only the standard
library.

Related #12

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
56quarters added a commit that referenced this pull request Sep 14, 2023
The instrumentation adds a dependency on the Prometheus client
and didn't end up being used by Loki (or any other database).
Removing so this client can be used with only the standard
library.

Related #12

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants