- 14 Apr, 2019 40 commits
-
-
Tim Wickberg authored
Have slurm_process_option() call exit(-1) on errors. Most parsing functions will already do this, but that should be revisited at a later point in time.
-
Tim Wickberg authored
-
Tim Wickberg authored
Remove the options argument now that no calling paths need it. Note that opt_string here is alphabetical *by the option name*, and not by the val any longer.
-
Tim Wickberg authored
-
Tim Wickberg authored
Move handling for parallel debugging into _opt_verify(), and explicitly call through slurm_process_option() to override the default settings.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
It's just an unused copy of nodefile at this point.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Drop _get_int() as this was the last calling location.
-
Tim Wickberg authored
All env vars are now handled through slurm_process_options().
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
There is a mention in a3449c0b that it was pulled out of slurm_opt_t previously because the Perl API needs it; I suspect the path requiring it was removed with either BG/Q or Cray ALPS, and this is no longer a strict requirement. At least there are no references to mpi_type left there.
-
Tim Wickberg authored
Drop the cpu_bind_type_set bool from srun_opt_t; this can already be deduced from whether cpu_bind_type has any bits (aside from CPU_BIND_VERBOSE) set.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Move warning about excessive values for --threads into arg_set_max_threads, but soften the language since the limit has not been enforced for a while. Do not reset this each pass. Especially since the only effect this option has is to bump pmi_server_max_threads() around, and that doesn't not know about HetJobs. Move MAX_THREADS to slurm_opt.h, and rename to SRUN_MAX_THREADS to avoid collision with the same macro name elsewhere. This should probably be made into a configuration value at some point.
-
Tim Wickberg authored
Drop relative_set in favor of direct test against NO_VAL. Zero is also a valid value to have set this to - change two tests to only check against NO_VAL.
-
Tim Wickberg authored
-
Tim Wickberg authored
Note that this argument's behavior is different from --wait in sbatch.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Drop --preserve-slurm-env as an alias; this has not been documented since at least version 2.6.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Drop the call to uname() as the name variable is unused.
-
Tim Wickberg authored
Move default setting of slurm_get_srun_epilog() to _opt_verify().
-
Tim Wickberg authored
Move default setting of slurm_get_srun_prolog() to _opt_verify().
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-