ok it appears we have association based accounting done with reservations.  More testing is needed for rollups but it appears to work.

    associated with nodes. Patch from Kumar Krishna.

  (don't require completed)

    if not specified use MaxTime for the partition. Patch from Par
    Andersson, National Supercomputer Centre, Sweden.

job id is completed

    given list of nodes.

  for command of this type "scontrol update reservationname=x users-=x

Fix parsing problem when adding or removeing accounts/users to an existing reservation

preserve a reservation's start_time_prev value

first association record and just use NO_VAL to match the account's 
association record

 is either COMPLETED or COMPLETING (which can hang for some time)

Set a job's resv_ptr only on slurmctld restart or reconfig, 
  then use xassert() rather than redundant calls to find a 
  reservation by name.

Fix logic for testing if reservations overlap considering DAILY reservation flag