[ARVADOS] updated: b351a877ff2d708acc5fe577d5ed5cc9b0469dde
Git user
git at public.curoverse.com
Fri Mar 31 10:17:28 EDT 2017
Summary of changes:
services/crunch-run/crunchrun.go | 2 +-
services/crunch-run/crunchrun_test.go | 20 ++++++++++----------
2 files changed, 11 insertions(+), 11 deletions(-)
via b351a877ff2d708acc5fe577d5ed5cc9b0469dde (commit)
from 1a4a2f3219906220c0b4d7fd9b90325fa529408a (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit b351a877ff2d708acc5fe577d5ed5cc9b0469dde
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date: Fri Mar 31 10:17:23 2017 -0400
9132: Fix test errors due to rebase.
diff --git a/services/crunch-run/crunchrun.go b/services/crunch-run/crunchrun.go
index ad0a337..a05f61a 100644
--- a/services/crunch-run/crunchrun.go
+++ b/services/crunch-run/crunchrun.go
@@ -780,7 +780,7 @@ func (runner *ContainerRunner) WaitFinish() error {
return fmt.Errorf("container wait: %v", err)
}
- runner.CrunchLog.Printf("container wait API status code: %v", waitDocker)
+ runner.CrunchLog.Printf("Container exited with code: %v", waitDocker)
code := int(waitDocker)
runner.ExitCode = &code
diff --git a/services/crunch-run/crunchrun_test.go b/services/crunch-run/crunchrun_test.go
index 5cd0d41..98462f8 100644
--- a/services/crunch-run/crunchrun_test.go
+++ b/services/crunch-run/crunchrun_test.go
@@ -683,11 +683,11 @@ func (s *TestSuite) TestNodeInfoLog(c *C) {
"output_path": "/tmp",
"priority": 1,
"runtime_constraints": {}
- }`, nil, func(t *TestDockerClient) {
- time.Sleep(time.Second)
- t.logWriter.Close()
- t.finish <- dockerclient.WaitResult{}
- })
+ }`, nil, 0,
+ func(t *TestDockerClient) {
+ time.Sleep(time.Second)
+ t.logWriter.Close()
+ })
c.Check(api.CalledWith("container.exit_code", 0), NotNil)
c.Check(api.CalledWith("container.state", "Complete"), NotNil)
@@ -710,11 +710,11 @@ func (s *TestSuite) TestContainerRecordLog(c *C) {
"output_path": "/tmp",
"priority": 1,
"runtime_constraints": {}
- }`, nil, func(t *TestDockerClient) {
- time.Sleep(time.Second)
- t.logWriter.Close()
- t.finish <- dockerclient.WaitResult{}
- })
+ }`, nil, 0,
+ func(t *TestDockerClient) {
+ time.Sleep(time.Second)
+ t.logWriter.Close()
+ })
c.Check(api.CalledWith("container.exit_code", 0), NotNil)
c.Check(api.CalledWith("container.state", "Complete"), NotNil)
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list