- 19 Oct, 2018 31 commits
-
-
Tim Wickberg authored
-
Tim Wickberg authored
The industry's settling on a term different than what we'd originally gone with, so make a concession to the newer term and make it easier to search for and discover.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Just return an xmalloc()'d string rather than handle all this manual buffer length tracking. The only calling path into here is trivially updated to work with this instead. Note that gres_plugin_init() always returns 0 at present, so take the opportunity to simplify that handling as well.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
The safe_read() and safe_write() macros are the better option, and these were only used in one location each. (Converted in previous patch.)
-
Tim Wickberg authored
-
Tim Wickberg authored
xmalloc cannot return NULL here - on error it would fatal() already.
-
Tim Wickberg authored
Fix function declaration and code to match style guide. Change the debug3 message to print raw timestamps, rather than convert, same as the other debug3 messages. This also then removes the last calls to timestr(), so remove that static function as well.
-
Tim Wickberg authored
strcpy would not have worked for the described purpose anyways.
-
Tim Wickberg authored
I was going to rework these to remove the strcpy() calls, but found that each of these have equivalent debug3() statements just without the timestamps rendered as time strings. Given that these are debug3(), and that you'd need to manually inject a DEBUG_TIME definition somewhere to expose them, I don't see the value of keeping them around. Especially as if you're digging this far in to the source, it seems easier to troubleshoot off the raw timestamp anyways IMNSHO.
-
Tim Wickberg authored
Replace strcpy() with strlcpy().
-
Tim Wickberg authored
Replace strcpy() with strlcpy(), and refactor the code. Fix the function prototype to match the style guide while here.
-
Tim Wickberg authored
Replace strlen(), xmalloc(), strcpy() sequence with xstrdup(). Add const annotation and fix function declaration style while here.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Cleanup other code around here as well, and replace msg_type with msg->msg_type now that select/serial cannot swap that out. Cleanup other src/slurmd locations as well.
-
Tim Wickberg authored
Without select/serial, replace_batch_job() collapses to a call to slurm_send_rc_msg(msg, SLURM_SUCCESS) and then returns false. So just do that instead, and delete this entire function.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Bug 5713.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
This is not recommended for use, and will be removed in 19.05.
-
Tim Wickberg authored
-
- 18 Oct, 2018 9 commits
-
-
Tim Wickberg authored
-
Tim Wickberg authored
Update slurm.spec and slurm.spec-legacy as well.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Update slurm.spec and slurm.spec-legacy as well.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Update slurm.spec and slurm.spec-legacy as well.
-
Tim Wickberg authored
-