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

Conflicts:
	src/slurmctld/reservation.c

If an advanced reservation contains no nodes (e.g. licenses or
  burst buffers only), then avoid logic used to replace failed
  or failing nodes which would reference a node bitmap that is
  NULL, which would result in an abort

Correct some logic for limit enforcement, including change for
  emulated Cray system and change some variable names for better
  clarity

Added flag of EmulateCray and dw_wlm_cli script