-
Notifications
You must be signed in to change notification settings - Fork 823
Fix cortex query seconds total #3589
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
Fix cortex query seconds total #3589
Conversation
Fixes cortexproject#3082 Signed-off-by: Wing924 <weihe924stephen@gmail.com>
Signed-off-by: Wing924 <weihe924stephen@gmail.com>
…oseconds. Signed-off-by: Wing924 <weihe924stephen@gmail.com>
3fb66d6
to
a91330e
Compare
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.
Good catch, Thanks!
@jtlisi Worth including in Cortex 1.6.0? |
Signed-off-by: Marco Pracucci <marco@pracucci.com>
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.
Thanks for the fix!
I think so. I'll make a note to cherry pick this for |
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
* Fixes `-config.expand-env` requires argument. Fixes #3082 Signed-off-by: Wing924 <weihe924stephen@gmail.com> * fix Signed-off-by: Wing924 <weihe924stephen@gmail.com> * Query-Frontend: cortex_query_seconds_total now return seconds not nanoseconds. Signed-off-by: Wing924 <weihe924stephen@gmail.com> * fix Signed-off-by: Wing924 <weihe924stephen@gmail.com> * Update CHANGELOG.md Signed-off-by: Marco Pracucci <marco@pracucci.com> Co-authored-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Jacob Lisi <jacob.t.lisi@gmail.com>
* Fixes `-config.expand-env` requires argument. Fixes #3082 Signed-off-by: Wing924 <weihe924stephen@gmail.com> * fix Signed-off-by: Wing924 <weihe924stephen@gmail.com> * Query-Frontend: cortex_query_seconds_total now return seconds not nanoseconds. Signed-off-by: Wing924 <weihe924stephen@gmail.com> * fix Signed-off-by: Wing924 <weihe924stephen@gmail.com> * Update CHANGELOG.md Signed-off-by: Marco Pracucci <marco@pracucci.com> Co-authored-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Jacob Lisi <jacob.t.lisi@gmail.com> Co-authored-by: Wei He <weihe924stephen@gmail.com> Co-authored-by: Marco Pracucci <marco@pracucci.com>
What this PR does:
bugfix: Query-Frontend:
cortex_query_seconds_total
now return seconds not nanoseconds.Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]