summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorNicolas Lacasse <nlacasse@google.com>2019-08-29 09:55:13 -0700
committerNicolas Lacasse <nlacasse@google.com>2019-08-29 14:00:18 -0700
commit6e8fc27dc5de8ed0a04fe943a6354c0133bfc63f (patch)
tree700a8164cd53b7a5eba5d0b9d4823af0d6ed10d2
parent599136b25e0e506eada1800d84f5d5db7318a7ec (diff)
Don't log an error when stopping the container if it is not running.
This removes a lot of confusing messages from the test logs. PiperOrigin-RevId: 266164001
-rw-r--r--runsc/test/testutil/docker.go6
1 files changed, 5 insertions, 1 deletions
diff --git a/runsc/test/testutil/docker.go b/runsc/test/testutil/docker.go
index 3f3e191b0..94e625259 100644
--- a/runsc/test/testutil/docker.go
+++ b/runsc/test/testutil/docker.go
@@ -297,7 +297,11 @@ func (d *Docker) Remove() error {
func (d *Docker) CleanUp() {
d.logDockerID()
if _, err := do("kill", d.Name); err != nil {
- log.Printf("error killing container %q: %v", d.Name, err)
+ if strings.Contains(err.Error(), "is not running") {
+ // Nothing to kill. Don't log the error in this case.
+ } else {
+ log.Printf("error killing container %q: %v", d.Name, err)
+ }
}
if err := d.Remove(); err != nil {
log.Print(err)