Skip to content

Commit

Permalink
Merge pull request #6517 from STEllAR-GROUP/fix-perftest-ci-env
Browse files Browse the repository at this point in the history
Fix perftests CI environment
  • Loading branch information
hkaiser authored Jul 10, 2024
2 parents 94f0ef0 + 7907b56 commit 17bbc63
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 3 deletions.
3 changes: 2 additions & 1 deletion .jenkins/lsu-perftests/launch_perftests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,8 @@ for executable in "${hpx_targets[@]}"; do

# Run performance tests
${perftests_dir}/driver.py -v -l $logfile_tmp perftest run --local True \
--run_output $result --targets-and-opts "${run_command[@]}" --n_executions $n_executions ||
--run_output $result --targets-and-opts "${run_command[@]}" \
--n_executions $n_executions -e $envfile ||
{
echo 'Running failed'
test_errors=1
Expand Down
13 changes: 11 additions & 2 deletions tools/perftests_ci/driver.py
Original file line number Diff line number Diff line change
Expand Up @@ -93,10 +93,14 @@ def build(build_type, environment, target, source_dir, build_dir, install_dir,
'-b',
action='store_true',
help='enable building of '+ var._project_name +' examples')
@args.arg('--environment', '-e', nargs='?', help='path to environment file')
def test(run_mpi_tests, perftests_only, verbose_ctest, examples_build_dir,
build_examples):
build_examples, environment):
import test

if environment:
env.load(environment)

if perftests_only:
test.run_perftests()
else:
Expand Down Expand Up @@ -131,7 +135,12 @@ def perftest():
type=int,
help='number of executions of the benchmark executable, whose \
results are then merged together')
def run(local, run_output, targets_and_opts, n_executions):
@args.arg('--environment', '-e', nargs='?', help='path to environment file')
def run(local, run_output, targets_and_opts, n_executions, environment):

if environment:
env.load(environment)

# options
targets_and_opts = ' '.join(targets_and_opts).lstrip()

Expand Down

0 comments on commit 17bbc63

Please sign in to comment.