- gid instead of uid used in group_cache_lookup_job().
- needle->username is optional, and will lead to a failure
  in getgrouplist() when NULL is passed in.

Hat tip to Brian for discovering.

in salloc, sbatch, srun before process the command line options.
Bug 3499

CID 177191.

Modify _open_as_other so that both _file_bcast_register_file
and _prolog_error can both use it. (That code was originally
extracted from _file_bcast_register_file, but tailored to
_prolog_error's requirements. Less duplication is good.)

In slurmctld:
  - Handle the lookup, and send results across.
  - Periodically flush the cache.

In slurmd:
  - Load value sent as part of prolog into cache
  - Have task launch use the new call instead to fetch
    the jobid-cached values when available
  - Remove old job-specific records from the cache as
    part of the prolog.
  - Ignore the slowly increasing memory footprint from the cache,
    as this is what the old implementation did anyways. It'll be
    O(users in cluster) at worst, and flushes on a reconfigure.
  - Simplify the slurmstepd communication path. group_cache_lookup
    will always succeed.

Left to do:
  - Convert other lookup locations in slurmd to the new approach,
    and remove the compatibility shim.

Bug 3322.

Loosely based on src/slurmd/req.c. But with major alterations
in preparation for use with PrologFlags=SendGIDs.

Convert slurmd to use a compatibility shim in here for now.

Bug 3322.

The slurmd is always sending the ngid / gids over
the pipe, so the setgroups() call is all that matters
here. Move it out to _drop_priviledges instead.

Bug 3322.

Patch was broken by a rebase before pushing. Fixing as a clean patch.

This reverts commit b0838485.

Regression caught my slight oversight from 88d5a801.

Bug 3322.

Always false in all supported plugins, except for BGQ.
Prior commit removed all calling paths.

mpi_hook_client_single_task_per_node() is always false,
except on BGQ systems. Move special handling code into
an #ifdef so it can be readily identified and removed
when BGQ support is retired.

CI 177110.

Otherwise the step count comes back incorrectly.

Bug 4157.

Bug 4147.

Was used with Moab integration, which was removed in 17.11.

Adding this cast trick prevents a "the address of 'id' will always
evaluate as 'true' warning message from GCC.

==9617== 42 bytes in 1 blocks are definitely lost in loss record 3,654 of 7,443
==9617==    at 0x4C2FB45: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9617==    by 0x1A4A81: slurm_xmalloc (xmalloc.c:84)
==9617==    by 0x1A505C: makespace (xstring.c:102)
==9617==    by 0x1A51A6: _xstrcat (xstring.c:133)
==9617==    by 0x1A58B6: _xstrfmtcat (xstring.c:292)
==9617==    by 0x3181D7: parse_resv_nodecnt (state_control.c:213)
==9617==    by 0x17611A: _set_resv_msg (resv_info.c:352)
==9617==    by 0x17650A: _admin_focus_out_resv (resv_info.c:472)

==9617== 136 bytes in 1 blocks are definitely lost in loss record 6,082 of 7,443
==9617==    at 0x4C2DB2F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9617==    by 0x63FF80F: __vasprintf_chk (vasprintf_chk.c:80)
==9617==    by 0x5C3E0D8: g_vasprintf (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.5200.0)
==9617==    by 0x5C18BAC: g_strdup_vprintf (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.5200.0)
==9617==    by 0x5C18C68: g_strdup_printf (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.5200.0)
==9617==    by 0x175F7A: _set_resv_msg (resv_info.c:306)
==9617==    by 0x17650A: _admin_focus_out_resv (resv_info.c:472)
==9617==    by 0x4F6B92B: ??? (in /usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0.2400.31)

Bug 2329.

Bug 2329.

This reverts commit 8ebe5f2ca0bd8b241273094f820f63f5d32a8752.

Bug 2329

Bug 2329

Bug 2329

sview Watts parsing and display was introduced in commit a503a770.
There were differences between scontrol and sview, so this commit moves
the way we parse and display reservation Watts to a common place and make
it so sview and scontrol use these functions. The parse and display
functions have been slightly refactored as compared to the original ones.

Bug 2329