pull

pkg/engine/mutation.go

@@ -29,7 +29,7 @@ func Mutate(policy kubepolicy.Policy, rawResource []byte, gvk metav1.GroupVersio
 			overlayPatches, err := ProcessOverlay(rule, rawResource, gvk)
 			if err != nil {
 				ri.Fail()
-				ri.Addf("Rule %s: Overlay application has failed, err %s.", rule.Name, err)
+				ri.Addf("overlay application has failed, err %v.", err)
 			} else {
 				ri.Addf("Rule %s: Overlay succesfully applied.", rule.Name)
 				//TODO: patchbytes -> string
@@ -44,7 +44,7 @@ func Mutate(policy kubepolicy.Policy, rawResource []byte, gvk metav1.GroupVersio
 			if len(errs) > 0 {
 				ri.Fail()
 				for _, err := range errs {
-					ri.Addf("Rule %s: Patches application has failed, err %s.", rule.Name, err)
+					ri.Addf("patches application has failed, err %v.", err)
 				}
 			} else {
 				ri.Addf("Rule %s: Patches succesfully applied.", rule.Name)

pkg/engine/validation.go

@@ -41,7 +41,7 @@ func Validate(policy kubepolicy.Policy, rawResource []byte, gvk metav1.GroupVers
 		err := validateResourceWithPattern(resource, rule.Validation.Pattern)
 		if err != nil {
 			ri.Fail()
-			ri.Addf("Rule %s: Validation has failed, err %s.", rule.Name, err)
+			ri.Addf("validation has failed, err %v.", err)
 		} else {
 			ri.Addf("Rule %s: Validation succesfully.", rule.Name)
 

pkg/webhooks/mutation.go

@@ -58,7 +58,7 @@ func (ws *WebhookServer) HandleMutation(request *v1beta1.AdmissionRequest) *v1be
 		if !policyInfo.IsSuccessful() {
 			glog.Infof("Failed to apply policy %s on resource %s/%s", policy.Name, rname, rns)
 			for _, r := range ruleInfos {
-				glog.Warning(r.Msgs)
+				glog.Warningf("%s: %s\n", r.Name, r.Msgs)
 			}
 		} else {
 			// TODO

pkg/webhooks/validation.go

@@ -65,7 +65,7 @@ func (ws *WebhookServer) HandleValidation(request *v1beta1.AdmissionRequest) *v1
 		if !policyInfo.IsSuccessful() {
 			glog.Infof("Failed to apply policy %s on resource %s/%s", policy.Name, rname, rns)
 			for _, r := range ruleInfos {
-				glog.Warning(r.Msgs)
+				glog.Warningf("%s: %s\n", r.Name, r.Msgs)
 			}
 		} else {
 			// CleanUp Violations if exists