Added additional comments preparing for sidekiq readiness probes

templates/deployment-sidekiq.yaml

@@ -201,7 +201,20 @@ spec:
           {{- with $context.Values.volumeMounts }}
             {{- toYaml . | nindent 12 }}
           {{- end }}
-          resources:
+          # NOTE: Readiness probe will only work on versions of Mastodon built after 2024-07-10.
+          # Will be un-commented once this is in an official release.
+          #
+          #readinessProbe:
+          #  failureThreshold: 10
+          #  exec:
+          #    command:
+          #    - cat
+          #    - /var/www/tmp/sidekiq_process_has_started_and_will_begin_processing_jobs
+          #  initialDelaySeconds: 10
+          #  periodSeconds: 2
+          #  successThreshold: 2
+          #  timeoutSeconds: 1
+          #resources:
             {{- toYaml (default (default $context.Values.resources $context.Values.mastodon.sidekiq.resources) .resources) | nindent 12 }}
       {{- include "mastodon.statsdExporterContainer" $ | indent 8 }}
       {{- with $context.Values.nodeSelector }}

values.yaml

@@ -138,7 +138,10 @@ mastodon:
     resources: {}
     # -- Affinity for all Sidekiq Deployments unless overwritten, overwrites .Values.affinity
     affinity: {}
-    # Rollout strategy to use when updating pods
+    # Rollout strategy to use when updating pods.
+    # Recreate is recommended for sidekiq pods for now because of possible data
+    # error between old/new versions. Please only use RollingUpdate if you are
+    # running a Mastodon version built after 2024-07-10.
     # ref: https://kubernetes.io/docs/concepts/workloads/controllers/deployment/#strategy
     updateStrategy:
       type: Recreate