From ee9631ebdb663a40609b43da63eea641b471b009 Mon Sep 17 00:00:00 2001
From: Manuel Alejandro de Brito Fontes <aledbf@gmail.com>
Date: Wed, 1 Apr 2020 18:33:38 -0300
Subject: [PATCH] Set default resource requests limits

---
 charts/ingress-nginx/values.yaml | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/charts/ingress-nginx/values.yaml b/charts/ingress-nginx/values.yaml
index b80a261bd..ac148f39e 100644
--- a/charts/ingress-nginx/values.yaml
+++ b/charts/ingress-nginx/values.yaml
@@ -224,13 +224,17 @@ controller:
 
   minAvailable: 1
 
-  resources: {}
+  # Define requests resources to avoid probe issues due to CPU utilization in busy nodes
+  # ref: https://github.com/kubernetes/ingress-nginx/issues/4735#issuecomment-551204903
+  # Ideally, there should be no limits.
+  # https://engineering.indeedblog.com/blog/2019/12/cpu-throttling-regression-fix/
+  resources:
   #  limits:
   #    cpu: 100m
-  #    memory: 64Mi
-  #  requests:
-  #    cpu: 100m
-  #    memory: 64Mi
+  #    memory: 90Mi
+    requests:
+      cpu: 100m
+      memory: 90Mi
 
   autoscaling:
     enabled: false