Pil0tXia commented on code in PR #4693:
URL: https://github.com/apache/eventmesh/pull/4693#discussion_r1446891659
##########
eventmesh-operator/config/samples/eventmesh_v1_runtime.yaml:
##########
@@ -271,6 +284,14 @@ spec:
name: runtime-config
# imagePullPolicy is the image pull policy
imagePullPolicy: Always
+ # resources describes the compute resource requirements and limits
+ resources:
+ requests:
+ memory: "1024Mi"
+ cpu: "250m"
+ limits:
+ memory: "2048Mi"
+ cpu: "500m"
Review Comment:
Are these resources set for EventMesh Runtime?
The memory limit set for EventMesh Runtime in start.sh is 8G. Although it
typically requires less, the limit of 2G is too strict.
The threadPools in EventMesh Runtime are designed for 4~8 core machines to
speed up async processor.
We can use eventmesh image under apache namespace in future updates.
Besides, why are the values of the properties enclosed in double quotation
marks here, while they are not in https://github.com/apache/eventmesh/pull/4727?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
