style95 commented on issue #4476: how to remove concurrency limits
URL: 
https://github.com/apache/incubator-openwhisk/issues/4476#issuecomment-491667975
 
 
   Hm, it seems this is the issue.
   
   But I think it should work with `131072`  because `cpushare` is calculated 
based on this:
   
https://github.com/apache/incubator-openwhisk/blob/master/common/scala/src/main/scala/org/apache/openwhisk/core/containerpool/ContainerFactory.scala#L47
   
   ```
   scala> (1024.0/(131072/256)).toInt
   res8: Int = 2
   ```
   And there is clearly the limit on the number of containers that can run on a 
host.
   
https://stackoverflow.com/questions/21799382/is-there-a-maximum-number-of-containers-running-on-a-docker-host
   
   It seems the issue comes out from 1000 containers.
   If you configure the memory as `131072`, there can be 1024 containers with 
`128MB` actions.
   So I think it would be better to configure the smaller memory than this.
   And it is obviously better to have a bigger number of invokers with smaller 
memory as Docker daemon cannot handle a huge number of concurrent requests in 
time.
   
   Also, you need to leave some memory for the invoker itself as well.
   
   Anyway, I think the value should be 2 in case the calculated value is 
smaller than 2.
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to