diff --git a/pkg/fieldpath/fieldpath.go b/pkg/fieldpath/fieldpath.go index bd4d2f371d0..ff707e05dce 100644 --- a/pkg/fieldpath/fieldpath.go +++ b/pkg/fieldpath/fieldpath.go @@ -63,7 +63,7 @@ func ExtractFieldPathAsString(obj interface{}, fieldPath string) (string, error) return accessor.GetNamespace(), nil } - return "", fmt.Errorf("Unsupported fieldPath: %v", fieldPath) + return "", fmt.Errorf("unsupported fieldPath: %v", fieldPath) } // TODO: move the functions below to pkg/api/util/resources @@ -146,7 +146,7 @@ func InternalExtractContainerResourceValue(fs *api.ResourceFieldSelector, contai return convertResourceMemoryToString(container.Resources.Requests.Memory(), divisor) } - return "", fmt.Errorf("Unsupported container resource : %v", fs.Resource) + return "", fmt.Errorf("unsupported container resource : %v", fs.Resource) } // findContainerInPod finds a container by its name in the provided pod @@ -159,7 +159,7 @@ func findContainerInPod(pod *v1.Pod, containerName string) (*v1.Container, error return nil, fmt.Errorf("container %s not found", containerName) } -// convertResourceCPUTOString converts cpu value to the format of divisor and returns +// convertResourceCPUToString converts cpu value to the format of divisor and returns // ceiling of the value. func convertResourceCPUToString(cpu *resource.Quantity, divisor resource.Quantity) (string, error) { c := int64(math.Ceil(float64(cpu.MilliValue()) / float64(divisor.MilliValue()))) diff --git a/pkg/fieldpath/fieldpath_test.go b/pkg/fieldpath/fieldpath_test.go index a7e74b1f61e..3e49a3d672e 100644 --- a/pkg/fieldpath/fieldpath_test.go +++ b/pkg/fieldpath/fieldpath_test.go @@ -100,7 +100,7 @@ func TestExtractFieldPathAsString(t *testing.T) { Namespace: "object-namespace", }, }, - expectedMessageFragment: "Unsupported fieldPath", + expectedMessageFragment: "unsupported fieldPath", }, } @@ -109,13 +109,13 @@ func TestExtractFieldPathAsString(t *testing.T) { if err != nil { if tc.expectedMessageFragment != "" { if !strings.Contains(err.Error(), tc.expectedMessageFragment) { - t.Errorf("%v: Unexpected error message: %q, expected to contain %q", tc.name, err, tc.expectedMessageFragment) + t.Errorf("%v: unexpected error message: %q, expected to contain %q", tc.name, err, tc.expectedMessageFragment) } } else { t.Errorf("%v: unexpected error: %v", tc.name, err) } } else if e := tc.expectedValue; e != "" && e != actual { - t.Errorf("%v: Unexpected result; got %q, expected %q", tc.name, actual, e) + t.Errorf("%v: unexpected result; got %q, expected %q", tc.name, actual, e) } } }