Better logging for output-only problems.

omegaup · May 7, 2019 · 6c4e301 · 6c4e301
1 parent fbdb74a
commit 6c4e301
Showing 1 changed file with 19 additions and 5 deletions.
diff --git a/runner/runner.go b/runner/runner.go
@@ -783,8 +783,9 @@ func Grade(
 				if file, ok := outputOnlyFiles[outName]; ok {
 					if err := ioutil.WriteFile(outPath, []byte(file.contents), 0644); err != nil {
 						ctx.Log.Error(
-							"failed to run",
+							"failed to write output file contents",
 							"case", caseData.Name,
+							"path", outPath,
 							"err", err,
 						)
 					}
@@ -796,14 +797,23 @@ func Grade(
 					}
 					if err := ioutil.WriteFile(metaPath, []byte("status:0"), 0644); err != nil {
 						ctx.Log.Error(
-							"failed to run "+caseData.Name,
+							"failed to write meta file",
+							"case", caseData.Name,
+							"path", metaPath,
 							"err", err,
 						)
 					}
 				} else {
+					ctx.Log.Error(
+						"missing an output file",
+						"case", caseData.Name,
+						"path", outPath,
+					)
 					if err := ioutil.WriteFile(outPath, []byte{}, 0644); err != nil {
 						ctx.Log.Error(
-							"failed to run "+caseData.Name,
+							"failed to write output file",
+							"case", caseData.Name,
+							"path", outPath,
 							"err", err,
 						)
 					}
@@ -812,15 +822,19 @@ func Grade(
 					}
 					if err := ioutil.WriteFile(metaPath, []byte("status:1"), 0644); err != nil {
 						ctx.Log.Error(
-							"failed to run "+caseData.Name,
+							"failed to write meta file",
+							"case", caseData.Name,
+							"path", metaPath,
 							"err", err,
 						)
 					}
 				}
 				errPath := path.Join(runRoot, errName)
 				if err := ioutil.WriteFile(errPath, []byte{}, 0644); err != nil {
 					ctx.Log.Error(
-						"failed to run "+caseData.Name,
+						"failed to write err file",
+						"case", caseData.Name,
+						"path", metaPath,
 						"err", err,
 					)
 				}