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
Remove redundant prometheus data points #5934
Conversation
Removed field minio_http_requests_total as it was redundant with minio_http_requests_duration_seconds_count Also removed field minio_server_start_time_seconds as it was redundant with process_start_time_seconds
Codecov Report
@@ Coverage Diff @@
## master #5934 +/- ##
=========================================
+ Coverage 59.57% 59.6% +0.03%
=========================================
Files 214 214
Lines 30786 30772 -14
=========================================
+ Hits 18342 18343 +1
+ Misses 10899 10881 -18
- Partials 1545 1548 +3
Continue to review full report at Codecov.
|
@@ -190,8 +190,6 @@ func (st *HTTPStats) updateStats(r *http.Request, w *httpResponseRecorder, durat | |||
st.successDELETEs.Duration.Add(durationSecs) | |||
} | |||
} | |||
// Increment the prometheus http request count with appropriate label | |||
httpRequests.With(prometheus.Labels{"request_type": r.Method}).Inc() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nitisht I am not sure how number of requests count is redundant with requests duration, can you elaborate ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM & works as expected
Description
Removed field minio_http_requests_total as it was redundant with
minio_http_requests_duration_seconds_count
Also removed field minio_server_start_time_seconds as it was
redundant with process_start_time_seconds
Motivation and Context
This PR removes couple of redundant fields in Prometheus data exported by Minio server.
How Has This Been Tested?
Manually
Types of changes
Checklist:
mint
PR # here: )