Add succesfull requests metrics #85
Labels
No labels
P0
P1
P2
P3
good first issue
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-s3-gw#85
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Is your feature request related to a problem? Please describe.
We have metrics for total number of requests. For monitoring purposes it would be nice to distinguish number of successful metrics to estimate gateway healthiness based on request success rate.
Describe the solution you'd like
Add new metric that return number of successful requests. It should count requests with all return codes except 5** codes (please verify it @a.bogatyrev).
Describe alternatives you've considered
Add new metrics that return only failed requests.
Actually, we already have such metric
frostfs_s3_errors_total
. So we can move this issue to v0.28.0 and discuss it a little bit laterAs for v0.30 release, there are:
frostfs_s3_gw_statistic_requests_total
metric that counts all requestsfrostfs_s3_gw_statistic_errors_total
metric that counts all codes except 2** codesThis should be sufficient to calculate all successful requests by subtracting values of these metrics.