bdoyle0182 commented on issue #5256:
URL: https://github.com/apache/openwhisk/issues/5256#issuecomment-1146219808

   Also just fyi, I'm working on a change that at least for the case of the 
scheduling decision maker for the initial case of an action having not being 
seen adding (or in the case of no op duration checker just if the action hasn't 
been seen in a while and the queue is stopped) a ratio of containers to bring 
up based on the number of stale activations while waiting for the first 
activation to be returned.
   
   So instead of, 
   
   ```
             case (Running, None) if staleActivationNum > 0 =>
               // we can safely get the value as we already checked the 
existence
               val num = ceiling(staleActivationNum - inProgress)
   ```
   
   ```          case (Running, None) if staleActivationNum > 0 =>
               // we can safely get the value as we already checked the 
existence
               val num = ceiling(schedulingConfig.initialStaleProvisionRatio * 
(staleActivationNum - inProgress))
   ```
   
   Though since the `checkInterval` is 100ms, this check will occur many times 
spinning up more containers while waiting for the first activation response 
when an activation takes 1-2 seconds to respond so the only way for me to test 
this right now is to make the `checkInterval` a few seconds. 
   
   That's just one aspect of this topic though but thought I'd point it out. 
I'll come back to your comment separately


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to