Signed-off-by: Danny Auble <da@schedmd.com>

Add tres_id to state (in addition to tres_pos).
Add calls to post persistent buffer create/delete to accounting
  using advanced reservation data structure.
Add ID to buffer for use in accounting.
Fix short malloc bug.
Fix for unitialized variable.
Fix locking problem (wrong variable previous used).

Remove duplicate should in configurator

Add infrastructure for persistent burst buffer accounting.
Treat them like an advanced reservation.
Add association string to buffer record.
Preserve state in emulation mode

If an advanced reservation contains burst buffer space, store that
information in the accounting database (previously was NULL).

Prevent duplicate buffer allocation for interactive jobs (salloc/srun).

The files used by datawarp have been updated, this change to Slurm
  logic reflects this change in DataWarp.

and INFINITE instead of the current NO_VAL64 and INFINITE64

Prevent possible problem converting from double to uint32_t.
Make sure that comparison of double to 1.0 when cast, does not
become a uint32_t value of zero by doing the type cast before
comparision

Export DataWarp environment variables to the srun environment.

This is a correction to 78c0bf9a
which could result in divide by zero and slurmstepd abort.
bug 1748

nice.

commit will make it limit

The burst_buffer/cray plugin sets some environment variables for
  use by the user. This adds those environment variables to the
  salloc command