PR introduced a few changes of CHE_CORE_JSONRPC which are most likely not needed and upstream values can be safely used - https://github.com/redhat-developer/rh-che/pull/1323
CHE_CORE_JSONRPC_MINOR__PROCESSOR__CORE__POOL__SIZE: '5'
CHE_CORE_JSONRPC_MINOR__PROCESSOR__MAX__POOL__SIZE: '100'
CHE_CORE_JSONRPC_MINOR__PROCESSOR__QUEUE__CAPACITY: '1000'
CHE_CORE_JSONRPC_PROCESSOR__CORE__POOL__SIZE: '30'
CHE_CORE_JSONRPC_PROCESSOR__MAX__POOL__SIZE: '100'
CHE_CORE_JSONRPC_PROCESSOR__QUEUE__CAPACITY: '500000'
Need to verify if che.openshift.io works stable without sideeffects with 1Gb of RAM and upstream settings of CHE_CORE_JSONRPC.
About default upstream values, I was able to cause the OOM crash on upstream defaults as well. They will require some tweaking (namely message queue size)
cc: @skabashnyuk ^ wdyt shoud upstream values be adjusted. IMO it would be great if che.openshift.io would use upstream setup for RAM / CHE_CORE_JSONRPC
I agree with @ScrewTSW CHE_CORE_JSONRPC_PROCESSOR__QUEUE__CAPACITY has to be adjusted to fit in the 1G limit. @ScrewTSW what would be your suggestion?
I agree with @ScrewTSW
CHE_CORE_JSONRPC_PROCESSOR__QUEUE__CAPACITYhas to be adjusted to fit in the 1G limit. @ScrewTSW what would be your suggestion?
I would try 200k at most. I think 150k is the safer value here.
@ScrewTSW are you planning to provide upstream PR for review as part of this issue?
Already doing some adjustment https://github.com/eclipse/che/pull/13034
@skabashnyuk wdyt, once the adjustments are done upsteam - does it makes sense to fall back on those defaults in rh-che?
Some data from real-life server


I think upstream defaults should be enaught for this type of load.
# Maximum size of the JSON RPC processing pool
# in case if pool size would be exceeded message execution will be rejected
che.core.jsonrpc.processor_max_pool_size=50
# Initial json processing pool. Minimum number of threads that used to process major JSON RPC messages.
che.core.jsonrpc.processor_core_pool_size=5
applied on production
Most helpful comment
Already doing some adjustment https://github.com/eclipse/che/pull/13034