Skip to content

Commit

Permalink
remove AUTOREMOVE_BUILDS_SKIP_THRESHOLD config
Browse files Browse the repository at this point in the history
We will rely on a separate, lower priority queue instead
  • Loading branch information
zackgalbreath committed Oct 8, 2024
1 parent 7a8ae64 commit 4f6cfe8
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 11 deletions.
4 changes: 0 additions & 4 deletions .env.example
Original file line number Diff line number Diff line change
Expand Up @@ -30,10 +30,6 @@ DB_PASSWORD=secret
# are slow to execute.
#AUTOREMOVE_BUILDS_BATCH_SIZE=10

# Skip the autoremove builds step if more than this number of submissions
# are waiting to be parsed.
#AUTOREMOVE_BUILDS_SKIP_THRESHOLD=100

# How long should CDash store parsed input files (in hours?)
# Set to 0 if you do not wish to backup parsed submission files.
#BACKUP_TIMEFRAME=48
Expand Down
6 changes: 0 additions & 6 deletions app/Jobs/PruneBuilds.php
Original file line number Diff line number Diff line change
Expand Up @@ -26,12 +26,6 @@ public function handle(): void
if (!(bool) config('cdash.autoremove_builds')) {
return;
}

$skip_threshold = (int) config('cdash.autoremove_builds_skip_threshold');
if (DB::table('jobs')->count() > $skip_threshold) {
return;
}

Artisan::call('build:remove all');
}
}
1 change: 0 additions & 1 deletion config/cdash.php
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@
'active_project_days' => env('ACTIVE_PROJECT_DAYS', 7),
'autoremove_builds' => env('AUTOREMOVE_BUILDS', true),
'autoremove_builds_batch_size' => env('AUTOREMOVE_BUILDS_BATCH_SIZE', 10),
'autoremove_builds_skip_threshold' => env('AUTOREMOVE_BUILDS_SKIP_THRESHOLD', 100),
'backup_timeframe' => env('BACKUP_TIMEFRAME', 48),
'builds_per_project' => env('BUILDS_PER_PROJECT', 0),
'coverage_dir' => env('CDASH_COVERAGE_DIR', '/cdash/_build/xdebugCoverage'),
Expand Down

0 comments on commit 4f6cfe8

Please sign in to comment.