Skip to content
GitLab
  1. 08 Nov, 2017 1 commit
  3. 07 Nov, 2017 3 commits
    • Danny Auble's avatar
      Make reoccurring reservations show up in the database. · 9afcf661
      Danny Auble authored 
      This requires changing the resv_id and then also updating that on all
the jobs.  Since this requires having the job write lock we spawn
a new thread instead of altering all the paths into
_advance_resv_time().

Bug 4246
      9afcf661
    • Alejandro Sanchez's avatar
      Fix issue when resetting the partition pointers on nodes. · e6b2bd2d
      Alejandro Sanchez authored 
      Issue could be triggered when updating a partition node(s) with node(s)
that were already in the partition, incorrectly increasing the
node_record->part_cnt (number of associated partitions) and thus
incorrectly extending the array of pointers to partitions associated
with this node, leading to an array with repeated associated
partitions pointers.

Bug 4318.
      e6b2bd2d
    • Brian Gilmer's avatar
      Cray module file - remove munge support. · 8b71b9fc
      Brian Gilmer authored 
      On CLE 6.0 mungedir is /usr; a 'module unload' call then removes /usr/bin
from PATH which is rather inconvenient.

Bug 4334.
      8b71b9fc
  5. 06 Nov, 2017 1 commit
  7. 03 Nov, 2017 1 commit
    • Isaac Hartung's avatar
      Fix updating of requested TRES memory · 994e4f5c
      Isaac Hartung authored 
      Memory TRES was getting the pn_min_memroy value when updating the job.
But the TRES memory value is the total memory of the job
(e.g pn_min_memory * cpus || pn_min_memory * nodes).

Bug 4177
      994e4f5c
  9. 02 Nov, 2017 1 commit
  11. 01 Nov, 2017 4 commits
  13. 31 Oct, 2017 6 commits
  15. 30 Oct, 2017 2 commits
  17. 28 Oct, 2017 2 commits
  19. 27 Oct, 2017 9 commits
  21. 25 Oct, 2017 2 commits
    • Danny Auble's avatar
      Fix layouts code to only allow setting a boolean. · b9273782
      Danny Auble authored 
      Before it would allow all sorts of things like
adding/subtracting/multiplying/etc.  It would cause warnings such as

/home/bart/slurm-tmp5/src/common/layouts_mgr.c: In function ‘_layouts_load_automerge’:
/home/bart/slurm-tmp5/src/common/layouts_mgr.c:363:21: error: ‘*’ in boolean context, suggest ‘&&’ instead [-Werror=int-in-bool-context]
   *lvalue = *lvalue * *rvalue;     \
             ~~~~~~~~^~~~
/home/bart/slurm-tmp5/src/common/layouts_mgr.c:1034:4: note: in expansion of macro ‘_entity_update_kv_helper’
    _entity_update_kv_helper(type_t, operator); \
    ^~~~~~~~~~~~~~~~~~~~~~~~
/home/bart/slurm-tmp5/src/common/layouts_mgr.c:1086:4: note: in expansion of macro ‘_layouts_load_merge’
    _layouts_load_merge(bool, s_p_get_boolean);
[tag] [reply] [−] Private Comment 16

Bug 4062
      b9273782
    • Felip Moll's avatar
      Work around issue with sysmacros.h and gcc7 / glibc 2.25. · 8706f388
      Felip Moll authored 
      Setting -Werror tricks the test into failing on the

"error: In the GNU C Library, "major" is defined by <sys/sysmacros.h>.
 For historical compatibility, it is  currently defined by <sys/types.h>
 as well, but we plan to remove this soon. To use "major", include
 <sys/sysmacros.h> directly. If you did not intend to use a system-defined
 macro "major", you should undefine it after including <sys/types.h>."

error. Since the normal Slurm build uses -Werror, this warning on including
both headers will then cause the build itself to fail.

Bug 3982.
      8706f388
  23. 24 Oct, 2017 8 commits