Commit f59f4aec authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub

Merge pull request #32660 from mksalawa/fix_usage_printing

Automatic merge from submit-queue Fix printing container usage in kubectl top **What this PR does / why we need it**: Fix a bug in kubectl top, which showed the same value of usage for all containers in a pod. **Release note**: ```release-note NONE ```
parents 71b2ebbc 7fe19e0c
...@@ -110,8 +110,9 @@ func printSinglePodMetrics(out io.Writer, m *metrics_api.PodMetrics, printContai ...@@ -110,8 +110,9 @@ func printSinglePodMetrics(out io.Writer, m *metrics_api.PodMetrics, printContai
for _, res := range MeasuredResources { for _, res := range MeasuredResources {
podMetrics[res], _ = resource.ParseQuantity("0") podMetrics[res], _ = resource.ParseQuantity("0")
} }
var usage api.ResourceList
for _, c := range m.Containers { for _, c := range m.Containers {
var usage api.ResourceList
err := api.Scheme.Convert(&c.Usage, &usage, nil) err := api.Scheme.Convert(&c.Usage, &usage, nil)
if err != nil { if err != nil {
return err return err
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment