Use the embedded pointer back to the job_record struct to print that
followed by the step_id value.

Also switch to __func__ and plugin_type where applicable.

Further work to convert StepId structures will be handled later.

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().

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.

Switch a few fatal() log messages to fatal_abort() where appropriate.

Permits use of %pJ. Convert the upstream calling location _preempt_jobs()
to %pJ while changing its signature as well.

Switch to plugin_type and __func__ while converting in select_serial.c.

Allows for %pJ conversion.

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.

No longer print out the job name field in debug messages as well.

Change to __func__ in messages while editing them.

Standardize on "JobId=..." where the job_ptr is not available.

Change function signature for _write_nid_file to use the job_ptr instead.

Convert any changed log messages to __func__ while here as well,
and standardize on "JobId=" for locations that cannot use %pJ.

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.

This seems fairly verbose.  We can change it back if/when needed.

Update slurm.spec and slurm.spec-legacy as well.

Fix NEWS header.

boot delay.

Code to determine if we need to reboot or not based on node_features

Bug 5308

Fixes bad casting caused by Tim on review.

Make one definitive copy of sort_part_tier() in partition_msg.c and use it.

Bug 3844, 5552.

and other drain + node state flags.

Bug 5505