From 7b0d495c677bc3f71f13c45f0b00a36d297adcd0 Mon Sep 17 00:00:00 2001 From: Serguei Dmitriev <65043014+sergueidmitriev@users.noreply.github.com> Date: Thu, 17 Dec 2020 12:18:00 -0500 Subject: [PATCH] Make lint error message more specific the original error message does not specify where 'name' object belongs too, it's confusing to troubleshoot the charts relying only on lint console output Signed-off-by: sergueidmitriev --- pkg/lint/rules/template.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/lint/rules/template.go b/pkg/lint/rules/template.go index 0bb9f8671..ec5317c2f 100644 --- a/pkg/lint/rules/template.go +++ b/pkg/lint/rules/template.go @@ -201,12 +201,12 @@ func validateYamlContent(err error) error { func validateMetadataName(obj *K8sYamlStruct) error { if len(obj.Metadata.Name) == 0 || len(obj.Metadata.Name) > 253 { - return fmt.Errorf("object name must be between 0 and 253 characters: %q", obj.Metadata.Name) + return fmt.Errorf("metadata object 'name' must be between 0 and 253 characters: %q", obj.Metadata.Name) } // This will return an error if the characters do not abide by the standard OR if the // name is left empty. if err := chartutil.ValidateMetadataName(obj.Metadata.Name); err != nil { - return errors.Wrapf(err, "object name does not conform to Kubernetes naming requirements: %q", obj.Metadata.Name) + return errors.Wrapf(err, "metadata object 'name' does not conform to Kubernetes naming requirements: %q", obj.Metadata.Name) } return nil }