1. 01 Sep, 2015 1 commit
  2. 28 Aug, 2015 1 commit
    • Morris Jette's avatar
      Requeue job if possible when slurmstepd aborts · d8e6f55d
      Morris Jette authored
      This problem is reproducible by launching a job then killing the
        slurmstepd process. Under those conditions, requeue the job if
        possible (i.e. batch job with requeue option/configuration).
        This patch also improves the slurmctld logging when this happens.
      bug 1889
      d8e6f55d
  3. 27 Aug, 2015 2 commits
    • Morris Jette's avatar
      Correct RebootProgram usage · 82068b6b
      Morris Jette authored
      Correct RebootProgram logic when executed outside of a maintenance
        reservation. Previous logic would mark the node up upon response
        to the reboot RPC (from slurmctld to slurmc) and when the node
        actually rebooted, flag that as an unexpected reboot. This new
        logic checks the node's up time to not mark the compute node as
        being usable until the reboot actually takes place.
      but 1866
      82068b6b
    • Danny Auble's avatar
      Fix some potential deadlock issues when state files don't exist in the · f6bc60cc
      Danny Auble authored
      association manager.
      f6bc60cc
  4. 26 Aug, 2015 3 commits
  5. 25 Aug, 2015 2 commits
  6. 21 Aug, 2015 5 commits
  7. 20 Aug, 2015 1 commit
    • David Bigagli's avatar
      Revert "Fix bug in conversion unit." · 696a4917
      David Bigagli authored
      This reverts commit 406cb529.
      
      Conflicts:
      	NEWS
      
      This works ok up until 1048576M then it will make the divisor
      1048576 and when it goes to divide it only does it once instead of 2 times
      returning 1G instead of 1T.
      
      We have decided to wait until 15.08 for a no convert option to be added.
      696a4917
  8. 14 Aug, 2015 2 commits
    • Daniel Ahlin's avatar
      Non-default settings of AuthInfo fix · 02c96859
      Daniel Ahlin authored
      We are using a non-default AuthInfo configuration and based on
      log-messages we see I believe this is not properly handled in certain
      parts of the code.
      
      Typical log message:
      Aug 12 17:06:15 t02n20 slurmd[27001]: error: Munge encode failed:
      Failed to access "/var/run/munge/munge.socket.2": No such file or
      directory
      Aug 12 17:06:15 t02n20 slurmd[27001]: error: Creating authentication
      credential: Socket communication error
      Aug 12 17:06:15 t02n20 slurmd[27001]: error: stepd_connect to 3165.0
      failed: Protocol authentication error
      Aug 12 17:06:15 t02n20 slurmd[27001]: error: If munged is up, restart
      with --num-threads=10
      02c96859
    • Morris Jette's avatar
      Rebuild core reservation bitmap as needed · 931d1814
      Morris Jette authored
      Unfortunately the reservation's core_bitmap is a global bitmap and the nodes
      in the system may have changed in terms of their node count or nodes have been
      added or removed. Make best effort to rebuild the reservation's core_bitmap
      on the limited information currently available. The specific cores might
      change, but this logic at least leaves their count constant and uses the
      same nodes
      bug 1850
      931d1814
  9. 11 Aug, 2015 2 commits
  10. 06 Aug, 2015 1 commit
  11. 04 Aug, 2015 2 commits
  12. 03 Aug, 2015 2 commits
  13. 31 Jul, 2015 1 commit
  14. 28 Jul, 2015 4 commits
  15. 27 Jul, 2015 1 commit
  16. 22 Jul, 2015 2 commits
  17. 21 Jul, 2015 2 commits
    • Chandler Wilkerson's avatar
      fix incorrect reading of cpuinfo on POWER systems · 962dea86
      Chandler Wilkerson authored
      This patch provides a rewrite of how /proc/cpuinfo is parsed in common_jag.c, as the original code made the incorrect assumption that cpuinfo follows a sane format across architectures ;-)
      
      The motivation for this patch is that the original code was producing stack smashing on a POWER7 running RHEL6.4 Red Hat adds -fstack-protector along with a lot of other protective CFLAGS when building RPMs. The code ran okay with -fno-stack-protector, but that is not the best work-around.
      
      So, the relevant /proc/cpuinfo line on an Intel (Xeon X5675) system looks like:
      
      cpu MHz                : 3066.915
      
      Whereas the relevant line in a POWER7 system is
      
      clock                : 3550.000000MHz
      
      My patch replaces the assumption that the relevant number starts 11 characters into the string with another assumption: That the relevant number starts two characters after a colon in a string that matches (M|G)Hz.
      
      All in all, the function has a few more calls, which may be a performance issue if it has to be called multiple times, but since the section I edited only gets evaluated if we don't know the cpu frequency, getting it right will actually result in fewer string operations and unnecessary opens of /proc/cpuinfo for systems likewise affected.
      
      Finally, I also read the actual value into a double and multiply it up to the size indicated by the suffix, so we end up with KHz? It was unclear what the original code intended, since it matched both MHz and GHz, replaced the decimal point with a zero, and read the result as an int.
      
      --
      Chandler Wilkerson
      Center for Research Computing
      Rice University
      962dea86
    • Danny Auble's avatar
      Revert "ALPS - Remove sanity code to work like it did in 2.5. This is an addition" · 6ab26aa6
      Danny Auble authored
      This reverts commit 2c95e2d2.
      
      Conflicts:
      	src/plugins/select/alps/basil_interface.c
      
      This is related to bug 1822.  It isn't clear why the code was taken out in
      this commit in the first place and based off of commit 2e2de6a4 (which is
      the reason for the conflict) we tried unsuccessfully to put it back.
      
      It appears the only difference here is the addition of
      always setting mppnppn = 1 instead of always to
      job_ptr->details->ntasks_per_node when no ntasks is set.
      
      This appears to only be an issue with salloc or sbatch as ntasks
      is always set for srun.
      6ab26aa6
  18. 18 Jul, 2015 1 commit
  19. 17 Jul, 2015 4 commits
  20. 16 Jul, 2015 1 commit