Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Align clean downloads schedule #2717

Merged
merged 1 commit into from
Apr 12, 2024
Merged

Conversation

AndreaFrancis
Copy link
Contributor

@AndreaFrancis AndreaFrancis commented Apr 12, 2024

Rollback of #2716 and align schedule with cache folder

Copy link

ArgoCD Diff for commit 65fdc9e

Updated at 4/12/2024, 7:06:29 PM CEST

App: datasets-server-prod
YAML generation: Success 🟢
App sync status: Synced ✅

===== batch/CronJob datasets-server/prod-datasets-server-job-clean-duckdb-downloads ======
--- /tmp/argocd-diff4188376021/prod-datasets-server-job-clean-duckdb-downloads-live.yaml	2024-04-12 17:06:28.594303787 +0000
+++ /tmp/argocd-diff4188376021/prod-datasets-server-job-clean-duckdb-downloads	2024-04-12 17:06:28.594303787 +0000
@@ -303,7 +303,7 @@
             persistentVolumeClaim:
               claimName: datasets-server-duckdb-pvc
       ttlSecondsAfterFinished: 3600
-  schedule: '*/10 * * * *'
+  schedule: 0 */3 * * *
   successfulJobsHistoryLimit: 3
   suspend: false
 status:

App: datasets-server-staging
YAML generation: Success 🟢
App sync status: Synced ✅

===== batch/CronJob datasets-server/staging-datasets-server-job-clean-duckdb-downloads ======
--- /tmp/argocd-diff1334316249/staging-datasets-server-job-clean-duckdb-downloads-live.yaml	2024-04-12 17:06:29.222308171 +0000
+++ /tmp/argocd-diff1334316249/staging-datasets-server-job-clean-duckdb-downloads	2024-04-12 17:06:29.218308142 +0000
@@ -300,7 +300,7 @@
             persistentVolumeClaim:
               claimName: datasets-server-duckdb-pvc
       ttlSecondsAfterFinished: 3600
-  schedule: 0 0 * * *
+  schedule: 0 */3 * * *
   successfulJobsHistoryLimit: 3
   suspend: false
 status:

Legend Status
The app is synced in ArgoCD, and diffs you see are solely from this PR.
⚠️ The app is out-of-sync in ArgoCD, and the diffs you see include those changes plus any from this PR.
🛑 There was an error generating the ArgoCD diffs due to changes in this PR.

@AndreaFrancis AndreaFrancis merged commit 2ee9156 into main Apr 12, 2024
2 checks passed
@AndreaFrancis AndreaFrancis deleted the rollback-clean-downloads-schedule branch April 12, 2024 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant