Hi, guys,
​
      Due to this pr (Adjust prewarm container dynamically): 
https://github.com/apache/openwhisk/pull/4871 is merged
I think it is a good chance to add this feature: take prewarmed container's 
memory as used memory: https://github.com/apache/openwhisk/pull/4911
​
for master branch,  if invoker has `16GB` physical memory, currently, if we 
configure 12GB user memory and 5GB prewarm pool, invokers will try to create 
containers up to 17GB memory and it may will end up with OOM.
​
After apply this feature, user just configure invoker memory to a precise 
value:  `16GB` , when  `used memory` reaches `16GB`, it will not create 
container any more, which mean, `16GB` is the max memory usuage for containers, 
this can avoid `OOM`
​
This has another benefit that make user must calculate the prewarmed 
container's memory carefully, and configure the invoker memory more accurately
```
"CONFIG_whisk_containerPool_userMemory": "{{ 
hostvars[groups['invokers'][invoker_index | int]].user_memory | 
default(invoker.userMemory) }}"
```
​
Have any idea or suggestion for this pr: take prewarmed container's memory as 
used memory: https://github.com/apache/openwhisk/pull/4911 ?

Reply via email to