- 27 Jan, 2012 6 commits
-
-
Morris Jette authored
-
Danny Auble authored
Lucero Palau.
-
Morris Jette authored
-
Morris Jette authored
Patch from Mark Nelson
-
Morris Jette authored
This patch was previously applied to SLURM v2.4 and is being back-ported due to problems being reported in SLURM v2.3. Original commit is here https://github.com/SchedMD/slurm/commit/4c0eea7b8c20ccb1cacad51838a1ea8257cc637d
-
Matthieu Hautreux authored
-
- 25 Jan, 2012 1 commit
-
-
Morris Jette authored
Set DEFAULT flag in partition structure when slurmctld reads the configuration file. Patch from Rémi Palancher. Note the flag is set when the information is sent via RPC for sinfo.
-
- 24 Jan, 2012 3 commits
-
-
Morris Jette authored
-
Morris Jette authored
-
Morris Jette authored
Do not test for CUDA environment variable on Cray system
-
- 23 Jan, 2012 3 commits
-
-
Morris Jette authored
needed for test24.1
-
Morris Jette authored
-
Philip D. Eckert authored
Moe, Here it is, I have added a subroutine to env.c to unset the user's environment and then called it from sbatch in main. I also removed the comment from the sbatch man page indicating that it wasn't working the same for a regular user as it did for Moab. It should now be functionally the same. I think there is still a difference between how sbatch functions with an environment in a file than it does from when Moab excve'd the environment. However, I'm not sure what it would be at this point. Again, for so many iterations.... Phil
-
- 22 Jan, 2012 1 commit
-
-
Philip D. Eckert authored
Moe, After doing more extensive testing, I came to realize that we had made a bad basic assumption. We believed that the user's environment should only be what was sent in the file via the --export-file option. However, that broke the previous behavior, especially in regard to Moab jobs. It also caused the SLURM defined environment variables to be lost as well. This patch will enable the correct behavior for Moab on top of SLURM whne using the --export-file option, but the behavior is less that pefect for using it stand alone with sbatch. When using the option with sbatch as a user, the file environment is read in, and then when the env_array_merge is made, some variables may get overwritten. This is good for the SLURM and MPI vairables, but not so good for others., The problem is trying to reconcile two sources of environment is very problematic. I also added a caveat in the man page. I made changes in my branch of SchedMD SLURM for 2.3, here is the patch. Phil
-
- 20 Jan, 2012 2 commits
-
-
Danny Auble authored
-
Morris Jette authored
Fix for possible invalid memory reference in slurmctld in job dependency logic. Patch from Carles Fenoy (Barcelona Supercomputer Center).
-
- 19 Jan, 2012 5 commits
-
-
Danny Auble authored
all jobs would be returned even if the flag was set. Patch from Bill Brophy, Bull.
-
Morris Jette authored
-
Morris Jette authored
We replaced references to "pipe" with a more generic "file descriptor". We also replaced a while loop in env.c with a for loop.
-
Morris Jette authored
-
Morris Jette authored
-
- 18 Jan, 2012 4 commits
-
-
Morris Jette authored
-
Morris Jette authored
-
Morris Jette authored
-
Morris Jette authored
Fix bug in --switch option with topology resulting in bad switch count use. Patch from Alejandro Lucero Palau (Barcelona Supercomputer Center).
-
- 17 Jan, 2012 2 commits
-
-
Morris Jette authored
-
Morris Jette authored
-
- 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 2 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
-