I'm tired of doing this manually.

Use new uid_to_string_or_null call instead; slurmd will fill in
if the lookup on the controller fails.

Adjust _rpc_prolog to handle the lack of a set user_name field,
as this was always set to something before.

And safe to/from state files, and send as part of job_info_t.

This is not always set, but if set can avoid a lookup call.

Create it by using the code from uid_to_string, making uid_to_string
use this but do the null handling logic before returning.

Send ngids/gids if requested by config. Make sure to
free the correct values here.

Formatting only; no functional changes.

And deal with marshelling them around the various structs.

A functional change for once. :)

No functional change, just gets a tab stop back.

Shuffle around the calling paths to make use of this instead
of passing in everything as an argument.

Again - no function changes implied, just further groundwork.

It's already accessible through req.

Setup 17.11 RPC format blocks while here, but leave them identical to
the previous format. Also get rid of the temporary len variable.

No functional change, as long as protocol_version has been set correctly.

Fixup calling paths to send the appropriate version numbers.

No functional change yet, this is ground work for that.

Base time limit upon the time that the latest job component is available
(after all nodes in all components booted and ready for use).

Bug 3909

Bug 4131

Bug 4131

Bug 4131

- refactor mpi/pmix plugin code to support multiple PMIx version without code duplications;
- add the support for PMIx version 2 API;
- address the issues caused by Slurm and PMIx component infrastructures.

Signed-off-by: Boris Karasev <karasev.b@gmail.com>

This reverts commit 2b483659.

when using xstrcasecmp. Matching up with other xstrcmp() functions.

was missing

Bug 4226

Was calling acct_policy_job_begin() inside of already locked
assoc_mgr_locks.

From 419f1fd0

Bug 4185

Revisions to commit 3d9c3fa26ad9775da86978ef30c27308dbff0e3b
Add hostlist_destroy() calls to avoid memory leaks
Add check for empty line
Avoid treating '\' as next to last character in a file as a line
  continuation
bug 4185

Fixes line continuation with srun --multi-prog option.
Bug 4185

Bug 3467

handled.

Bug 3467

Avoid API call, just grab sched_params directly out of the
struct.

Be less cute with the conditional, just spell out the logic cleanly.