-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
cAdvisor web UI port has been deprecated since v1.10. This patch updates the description about it. #11426
Conversation
62772c7
to
d93abc6
Compare
Deploy preview for kubernetes-io-master-staging ready! Built with commit 62772c7 https://deploy-preview-11426--kubernetes-io-master-staging.netlify.com |
Deploy preview for kubernetes-io-master-staging ready! Built with commit 0308221 https://deploy-preview-11426--kubernetes-io-master-staging.netlify.com |
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.
Version support isn't clear in the original. Copyedits make the issue more obvious. Please clarify!
|
||
![cAdvisor](/images/docs/cadvisor.png) | ||
|
||
In version 1.10 and later, the Advisor module does not provide this UI. Instead, you can [run cAdvisor as DaemonSet](https://github.com/google/cadvisor/tree/master/deploy/kubernetes). |
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.
See also line 55. The original is unclear (before 1.10 and after 1.10 -- but what about IN 1.10?). Are lines 55 and 59 now correct? They are clearer, but the original could also have meant that the UI was available in 1.10 but not in 1.11. Please specify either in new commit or in comment. Thanks!
…te the description about it.
2ac46ea
to
16de424
Compare
@Bradamant3 What do you think this time |
@Bradamant3 friendly ping |
@Bradamant3 PTAL |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: tengqm The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…tes the description about it. (kubernetes#11426) * cAdvisor web UI port has been deprecated since v1.10. This patch update the description about it. * Update resource-usage-monitoring.md * Update resource-usage-monitoring.md
…tes the description about it. (kubernetes#11426) * cAdvisor web UI port has been deprecated since v1.10. This patch update the description about it. * Update resource-usage-monitoring.md * Update resource-usage-monitoring.md
…tes the description about it. (kubernetes#11426) * cAdvisor web UI port has been deprecated since v1.10. This patch update the description about it. * Update resource-usage-monitoring.md * Update resource-usage-monitoring.md
…tes the description about it. (kubernetes#11426) * cAdvisor web UI port has been deprecated since v1.10. This patch update the description about it. * Update resource-usage-monitoring.md * Update resource-usage-monitoring.md
…tes the description about it. (kubernetes#11426) * cAdvisor web UI port has been deprecated since v1.10. This patch update the description about it. * Update resource-usage-monitoring.md * Update resource-usage-monitoring.md
cAdvisor web UI port has been deprecated since v1.10. This patch updates the description about it.
ref:kubernetes/kubernetes#65707