Skip to content

Commit

Permalink
hydra-eval-jobset: pass --workers and --max-memory-size to n-e-j
Browse files Browse the repository at this point in the history
Lost in the h-e-j -> n-e-j migration, causing evaluation to always be
single threaded and limited to 4GiB RAM. Follow the config settings like
h-e-j used to do (via C++ code).

(cherry picked from commit 4b107e6)
  • Loading branch information
delroth authored and Ericson2314 committed Nov 25, 2024
1 parent 72e2516 commit d97956d
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/script/hydra-eval-jobset
Original file line number Diff line number Diff line change
Expand Up @@ -374,6 +374,8 @@ sub evalJobs {
push @cmd, "--constituents";
push @cmd, "--force-recurse";
push @cmd, ("--option", "allow-import-from-derivation", "false") if $config->{allow_import_from_derivation} // "true" ne "true";
push @cmd, ("--workers", $config->{evaluator_workers} // 1);
push @cmd, ("--max-memory-size", $config->{evaluator_max_memory_size} // 4096);

if (defined $ENV{'HYDRA_DEBUG'}) {
sub escape {
Expand Down

0 comments on commit d97956d

Please sign in to comment.