Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Script to gather Kruize metrics #967

Merged
merged 11 commits into from
Nov 3, 2023
Merged

Script to gather Kruize metrics #967

merged 11 commits into from
Nov 3, 2023

Conversation

kusumachalasani
Copy link
Contributor

Python script to gather kruize metrics

@kusumachalasani
script to gather kruize_metrics
39c7127 
Signed-off-by: kusumachalasani <[email protected]>
@kusumachalasani
changes to headers
d5f6af2 
Signed-off-by: kusumachalasani <[email protected]>
@kusumachalasani
add missing metrics
9635984 
Signed-off-by: kusumachalasani <[email protected]>
@kusumachalasani
Add resource metrics
d12fd8b 
Signed-off-by: kusumachalasani <[email protected]>
@kusumachalasani kusumachalasani changed the base branch from mvp_demo to remote_monitoring September 29, 2023 05:15
@kusumachalasani kusumachalasani changed the base branch from remote_monitoring to mvp_demo September 29, 2023 05:36
@dinogun dinogun added this to the Kruize 0.0.20_rm Release milestone Nov 2, 2023
@kusumachalasani
Add max metrics
b6f0a08 
Signed-off-by: kusumachalasani <[email protected]>
@dinogun
Copy link
Contributor

dinogun commented Nov 3, 2023

@kusumachalasani The server parameter needs to be explained more ... it is still not clear

@kusumachalasani
Update usage info
4e68732 
Signed-off-by: kusumachalasani <[email protected]>
@kusumachalasani
add appropriate help message
ae3e467 
Signed-off-by: kusumachalasani <[email protected]>
@kusumachalasani
relocate
b325afd 
Signed-off-by: kusumachalasani <[email protected]>
dinogun
dinogun approved these changes Nov 3, 2023
View reviewed changes
Copy link
Contributor

@dinogun dinogun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dinogun dinogun merged commit 80e4f5f into kruize:mvp_demo Nov 3, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dinogun dinogun dinogun approved these changes

@chandrams chandrams Awaiting requested review from chandrams

Assignees

@kusumachalasani kusumachalasani

Labels
remote_monitoring
Projects
Monitoring
Status: Done
Milestone
Kruize 0.0.20_rm Release
Development

Successfully merging this pull request may close these issues.
3 participants
@kusumachalasani @dinogun @rbadagandi1