openwhisk-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rab...@apache.org
Subject [incubator-openwhisk] branch master updated: Add additional debug information for intermittently failing ContainerProxyTests (#4278)
Date Wed, 13 Feb 2019 14:38:43 GMT
This is an automated email from the ASF dual-hosted git repository.

rabbah pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk.git


The following commit(s) were added to refs/heads/master by this push:
     new a205f8b  Add additional debug information for intermittently failing ContainerProxyTests
(#4278)
a205f8b is described below

commit a205f8b50311181e14e8af6d8eb91eba84045ea6
Author: Christian Bickel <github@cbickel.de>
AuthorDate: Wed Feb 13 15:38:38 2019 +0100

    Add additional debug information for intermittently failing ContainerProxyTests (#4278)
    
    The `ContainerProxyTests` are failing intermittently in the pipeline. All of these intermittent
test failures have either a wrong initialisation-time or it is missing completely.
    Our theory currently is, that the Buffer in `acker` is called in the wrong order, as these
calls are made asynchronously. If that theory is correct, this can be proven with this additional
output.
---
 .../core/containerpool/test/ContainerProxyTests.scala    | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/tests/src/test/scala/org/apache/openwhisk/core/containerpool/test/ContainerProxyTests.scala
b/tests/src/test/scala/org/apache/openwhisk/core/containerpool/test/ContainerProxyTests.scala
index b51832b..1c4bb51 100644
--- a/tests/src/test/scala/org/apache/openwhisk/core/containerpool/test/ContainerProxyTests.scala
+++ b/tests/src/test/scala/org/apache/openwhisk/core/containerpool/test/ContainerProxyTests.scala
@@ -303,6 +303,10 @@ class ContainerProxyTests
       collector.calls should have size 2
       container.suspendCount shouldBe 0
       acker.calls should have size 2
+
+      // Add debugging information for the case of failure.
+      println(acker.calls.map(_._2.toJson.compactPrint))
+
       store.calls should have size 2
 
       val initRunActivation = acker.calls(0)._2
@@ -360,6 +364,10 @@ class ContainerProxyTests
       container.suspendCount shouldBe 1
       container.resumeCount shouldBe 1
       acker.calls should have size 2
+
+      // Add debugging information for the case of failure.
+      println(acker.calls.map(_._2.toJson.compactPrint))
+
       store.calls should have size 2
       acker
         .calls(0)
@@ -548,6 +556,10 @@ class ContainerProxyTests
       container.suspendCount shouldBe 1
       container.resumeCount shouldBe 0
       acker.calls should have size 6
+
+      // Add debugging information for the case of failure.
+      println(acker.calls.map(_._2.toJson.compactPrint))
+
       store.calls should have size 6
       acker
         .calls(0)
@@ -611,6 +623,10 @@ class ContainerProxyTests
       container.suspendCount shouldBe 0
       container.destroyCount shouldBe 0
       acker.calls should have size 2
+
+      // Add debugging information for the case of failure.
+      println(acker.calls.map(_._2.toJson.compactPrint))
+
       store.calls should have size 2
 
       val initErrorActivation = acker.calls(0)._2


Mime
View raw message