[Condor-devel] minor change: negotiator prints startd ranks when it preempts, any objections?

[Erik Paulson <epaulson@xxxxxxxxxxx>]

If we preempt something, put into the log file what the old and
new startd rank was to help debugging.

(I did a basic test, my negotiator didn't segfault when it preempted
for startd rank :)

-Erik


cvs diff: Diffing .
Index: matchmaker.C
===================================================================
RCS file: /p/condor/repository/CONDOR_SRC/src/condor_negotiator.V6/matchmaker.C,v
retrieving revision 1.71.10.19.10.8.12.31
diff -r1.71.10.19.10.8.12.31 matchmaker.C
1896a1897,1904
> 
>                               float newStartdRank;
>                               float oldStartdRank = 0.0;
>                               if(! offer->EvalFloat(ATTR_RANK, &request, newStartdRank)) {
>                                       newStartdRank = 0.0;
>                               }
>                               offer->LookupFloat(ATTR_CURRENT_RANK, oldStartdRank);
> 
1898,1901c1906,1909
<                               dprintf( D_ALWAYS, "      Preempting %s (prio=%.2f) on %s "
<                                                "for %s (prio=%.2f)\n", remoteUser,
<                                                remotePriority, remoteHost, scheddName,
<                                                priority );
---
>                               dprintf( D_ALWAYS, "      Preempting %s (prio=%.2f, rank=%.2f) on %s "
>                                                "for %s (prio=%.2f, rank=%.2f)\n", remoteUser,
>                                                remotePriority, oldStartdRank, remoteHost, scheddName,
>                                                priority, newStartdRank );