Closed ichekrygin closed 9 months ago
The committers listed above are authorized under a signed CLA.
This issue is currently awaiting triage.
If usage-metrics-collector contributors determine this is a relevant issue, they will accept it by applying the triage/accepted
label and provide further guidance.
The triage/accepted
label can be added by org members by writing /triage accepted
in a comment.
Hi @ichekrygin. Thanks for your PR.
I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
/ok-to-test
/approve /lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: campoy, ichekrygin, ndipebot
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest
/lgtm
Replace the CAdvisor client with the HttpClient analog.
Kubernetes kubelet does not expose the CAdvsor RESTFull API endpoints (as of 1.16 or so). Thus, we cannot use the cadvisor's "native" client to export metrics 😔. Instead, Kubelet exposes cadvisor metrics via the "/metrics/cadvisor" endpoint, which could be "scraped" to collect metrics in the prometheus format.
This PR replaces cadvisor client with http.client alternative to collect cadvisor metrics via scraping kubelet's "/metrics/cadvisor".