Merge pull request #275 from dims/bailout-after-printing-errors

k8s-ci-robot · web-flow · commit f3ffbc9c0696 · 2021-12-06T08:00:32.000-08:00
Whoops! bailout after printing errors
diff --git a/cmd/validate-rules/main.go b/cmd/validate-rules/main.go
@@ -37,10 +37,10 @@ func main() {
 		}
 		errors := staging.EnsureStagingDirectoriesExist(rules)
 		if errors != nil {
-			glog.Fatalf("Invalid rules file %q", f)
 			for _, err := range errors {
 				glog.Errorf("Error : %s", err)
 			}
+			glog.Fatalf("Invalid rules file %q", f)
 		}
 		glog.Infof("validation successful")
 	}

Original file line number	Diff line number	Diff line change
`@@ -37,10 +37,10 @@ func main() {`
`37`	`37`	`}`
`38`	`38`	`errors := staging.EnsureStagingDirectoriesExist(rules)`
`39`	`39`	`if errors != nil {`
`40`		`- glog.Fatalf("Invalid rules file %q", f)`
`41`	`40`	`for _, err := range errors {`
`42`	`41`	`glog.Errorf("Error : %s", err)`
`43`	`42`	`}`
	`43`	`+ glog.Fatalf("Invalid rules file %q", f)`
`44`	`44`	`}`
`45`	`45`	`glog.Infof("validation successful")`
`46`	`46`	`}`