You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@openwhisk.apache.org by GitBox <gi...@apache.org> on 2022/11/01 07:45:58 UTC

[GitHub] [openwhisk] bdoyle0182 commented on a diff in pull request #5346: Make the test stable

bdoyle0182 commented on code in PR #5346:
URL: https://github.com/apache/openwhisk/pull/5346#discussion_r1010174107


##########
tests/src/test/scala/org/apache/openwhisk/core/containerpool/v2/test/FunctionPullingContainerProxyTests.scala:
##########
@@ -1575,6 +1575,9 @@ class FunctionPullingContainerProxyTests
 
     machine ! Initialize(invocationNamespace.asString, fqn, action, schedulerHost, rpcPort, messageTransId)
     probe.expectMsg(Transition(machine, Uninitialized, CreatingClient))
+    awaitAssert {
+      machine.stateData shouldBe a[ContainerCreatedData]

Review Comment:
   Was this test instability introduced recently? I know this is just a test, but I'm wondering if there could be an issue with this https://github.com/apache/openwhisk/pull/5333 since I'm seeing weird orphaned non-existent containers in etcd



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org