digitalocean / clusterlint

A best practices checker for Kubernetes clusters. 🤠
Apache License 2.0
542 stars 45 forks source link

Add a check for cronjobs with a concurrencyPolicy of Allow #95

Closed timoreimann closed 3 years ago

timoreimann commented 3 years ago

A cronjob configured with a concurrencyPolicy of Allow has the potential for overloading a cluster if a cronjob-managed pod cannot manage to finish in time. The impact we are occasionally seeing is that job-pods created from subsequent cron intervals pile up over time, sometimes leading to hundreds or thousands of pods that stay pending.

We should implement a check that triggers a warning for Allow cronjobs and suggests to use one of the safer alternative modes Forbid or Replace.

jasimmons commented 3 years ago

Hey @timoreimann, I'm going to pick this up! 😄

varshavaradarajan commented 3 years ago

fixed by #97