only the fields we care about from the database instead of always getting
everything.

The function should probably be a extern so we can use it in other packing.
But that will wait for some future day.

Requires a bit of shuttling around to pass the sbcast_cred_arg_t
to where it's needed. Adds additional checks that someone hasn't
managed to steal an sbcast credential somehow by comparing uid/gid
between auth header and the cred.

Remove some stray xfree(gids) calls that cause a double-free; before
the gids were fetched only for this call path, but now they're
handled as part of the cred.

It's not clear how this would ever happen - there is no
'sbcast --no-alloc', and 'srun --no-alloc --bcast' doesn't work.

Don't keep looking them up on each step / sbcast cred creation.
If they're not there, then slurmd will fill them in for us.

(Same technique used on user_name field.)

Much tidier. Should have done that many commits ago.

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