- 13 Jan, 2012 6 commits
-
-
Mark A. Grondona authored
Add test7.15 to test for any signals blocked by default in job tasks.
-
Mark A. Grondona authored
It was found that slurmstepd was intermittently leaving SIGPIPE blocked when launching user tasks. This may have something to do with the fact that the xsignal_unblock() call in _fork_all_tasks() is referencing an extern array (nominally this should have unblocked SIGPIPE), but I didn't spend the time to fully track this issue down. Instead, I figured there is probably no reason we would _not_ want to unblock *all* signals, so this patch does that. Before this change, the following program fails every once in awhile: #include <stdio.h> #include <signal.h> int main (int ac, char **av) { int i, rc = 0; struct sigaction act; for (i = 1; i < SIGRTMAX; i++) { sigaction (i, NULL, &act); if (act.sa_handler == SIG_DFL) continue; fprintf (stderr, "Signal %d appears to be ignored!\n", i); rc = 1; } return (rc); } with: srun -N1 -n1 ./test Signal 13 appears to be ignored! after the change, the program succeeds.
-
Danny Auble authored
number.
-
Morris Jette authored
-
Phil Eckert authored
This new sbatch option allows the user to specify a file containing environment variables to be used in the jobs execution environment.
-
Morris Jette authored
Let operators see reservation data even if "PrivateData=reservations" flag is set in slurm.conf. Patch from Don Albert, Bull.
-
- 11 Jan, 2012 1 commit
-
-
Morris Jette authored
It used to be listed as "--switch=" rather than "--switches="
-
- 09 Jan, 2012 3 commits
-
-
Morris Jette authored
-
Morris Jette authored
Fix bug in srun --multi-prog configuration file to avoid printing duplicate record error when "*" is used at the end of the file for the task ID. It means all task IDs not otherwise identified.
-
Morris Jette authored
Fix race condition where sbcast command can result in deadlock of slurmd daemon. Patch by Don Albert, Bull.
-
- 28 Dec, 2011 1 commit
-
-
Morris Jette authored
-
- 21 Dec, 2011 3 commits
-
-
Mark A. Grondona authored
In commit commit b33cd7c8 Author: Mark A. Grondona <mgrondona@llnl.gov> Date: Fri Oct 7 13:52:43 2011 -0700 a minor bug was introduced because the io_dup_stdio() call was moved out from under the 'else' case of the empty if if (job->pty && (task->gtid == 0)) in task.c:exec_task() This results in the error dup2(stdin): Bad file descriptor For task 0 when --pty is used, because io_dup_stdio() is inappropriately being called after login_tty(). This patch collects the stdio setup into prepare_tty(), renames that function prepare_stdio(), and removes the io_dup_stdio() call from exec_task(). This removes the last io_* call from slurmstepd/task.c, so the io.h header is removed. I didn't see a problem with moving the io_dup_stdio() call earlier in the task setup path, but it could probably use another set of eyes. ---------- Note: This problem was at least partly fixed with https://github.com/SchedMD/slurm/commit/4d17300c5de7a8f146072bae8baaf773ca1fe98a
-
Morris Jette authored
-
Morris Jette authored
-
- 20 Dec, 2011 1 commit
-
-
Morris Jette authored
-
- 19 Dec, 2011 1 commit
-
-
Morris Jette authored
-
- 17 Dec, 2011 3 commits
-
-
Morris Jette authored
-
Mark A. Grondona authored
When importing an exported shell function, the bracket count code didn't handle negative return value from _bracket_cnt(), so a shell function (or a shell variable starting with '(') would loop until the end of the env output. Since the test for no open bracket wasn't handled anyway, remove this and clean up _bracket_cnt() code (which now returns negative if there are more closing than opening brackets). Also, remove the unneded local integer bracket_cnt from the while loops and only continue processing while the bracket tally is > 0. Note: There are still many unhandled corner cases in this code.
-
Mark A. Grondona authored
src/common/env.c:env_array_user_default() tries to import a "default" user environment by parsing the output of something like su - <user> -c "/bin/env" ( or "slurmstepd getenv") (or an equivalent) when the job is run with --get-user-env. There is some code that tries to detect and import exported bash shell functions, which usually have the form func=() { ... } into the job environment. However, one of the many problems with the code is that the test for bounds overflow: if ((strlen(value) + strlen(line)) > (sizeof(value) - 1)) break; useis sizeof(value) and value is a char *. This means that all multiline shell functions are truncated, and the following error appears in users batch script output: /bin/sh: func: line 1: syntax error: unexpected end of file /bin/sh: error importing function definition for `func' This patch fixes the bounds test. Similar cut-and-paste code in _load_env_cache() was also fixed. However, it should be noted that there are many other potential cases where shell functions will not be properly imported by this code. (For example, quoted unbalanced brackets used in the function) Someday this should be fixed.
-
- 15 Dec, 2011 1 commit
-
-
Morris Jette authored
Prevent resetting a held job's priority when updating other job parameters. Patch from Alejandro Lucero Palau, BSC.
-
- 14 Dec, 2011 4 commits
-
-
Morris Jette authored
Patch from John Thiltges, University of Nebraska-Lincoln.
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
-
- 09 Dec, 2011 6 commits
-
-
Danny Auble authored
-
Danny Auble authored
starts in accounting.
-
Danny Auble authored
-
Morris Jette authored
-
Morris Jette authored
There was an oversight in the GracTime preemption logic that resulted in repeatedly reading the PreemptType configuration parameter and doing a string comparison. This change will improve efficiency of that logic.
-
Morris Jette authored
-
- 08 Dec, 2011 2 commits
-
-
Danny Auble authored
-
Danny Auble authored
-
- 06 Dec, 2011 2 commits
-
-
Morris Jette authored
One of our testers discovered a regression in version 2.3.1. If a job is pending due to PartitionNodeLimit and the limit is relieved with a 'sacctmgr modify qos name=<qos name> set flags=partitionmaxnodes' new jobs exceeding the partition limit (but not the QOS limit) are allowed to run. However, the pending job is never allowed to run. Attached is a patch to address this problem. FYI, this problem doesn't exist in version 2.4. Patch from Bill Brophy, Bull.
-
Yuri D'Elia authored
-
- 05 Dec, 2011 3 commits
-
-
Morris Jette authored
Patch by Alexander Bersenev (Institute of Mathematics and Mechanics, Russia).
-
Morris Jette authored
-
Morris Jette authored
-
- 02 Dec, 2011 2 commits
-
-
Danny Auble authored
There was also some bad code that would reset the conn_type of a block to SMALL no matter what type of SMALL it was.
-
Morris Jette authored
Patsh from Rod Schulz, Bull.
-
- 01 Dec, 2011 1 commit
-
-
jette authored
This was due to a bug in select/cons_res with some configuration optiions and job options, especially if there is more than one thread per core and the job option includes "--threads-per-core=1". Fixes problem reported by CSCS.
-