Merge branch '10658-crunch-run-missing-files' closes #10658
authorPeter Amstutz <peter.amstutz@curoverse.com>
Tue, 6 Dec 2016 14:40:12 +0000 (09:40 -0500)
committerPeter Amstutz <peter.amstutz@curoverse.com>
Tue, 6 Dec 2016 14:40:12 +0000 (09:40 -0500)
services/crunch-run/crunchrun.go

index ade40c6b03a4d4a98812172aab31da5173453c4e..2e475c72e64842b15aa6c7dee88446bc0056b802 100644 (file)
@@ -800,6 +800,7 @@ func (runner *ContainerRunner) Run() (err error) {
                checkErr(err)
 
                if runner.finalState == "Queued" {
+                       runner.CrunchLog.Close()
                        runner.UpdateContainerFinal()
                        return
                }
@@ -832,6 +833,7 @@ func (runner *ContainerRunner) Run() (err error) {
        // check for and/or load image
        err = runner.LoadImage()
        if err != nil {
+               runner.finalState = "Cancelled"
                err = fmt.Errorf("While loading container image: %v", err)
                return
        }
@@ -839,6 +841,7 @@ func (runner *ContainerRunner) Run() (err error) {
        // set up FUSE mount and binds
        err = runner.SetupMounts()
        if err != nil {
+               runner.finalState = "Cancelled"
                err = fmt.Errorf("While setting up mounts: %v", err)
                return
        }