Merge pull request #272 from Catalysts/validate-name

validate if name is set

Author: Marko Mikulicic

pkg/kubecfg/diff.go

@@ -67,6 +67,10 @@ func (c DiffCmd) Run(apiObjects []*unstructured.Unstructured, out io.Writer) err
 			return err
 		}
 
+		if obj.GetName() == "" {
+			return fmt.Errorf("Error fetching one of the %s: it does not have a name set", utils.ResourceNameFor(c.Mapper, obj))
+		}
+
 		liveObj, err := client.Get(obj.GetName(), metav1.GetOptions{})
 		if err != nil && errors.IsNotFound(err) {
 			log.Debugf("%s doesn't exist on the server", desc)

pkg/kubecfg/validate.go

@@ -82,6 +82,9 @@ func (c ValidateCmd) Run(apiObjects []*unstructured.Unstructured, out io.Writer)
 			for _, err := range schema.Validate(obj) {
 				allErrs = append(allErrs, err)
 			}
+			if obj.GetName() == "" {
+				allErrs = append(allErrs, fmt.Errorf("An Object does not have a name set"))
+			}
 		}
 
 		for _, err := range allErrs {