jordigilh commented on code in PR #337:
URL: 
https://github.com/apache/incubator-kie-kogito-serverless-operator/pull/337#discussion_r1437117273


##########
controllers/platform/services/services.go:
##########
@@ -114,8 +114,8 @@ func (d DataIndex) GetEnvironmentVariables() 
[]corev1.EnvVar {
 func (d DataIndex) GetPodResourceRequirements() corev1.ResourceRequirements {
        return corev1.ResourceRequirements{
                Limits: corev1.ResourceList{
-                       corev1.ResourceCPU:    resource.MustParse("100m"),
-                       corev1.ResourceMemory: resource.MustParse("256Mi"),
+                       corev1.ResourceCPU:    resource.MustParse("500m"),
+                       corev1.ResourceMemory: resource.MustParse("1Gi"),
                },

Review Comment:
   I got OOMKill with 256Mi for the DI. I increased it to 512Mi as well as the 
CPU limits to try to speed up the deployment as it takes 2:40 seconds for the 
container to reach ready status, which is significant more than for the Job 
Service (90 seconds). 
   
   It didn't help much on both accounts. I can reduce it to 512Mi and the CPU 
to 100m for limits if you think it aligns with your expectations. 
   
   Is there is any previous test done on the resource limits for the DI 
container or these values were selected on best effort?



-- 
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]

Reply via email to