Merge pull request #159 from johanneswuerbach/cache-not-found

Cache missing schemas

kubeval/kubeval.go

@@ -174,13 +174,15 @@ func validateAgainstSchema(body interface{}, resource *ValidationResult, schemaC
 		schemaLoader := gojsonschema.NewReferenceLoader(schemaRef)
 		var err error
 		schema, err = gojsonschema.NewSchema(schemaLoader)
+		schemaCache[schemaRef] = schema
+
 		if err != nil {
-			if IgnoreMissingSchemas {
-				return []gojsonschema.ResultError{}, nil
-			}
-			return []gojsonschema.ResultError{}, fmt.Errorf("Failed initalizing schema %s: %s", schemaRef, err)
+			return handleMissingSchema(fmt.Errorf("Failed initalizing schema %s: %s", schemaRef, err))
 		}
-		schemaCache[schemaRef] = schema
+	}
+
+	if schema == nil {
+		return handleMissingSchema(fmt.Errorf("Failed initalizing schema %s: see first error", schemaRef))
 	}
 
 	// Without forcing these types the schema fails to load
@@ -202,6 +204,13 @@ func validateAgainstSchema(body interface{}, resource *ValidationResult, schemaC
 	return []gojsonschema.ResultError{}, nil
 }
 
+func handleMissingSchema(err error) ([]gojsonschema.ResultError, error) {
+	if IgnoreMissingSchemas {
+		return []gojsonschema.ResultError{}, nil
+	}
+	return []gojsonschema.ResultError{}, err
+}
+
 // NewSchemaCache returns a new schema cache to be used with
 // ValidateWithCache
 func NewSchemaCache() map[string]*gojsonschema.Schema {