-
Notifications
You must be signed in to change notification settings - Fork 814
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
Added cortex_distributor_latest_ingested_sample_timestamp_seconds metric #2371
Added cortex_distributor_latest_ingested_sample_timestamp_seconds metric #2371
Conversation
adding tests now... |
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.
The change LGTM (I just left a nit). Just a confirmation: do we want the last seen or the max seen so far?
I think we want last seen, so that we know what Prometheus is sending at the moment. |
Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
What this PR does: This PR adds
cortex_distributor_latest_seen_sample_timestamp_seconds
with latest timestamp of sample sent to Cortex. Useful for tracking how far behind prometheus servers are from Cortex point of view.Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]