-
Notifications
You must be signed in to change notification settings - Fork 21
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
Plot memory data point #148
base: main
Are you sure you want to change the base?
Conversation
@abraham2512 can you rebase this? |
25dae62
to
dff5c04
Compare
@vishnuchalla thanks vishnu! Resolved conflicts and pushed. |
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.
@abraham2512 This change is mixing up both CPU and memory metrics in the UI. Would you mind taking memory usage to a separate panel and also defining a threshold to it?
Thanks ... I had the same reaction initially, and I thought I'd commented ... but apparently I decided to wait to see what you or Vicente said, since you've got vastly greater "cultural experience" with the dashboard. 😆 |
Type of change
Description
This change is intended to include memory usage data from splunk in the cpu_util graph plot
Related Tickets & Documents
Jira issue key:
Checklist before requesting a review
Testing