Skip to content
This repository has been archived by the owner on Jul 7, 2023. It is now read-only.

Store metricses in milli, but does not seem to be retrieved in milli #24

Closed
shu-mutou opened this issue Jan 30, 2020 · 2 comments · Fixed by #25
Closed

Store metricses in milli, but does not seem to be retrieved in milli #24

shu-mutou opened this issue Jan 30, 2020 · 2 comments · Fixed by #25

Comments

@shu-mutou
Copy link

shu-mutou commented Jan 30, 2020

Storing metricses in milli as follows:
https://github.com/kubernetes-sigs/dashboard-metrics-scraper/blob/master/pkg/database/database.go#L42
https://github.com/kubernetes-sigs/dashboard-metrics-scraper/blob/master/pkg/database/database.go#L56

But retriving and modified in pkg/api/dashboard/dashboard.go as follows:

I could not test these things, but I guess this might cause issue like follow. kubernetes/dashboard#4827

@shu-mutou
Copy link
Author

/cc @jeefy

@floreks
Copy link
Contributor

floreks commented Jan 30, 2020

All the metrics that are stored in DB by the scraper are in milli (pods/nodes).

https://github.com/kubernetes-sigs/dashboard-metrics-scraper/blob/master/pkg/database/database.go#L42

Since for CPU we do want milli and for memory we prefer raw bytes so we can transform it to Mi/Gi or whatever on our own, it seems for me that everything is fine. The results are also consistent with what kubectl top nodes/pods shows. The only issue is on our side (dashboard).

I've found also some issues on the scraper side. Some values were stripped down or changed in the wrong way.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants