Ticket 743

Summary: qsub wrapper doesn't parse -m in job script
Product: Slurm Reporter: peter.ruprecht
Component: OtherAssignee: Moe Jette <jette>
Status: RESOLVED FIXED QA Contact:
Severity: 4 - Minor Issue    
Priority: --- CC: da
Version: 14.03.0   
Hardware: Linux   
OS: Linux   
Site: University of Colorado Alineos Sites: ---
Atos/Eviden Sites: --- Confidential Site: ---
Coreweave sites: --- Cray Sites: ---
DS9 clusters: --- HPCnow Sites: ---
HPE Sites: --- IBM Sites: ---
NOAA SIte: --- OCF Sites: ---
Recursion Pharma Sites: --- SFW Sites: ---
SNIC sites: --- Linux Distro: ---
Machine Name: CLE Version:
Version Fixed: 14.03.2 Target Release: ---
DevPrio: --- Emory-Cloud Sites: ---
Attachments: fix parsing

Description peter.ruprecht 2014-04-23 05:13:13 MDT
Specifying mail options on the command line using the qsub wrapper works fine:

qsub -m abe mailtest.run

However, when -m is used as a PBS directive in the job script, eg:

#!/bin/sh
#PBS -N cpi
#PBS -l nodes=2:ppn=8
#PBS -l walltime=00:08:00
#PBS -u ruprech
#PBS -m abe
#PBS -M my.email@colorado.edu
#PBS -j 
module load openmpi/openmpi-1.6.4_intel-13.0.0_torque-4.1.4_ib
time mpirun -n 16 ./cpi-rh6

the following error results:

qsub mailtest.run 
There was an error running the SLURM sbatch command.
The command was:
'/curc/slurm/slurm/14.03.0/bin/sbatch mailtest.run  2>&1'
and the output was:
'sbatch: error: -m=abe invalid
'

If instead I do
#PBS -m a
#PBS -m b
#PBS -m e

then it works as expected.

I don't see an obvious error in the qsub perl script.  Any ideas on how to go about fixing this?

Thanks,
Peter Ruprecht
University of Colorado Boulder - Research Computing
Comment 1 Moe Jette 2014-04-24 04:45:05 MDT
Created attachment 790 [details]
fix parsing

Fix will be in v14.03.2
Comment 2 Moe Jette 2014-04-24 04:47:44 MDT
For some reason the logic was looking for specific values, no good. "abe" and "aeb" were not checked for. Also something like "bebe" would fail. The new logic just search letter by letter, which is more flexible.

The commit is here:
https://github.com/SchedMD/slurm/commit/c24efd21386d5de714c27ccd5a8bef038bda06d9