[GitHub] cbickel commented on issue #4281: Make intermittently failing concurrencyTests more stable.

[GitBox <gi...@apache.org>]

cbickel commented on issue #4281: Make intermittently failing concurrencyTests more stable.
URL: https://github.com/apache/incubator-openwhisk/pull/4281#issuecomment-463516408
 
 
   @tysonnorris 
   The test `execute activations sequentially when concurrency = 1` creates the `concurrent.js` action with a concurrency of 1.
   Afterwards, 4 activations are started in parallel.
   As the concurrency is set to one, there should be 4 user-containers. In each container, the counter will be set to one.
   But there could be the issue, that one of the 4 actions arrives with a delay at the invoker. If the delay is high enough, that one of the other 3 actions is already finished, the warm container will be reused, instead of a new one. As the counter is not set to 0 in a warm container, the condition `counter == requestCount` will never be true (conter is already set to 2, but requestCount is 1).
   If an action decrements the counter on leaving again, the counter will always return the amount of actions that are currently inside this container.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services