sustainable-computing-io / kepler-operator

Kepler Operator
Apache License 2.0
25 stars 26 forks source link

feat(dashboard): enhance OpenShift dashboard #247

Closed jgbernalp closed 1 year ago

jgbernalp commented 1 year ago

Fixes: #246

https://issues.redhat.com/browse/OU-250

https://github.com/sustainable-computing-io/kepler-operator/assets/5461414/21c52a33-f013-4b65-be62-95e0d2f6b3a1

vprashar2929 commented 1 year ago

Quick question can we add units to the panel value like Watt or Killowatt? For eg. In the case of Total Energy Consumption panel the value we get is in Killowatt-hour

jgbernalp commented 1 year ago

@vprashar2929 yes, I added the units in the total energy consumption panel. Unfortunately other panels, like tables and line charts are limited on that

jgbernalp commented 1 year ago

Screenshot 2023-09-25 at 16 47 06

vimalk78 commented 1 year ago

@jgbernalp can we sort the top10 namespace table by values?

jgbernalp commented 1 year ago

@vimalk78 this is a known limitation, by default the table content is sorted based on the first column values, the values can be sorted clicking on the table header. I understand it will be important to sort so the user gets proper information. We could potentially raise this as a bug.

vprashar2929 commented 1 year ago

@jgbernalp Also can you move Total Power Consumption (W) in Namespace and Total Power Consumption (kWh per day) in Namespace panels below Total Power Consumption for Top 10 Namespaces (kWh per day)? Since these represent the aggregate power consumption I think it's better to put them first rather than bottom. Wdyt?

jgbernalp commented 1 year ago

@vprashar2929 yes makes sense, i've moved them.

fkargbo commented 1 year ago

With regards to Phase 1 of this initiative, looks good to me. Thanks!

jgbernalp commented 1 year ago

As discussed in our meeting of Sep 28th

vprashar2929 commented 1 year ago

Just one comment about moving Total Power Consumption (W) in Namespace and Total Power Consumption (kWh per day) in Namespace panels to the top in kepler-dashboard-namespace.json

jgbernalp commented 1 year ago

Changes were included by #258