From abd29e856f368869b792668d5baa63b9d935032d Mon Sep 17 00:00:00 2001 From: Miguel Duarte Barroso Date: Mon, 29 Nov 2021 16:50:10 +0100 Subject: [PATCH] Safer reconciler spec (#167) * reconciler, spec: forbid concurrent executions Signed-off-by: Miguel Duarte Barroso * reconciler, spec: remove complete pods We leave the default `failedJobsHistoryLimit` - 1 - to enable the user to debug the reconciler in case it fails. Signed-off-by: Miguel Duarte Barroso * reconciler, spec: do not retry on error Since a new pod will be scheduled in 5 minutes, it is safer to just wait for the new reconcile cycle than insist on reconciling right now. This is important to prevent the cluster to clog further on constrained clusters. Signed-off-by: Miguel Duarte Barroso --- doc/crds/ip-reconciler-job.yaml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/crds/ip-reconciler-job.yaml b/doc/crds/ip-reconciler-job.yaml index 80b62318f..df61a4e02 100644 --- a/doc/crds/ip-reconciler-job.yaml +++ b/doc/crds/ip-reconciler-job.yaml @@ -7,9 +7,12 @@ metadata: tier: node app: whereabouts spec: + concurrencyPolicy: Forbid + successfulJobsHistoryLimit: 0 schedule: "*/5 * * * *" jobTemplate: spec: + backoffLimit: 0 template: metadata: labels: