forked from cortexproject/cortex
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add a new config and metric for reporting ruler query execution wall …
…time. (cortexproject#4317) * Add a new config and metric for reporting ruler query execution wall time. Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Spacing and PR number fixup Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Wrap the defer in a function to make it defer after the return rather than after the if block. Add a unit test to validate we're tracking time correctly. Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Use seconds for our duration rather than nanoseconds Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Review comment fixes Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Update config flag in the config docs Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Pass counter rather than counter vector for metrics query function Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Fix comment in MetricsQueryFunction Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Move query metric and log to separate function. Add log message for ruler query time. Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Update config file and change log to show this a per user metric Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * code review fixes Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * update log message for ruler query metrics Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Remove append and just use the array for key values in the log messag Signed-off-by: Tyler Reid <tyler.reid@grafana.com> * Add query-frontend component to front end log message Signed-off-by: Tyler Reid <tyler.reid@grafana.com> Signed-off-by: Alvin Lin <alvinlin@amazon.com>
- Loading branch information
1 parent
c3d412c
commit 51b2a85
Showing
7 changed files
with
68 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters