FIX: Remove readiness probe + remove resource limits (#357)

busy-beaver-dev · Aug 28, 2021 · 1d2c06d · 1d2c06d
1 parent 0e59786
commit 1d2c06d
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 20 deletions.
diff --git a/busy_beaver/templates/github_summary_settings.html b/busy_beaver/templates/github_summary_settings.html
@@ -44,7 +44,7 @@ <h3>Configure GitHub Summary Feature</h3>
 <p>You can use the following text to publicize the bot:</p>
 
 <blockquote>
-  Busy Beaver is a community engagement bot that shares daily sumarries of
+  Busy Beaver is a community engagement bot that shares daily summaries of
   public GitHub activity for registered users. Find out what everybody's
   working on in #{{ dict(form.channel.choices).get(form.channel.data) }}!
 </blockquote>

diff --git a/helm/charts/busybeaver/templates/deployment--scheduler.yaml b/helm/charts/busybeaver/templates/deployment--scheduler.yaml
@@ -25,10 +25,10 @@ spec:
         image: {{ .Values.image.repository }}:{{ .Values.image.version }}
         imagePullPolicy: {{ .Values.image.pullPolicy }}
         args: ["scheduler"]
-        resources:
-          limits:
-            memory: "128Mi"
-            cpu: "100m"
+        # resources:
+        #   limits:
+        #     memory: "256Mi"
+        #     cpu: "200m"
         env:
           {{- include "busybeaver.env_vars" . | indent 10 }}
       serviceAccountName: {{ include "busybeaver.serviceAccountName" . }}

diff --git a/helm/charts/busybeaver/templates/deployment--web.yaml b/helm/charts/busybeaver/templates/deployment--web.yaml
@@ -25,10 +25,10 @@ spec:
         image: {{ .Values.image.repository }}:{{ .Values.image.version }}
         imagePullPolicy: {{ .Values.image.pullPolicy }}
         args: ["webserver"]
-        resources:
-          limits:
-            memory: "256Mi"
-            cpu: "200m"
+        # resources:
+        #   limits:
+        #     memory: "512Mi"
+        #     cpu: "500m"
         env:
           {{- include "busybeaver.env_vars" . | indent 10 }}
         ports:

diff --git a/helm/charts/busybeaver/templates/deployment--worker.yaml b/helm/charts/busybeaver/templates/deployment--worker.yaml
@@ -25,18 +25,11 @@ spec:
         image: {{ .Values.image.repository }}:{{ .Values.image.version }}
         imagePullPolicy: {{ .Values.image.pullPolicy }}
         args: ["worker"]
-        resources:
-          limits:
-            memory: "256Mi"
-            cpu: "200m"
+        # resources:
+        #   limits:
+        #     memory: "512Mi"
+        #     cpu: "500m"
         env:
           {{- include "busybeaver.env_vars" . | indent 10 }}
-        readinessProbe:
-          exec:
-            command:
-              - bash
-              - ./scripts/check_worker_box.sh
-          initialDelaySeconds: 10
-          periodSeconds: 10
       serviceAccountName: {{ include "busybeaver.serviceAccountName" . }}
       automountServiceAccountToken: false