Skip to content

Conversation

karlskewes
Copy link
Collaborator

@karlskewes karlskewes commented Aug 4, 2021

closes #3

controller_invocations_total is an enhanced version of Springs
http_server_requests metric with nicer label formatting and other
differences.

okhttp_requests_seconds_{count|max|sum} is for outbound requests.

As all Java services expose these metrics pull them out to a separate
file that we can import accordingly.

@karlskewes
Copy link
Collaborator Author

Note Clouddriver also has some controller_invocations_total{...account=~"$Account"}.

I've left these for now.

Ultimately we should add recording rules to reduce dashboard query time load on Prometheus.

`controller_invocations_total` is an enhanced version of Springs
`http_server_requests` metric with nicer label formatting and other
differences.

`okhttp_requests_seconds_{count|max|sum}` is for outbound requests.

As all Java services expose these metrics pull them out to a separate
file that we can import accordingly.
@karlskewes karlskewes merged commit d0354c1 into uneeq-oss:main Aug 4, 2021
@karlskewes karlskewes deleted the 3-http branch August 4, 2021 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Standardise HTTP Request Metrics
1 participant