From 613dd89fb06713a93d07e4caf977e328e4e08331 Mon Sep 17 00:00:00 2001
From: "SPEARS, DUSTIN (ds443n)" <ds443n@att.com>
Date: Tue, 27 Jun 2023 11:58:18 -0400
Subject: [PATCH] Add ttlSecondsAfterFinished field to cronjob

Adding ttlSecondsAfterFinished option to the chart for db
clean up cronjob
Add history limit options
Add concurrency policy to forbid

Change-Id: I431a9a3692fee36f77c6037031965e58c2c343c0
---
 charts/drydock/templates/cronjob-drydock-db-cleanup.yaml | 5 ++++-
 charts/drydock/values.yaml                               | 4 ++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/charts/drydock/templates/cronjob-drydock-db-cleanup.yaml b/charts/drydock/templates/cronjob-drydock-db-cleanup.yaml
index af45a1ad..c58d8ced 100644
--- a/charts/drydock/templates/cronjob-drydock-db-cleanup.yaml
+++ b/charts/drydock/templates/cronjob-drydock-db-cleanup.yaml
@@ -28,9 +28,13 @@ metadata:
   annotations:
     {{ tuple $envAll | include "helm-toolkit.snippets.release_uuid" }}
 spec:
+  successfulJobsHistoryLimit: {{ .Values.endpoints.postgresql.history.success }}
+  failedJobsHistoryLimit: {{ .Values.endpoints.postgresql.history.failed }}
+  concurrencyPolicy: Forbid
   schedule: {{ .Values.endpoints.postgresql.cleanup_schedule | quote }}
   jobTemplate:
     spec:
+      ttlSecondsAfterFinished: {{ .Values.endpoints.postgresql.ttl_finished }}
       template:
         metadata:
           labels:
@@ -68,4 +72,3 @@ spec:
                 defaultMode: 0555
 ...
 {{- end }}
-
diff --git a/charts/drydock/values.yaml b/charts/drydock/values.yaml
index c60bf2cd..b5e65935 100644
--- a/charts/drydock/values.yaml
+++ b/charts/drydock/values.yaml
@@ -326,8 +326,12 @@ endpoints:
   postgresql:
     # frequency to run table purges
     cleanup_schedule: "0 1 * * *"
+    ttl_finished: 86400
     # number of days to retain for the tasks table
     days_to_retain: "90"
+    history:
+      success: 3
+      failed: 1
     name: postgresql
     auth:
       admin: