- 12 Aug, 2018 28 commits
-
-
Tim Wickberg authored
Add xasserts() to crash development builds if the wrong struct is fed in, and fall back to checking the field and returning 'CORRUPT' if the magic value is not found for production builds rather than print garbage (and risk an out-of-bounds memory access).
-
Tim Wickberg authored
Set when step is created, xassert() when the step is being destroyed.
-
Tim Wickberg authored
Give the xassert a chance to fail instead of an invalid memory access. Also help avoid issues with new logging code if the wrong struct was passed in.
-
Tim Wickberg authored
Need to set found_other_formats to still consume the %.0s if not other format strings are included.
-
Tim Wickberg authored
Use the embedded pointer back to the job_record struct to print that followed by the step_id value.
-
Tim Wickberg authored
-
Tim Wickberg authored
Also switch to __func__ and plugin_type where applicable.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Further work to convert StepId structures will be handled later.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Do this instead of looking the job_id back up immediately. Convert this to %pJ while here, and fix incorrect comments for srun_allocate_abort().
-
Tim Wickberg authored
Rather than pulling the job_id out of job_ptr just to have it call find_job_record() to find it again, just cut out the indirection.
-
Tim Wickberg authored
Switch a few fatal() log messages to fatal_abort() where appropriate.
-
Tim Wickberg authored
-
Tim Wickberg authored
Permits use of %pJ. Convert the upstream calling location _preempt_jobs() to %pJ while changing its signature as well.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Switch to plugin_type and __func__ while converting in select_serial.c.
-
Tim Wickberg authored
Allows for %pJ conversion.
-
Tim Wickberg authored
Allows conversion to use %pJ to print in canonical format, although some care is needed here to avoid introducing a circular dependency between structs from slurmctld.h and job_resources.h.
-
Tim Wickberg authored
-
Tim Wickberg authored
No longer print out the job name field in debug messages as well. Change to __func__ in messages while editing them.
-
Tim Wickberg authored
-
- 11 Aug, 2018 12 commits
-
-
Tim Wickberg authored
-
Tim Wickberg authored
Standardize on "JobId=..." where the job_ptr is not available. Change function signature for _write_nid_file to use the job_ptr instead.
-
Tim Wickberg authored
-
Tim Wickberg authored
Convert any changed log messages to __func__ while here as well, and standardize on "JobId=" for locations that cannot use %pJ.
-
Tim Wickberg authored
-
Tim Wickberg authored
Slurm has used used both "JobID" and "JobId", but treat the latter as the canonical format given the long-standing use in slurm_sprint_job_info(), and several user-visible locations in scontrol.
-
Brian Christiansen authored
-
Tim Wickberg authored
-
Danny Auble authored
This seems fairly verbose. We can change it back if/when needed.
-
Tim Wickberg authored
Update slurm.spec and slurm.spec-legacy as well. Fix NEWS header.
-
Tim Wickberg authored
-
Danny Auble authored
-