per Paul's suggestion I added the shebang line to the --directives output so it now produces output that can be redirected to a file and used as a script input to PBS.  I also added a # before the two note lines so they are commands and don't cause errors in the script

changed resource name from host (already defined) to system, added the missing = in the select (how the hell did that disappear?) and also caught another error.  Need to add the -- when directly running an executable, which required another command line option

Cluster node state change accounting log

See merge request aig/cobalt!125

Merge remote-tracking branch 'refs/remotes/origin/cluster_node_state_change' into cluster_node_state_change

removing location from the email subject.  The subject line is too long on...

See merge request aig/cobalt!124

Make cluster systems respect --attrs location

See merge request aig/cobalt!123

Enhance reservation record information

See merge request aig/cobalt!122

removing location from the email subject.  The subject line is too long on large systems, otherwise.

fix for a display issue with cluster node names that have additional device information.

See merge request aig/cobalt!121

Addressed comments from code review and found a further corner
case while working on those with one-off nodenames that sometimes
get added to clusters.  Tests added and behavior fixed for those.

Resolve "Cray system draining flap"

Closes #158

See merge request aig/cobalt!120

A node was getting added to the candidate list twice when looking over
currently running jobs. resulting in a duplicate node and an inflated
count which resulted in an early break.  When this list is later
converted to a set, the duplicates are removed and results in a set
of nodes too small for the job to use for draining, so the job doesn't
get drained for.

Test case added for the drain flap now works correctly now that a check
has been added so a duplicate node doesn't get added.