instead of sometimes calling functions directly.

As well as jobcomp functions.

This also renames some to match the calls of other things.

Bug 4292

There is more possible this was just low hanging fruit.

This is needed to figure out the unused wall clock time on them.

Bug 4246

No other options use underscores, so make these consistent with
the rest. Both old and new syntax is supported for now.

Change all 'verbose' option output to use dashes, and adjust
the testsuite to match.

Fix a few related documentation errors while here.

Turns out what was there before was completely incorrect.

Bug 4246

If configured with NodeFeatures=knl_cray and there are non-KNL
  nodes which include no features the slurmctld will abort without
  this patch when attemping xstrdup(NULL).
bug 4294

the same id.

There is no /slurm page, this was missed in the new style conversion.

Display the commands that are being exec'd

nodes and didn't specify the cluster.  This shouldn't matter for sacctmgr after
the last commit.

with sacctmgr.

reservations instead of adding them which will create a new record in the
database with correct node indexes.

end_time second in the query, it will be the same as the start time of the
next cluster in the list.

We only need to fill in the end time for the 'slurmctld disconnect' if
nothing has changed.  Otherwise the very next update will do it for everything.

This hasn't been needed for a while and is now handled
in the other 'ifs' above it.

ACCOUNTING_TRES_CHANGE_DB.

In a future commit we will code what to do with these in the slurmctld.

This will be used to tell if we are 17.02 or less when sending in the TRES.

We need this to determine if we can return new rc codes.  This can be reverted
if so desired in 18.08.

Bug 4279

(-O1 makes this happen).

This should be handled differently in the future meaning Slurm should figure
out the null part and not rely on *printf to do that correctly.