Skip to content

Commit

Permalink
testsuite: remove test that checks for job shell path on error
Browse files Browse the repository at this point in the history
Problem: A test in t2400-job-exec-test.t checks that the path to
the job is emitted on a job-exec subprocess exec error, but it has
been determined that including this path in error message is more
confusing than helpful.

Remove the test.
  • Loading branch information
grondo committed Jan 24, 2025
1 parent 01cad95 commit 7ff6160
Showing 1 changed file with 0 additions and 16 deletions.
16 changes: 0 additions & 16 deletions t/t2400-job-exec-test.t
Original file line number Diff line number Diff line change
Expand Up @@ -204,20 +204,4 @@ test_expect_success FLUX_SECURITY \
flux job wait-event -vHt 15s $jobid clean &&
flux job status -vvv $jobid
'

if ! grep 'release 7' /etc/centos-release >/dev/null 2>&1 \
&& ! grep 'release 7' /etc/redhat-release >/dev/null 2>&1
then
test_set_prereq NOT_DISTRO7
fi

# The following test does not work on CentOS 7 / RHEL7 since exec errno does
# not work with job-exec (for as yet unknown reason). Skip the test on
# these distros:
test_expect_success NOT_DISTRO7 'job-exec: path to shell is emitted on exec error' '
test_expect_code 127 flux run \
--setattr=exec.job_shell=/foo/flux-shell hostname 2>exec.err &&
test_debug "cat exec.err" &&
grep /foo/flux-shell exec.err
'
test_done

0 comments on commit 7ff6160

Please sign in to comment.