-
Notifications
You must be signed in to change notification settings - Fork 97
Add troubleshooting of vm metrics missing #958
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
Conversation
Signed-off-by: Jian Wang <[email protected]>
|
martindekov
left a comment
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.
Looks good Jian, I like the exact sequence, left minor comment regarding a place where it's ambiguous what we wait for, but apart from that no other comments, thanks!
jillian-maroket
left a comment
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.
Review done. I based some of my suggestions on @ihcsim 's.
Signed-off-by: Jian Wang <[email protected]>
|
@jillian-maroket @ihcsim @martindekov Thanks for your kind review, the comments have been addressed, please take a new look on the updated PR. |
martindekov
left a comment
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 Jian
Problem:
Solution:
Add troubleshooting of vm metrics missing
Related Issue(s):
harvester/harvester#9674
Test plan:
Additional documentation or context