Merge pull request #102177 from atiratree/quotaunitscli

kubectl: show consistent unit format in quota describe
This commit is contained in:
Kubernetes Prow Robot 2021-07-09 16:49:18 -07:00 committed by GitHub
commit 303fe39e1c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 6 deletions

View File

@ -672,10 +672,13 @@ func describeQuota(resourceQuota *corev1.ResourceQuota) (string, error) {
msg := "%v\t%v\t%v\n"
for i := range resources {
resource := resources[i]
hardQuantity := resourceQuota.Status.Hard[resource]
usedQuantity := resourceQuota.Status.Used[resource]
w.Write(LEVEL_0, msg, resource, usedQuantity.String(), hardQuantity.String())
resourceName := resources[i]
hardQuantity := resourceQuota.Status.Hard[resourceName]
usedQuantity := resourceQuota.Status.Used[resourceName]
if hardQuantity.Format != usedQuantity.Format {
usedQuantity = *resource.NewQuantity(usedQuantity.Value(), hardQuantity.Format)
}
w.Write(LEVEL_0, msg, resourceName, usedQuantity.String(), hardQuantity.String())
}
return nil
})

View File

@ -4066,7 +4066,7 @@ func TestDescribeResourceQuota(t *testing.T) {
corev1.ResourceName(corev1.ResourceLimitsMemory): resource.MustParse("0G"),
corev1.ResourceName(corev1.ResourceMemory): resource.MustParse("0G"),
corev1.ResourceName(corev1.ResourceRequestsCPU): resource.MustParse("0"),
corev1.ResourceName(corev1.ResourceRequestsMemory): resource.MustParse("0G"),
corev1.ResourceName(corev1.ResourceRequestsMemory): resource.MustParse("1000Ki"),
},
},
})
@ -4076,7 +4076,7 @@ func TestDescribeResourceQuota(t *testing.T) {
if err != nil {
t.Errorf("unexpected error: %v", err)
}
expectedOut := []string{"bar", "foo", "limits.cpu", "2", "limits.memory", "2G", "requests.cpu", "1", "requests.memory", "1G"}
expectedOut := []string{"bar", "foo", "limits.cpu", "2", "limits.memory", "2G", "requests.cpu", "1", "requests.memory", "1024k", "1G"}
for _, expected := range expectedOut {
if !strings.Contains(out, expected) {
t.Errorf("expected to find %q in output: %q", expected, out)