-
Notifications
You must be signed in to change notification settings - Fork 124
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
feat: mvtx gauge metrics support #2259
Conversation
veds-g
commented
Dec 5, 2024
•
edited
Loading
edited
![image](https://private-user-images.githubusercontent.com/49195734/394328776-2a6016a4-8bc7-4485-ad73-0a946d76ceef.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk1ODk4OTQsIm5iZiI6MTczOTU4OTU5NCwicGF0aCI6Ii80OTE5NTczNC8zOTQzMjg3NzYtMmE2MDE2YTQtOGJjNy00NDg1LWFkNzMtMGE5NDZkNzZjZWVmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjE1VDAzMTk1NFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPThmZGQ4YTQ2YjAzOTRhYWQ3M2EzMTE4MzU1Mzk3Yzk1ODJjMmQxZWM2MDk2YTdiMGFhNTc3Mzg2MDE4ZjVhMTcmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.KpjEL6Eu8fUzhEG3_DuJoUp4bEgTEuPBdfxErihGb24)
title: Pending Messages Lag | ||
description: This query is the total number of pending messages for the mono vertex | ||
expr: | | ||
$metric_name{$filters} |
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.
will be adding support for operation like max, min or avg in query expression.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2259 +/- ##
==========================================
- Coverage 65.60% 65.51% -0.10%
==========================================
Files 346 346
Lines 42373 42373
==========================================
- Hits 27799 27759 -40
- Misses 13499 13544 +45
+ Partials 1075 1070 -5 ☔ View full report in Codecov by Sentry. |
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.
codegen is failing, also please add unit tests to cover gauge metrics.
...partials/Pods/partials/PodDetails/partials/Metrics/partials/common/FiltersDropdown/index.tsx
Show resolved
Hide resolved
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.
you can use make manifests so that changes are reflected in other configs. That may resolve codegen issue
Signed-off-by: veds-g <[email protected]>
Signed-off-by: veds-g <[email protected]>
9d27795
to
f626686
Compare
Signed-off-by: veds-g <[email protected]>
f626686
to
bd74a37
Compare
Signed-off-by: veds-g <[email protected]>