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 <serguei.dmitriev@ssense.com>
pull/9143/head
Serguei Dmitriev 5 years ago committed by sergueidmitriev
parent bed1a42a39
commit 7b0d495c67

@ -201,12 +201,12 @@ func validateYamlContent(err error) error {
func validateMetadataName(obj *K8sYamlStruct) error { func validateMetadataName(obj *K8sYamlStruct) error {
if len(obj.Metadata.Name) == 0 || len(obj.Metadata.Name) > 253 { 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 // This will return an error if the characters do not abide by the standard OR if the
// name is left empty. // name is left empty.
if err := chartutil.ValidateMetadataName(obj.Metadata.Name); err != nil { 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 return nil
} }

Loading…
Cancel
Save