Skip to content

Force delete very old delayed jobs#4395

Merged
svkrieger merged 1 commit into
mainfrom
force-delete-very-old-delayed-jobs
Jun 12, 2025
Merged

Force delete very old delayed jobs#4395
svkrieger merged 1 commit into
mainfrom
force-delete-very-old-delayed-jobs

Conversation

@svkrieger

@svkrieger svkrieger commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

A short explanation of the proposed change:

Force delete delayed_jobs, which are older (run_at) then the configured cutoff_age_in_days + 1 day.

An explanation of the use cases your change solves

We found some old jobs in the delayed_jobs table, which are beyond the cutoff_age_in_days interval. They had either no failed_at set or were still locked. With this change the FailedJobsCleanup job will force delete all jobs, which are older than the cutoff_age_in_days + 1 day.

Links to any other associated PRs

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@svkrieger svkrieger force-pushed the force-delete-very-old-delayed-jobs branch from 256e517 to f985f7c Compare June 10, 2025 12:39
@svkrieger svkrieger force-pushed the force-delete-very-old-delayed-jobs branch from f985f7c to 9a9248f Compare June 11, 2025 07:52
Comment thread spec/unit/jobs/runtime/failed_jobs_cleanup_spec.rb Outdated
We found some old jobs in the delayed_jobs table, which are beyond the cutoff_age_in_days interval.
They had either no failed_at set or were still locked. With this change the FailedJobsCleanup job will force delete all jobs,
which are older than the cutoff_age_in_days + 1 day.
@svkrieger svkrieger force-pushed the force-delete-very-old-delayed-jobs branch from 9a9248f to c4bca63 Compare June 12, 2025 09:00
@svkrieger svkrieger requested a review from philippthun June 12, 2025 09:04
@svkrieger svkrieger merged commit 0dd3a61 into main Jun 12, 2025
11 of 12 checks passed
@svkrieger svkrieger deleted the force-delete-very-old-delayed-jobs branch June 12, 2025 12:37
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Jun 12, 2025
Changes in cloud_controller_ng:

- Force delete very old delayed jobs
    PR: cloudfoundry/cloud_controller_ng#4395
    Author: Sven Krieger <37476281+svkrieger@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants