8015: check error when deleting temporary directory.
authorPeter Amstutz <peter.amstutz@curoverse.com>
Mon, 15 Feb 2016 21:58:34 +0000 (16:58 -0500)
committerPeter Amstutz <peter.amstutz@curoverse.com>
Mon, 15 Feb 2016 21:58:34 +0000 (16:58 -0500)
services/crunch-run/crunchrun.go

index 775b06d78e167110fc77a5628c79f9e2dd30f271..1b04bb4424e1e98c8bede6132c346f4c7e591a14 100644 (file)
@@ -429,6 +429,8 @@ func (runner *ContainerRunner) AttachLogs() (err error) {
 // WaitFinish waits for the container to terminate, capture the exit code, and
 // close the stdout/stderr logging.
 func (runner *ContainerRunner) WaitFinish() error {
+       runner.CrunchLog.Print("Waiting for container to finish")
+
        result := runner.Docker.Wait(runner.ContainerID)
        wr := <-result
        if wr.Error != nil {
@@ -516,7 +518,9 @@ func (runner *ContainerRunner) CleanupDirs() {
 
        for _, tmpdir := range runner.CleanupTempDir {
                rmerr := os.RemoveAll(tmpdir)
-               runner.CrunchLog.Printf("While cleaning up temporary directories: %v", rmerr)
+               if rmerr != nil {
+                       runner.CrunchLog.Printf("While cleaning up temporary directory %s: %v", tmpdir, rmerr)
+               }
        }
 }