Project

General

Profile

Bug #5909

Bug #5654: Glidein summary broken for multicore glideins

Scale down the monitoring info on number of jobs run by the glideins

Added by Parag Mhashilkar almost 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Igor Sfiligoi
Category:
-
Target version:
Start date:
04/15/2014
Due date:
% Done:

0%

Estimated time:
First Occurred:
Occurs In:
Stakeholders:
Duration:

Description

Following changes from #5654 did not make into v3_2_4.

Updated by Igor Sfiligoi 7 days ago
Status changed from Closed to Feedback
Comment Edit
After some more experience, Jeff and I think the current solution is not fully satisfactory.

I.e. we do not scale down the number of jobs by the number of cores used.
(while we do it for walltime)

This is problematic for 2 reasons:
Different attributes have different semantics
It screws up the avg job duration calculation in the monitoring
I thus propose to scale down the number of jobs.
This will require rounding, since this count is supposed to be an integer.
I propose to round it up all the time (e.g. 1.25 -> 2.0)

If I don't hear otherwise, I am going to implement this ASAP.

Updated by Igor Sfiligoi 6 days ago
Assignee changed from Igor Sfiligoi to Parag Mhashilkar
Comment Edit
I have made the changes, and comitted them to a new branch (out of latest branch_v3_2)
called v3/5654_p2

Please review.

Update Log time Unwatch Copy Delete

History

#1 Updated by Parag Mhashilkar almost 6 years ago

  • Status changed from New to Assigned
  • Assignee changed from Parag Mhashilkar to Igor Sfiligoi

Please use the ceil() function in awk rather than indirect means like dstr=sprintf("%d",t+0.99);

Also, you can easily use the something like following which is much easy to read and faster. My example doesn't check for the accuracy so please modify the suggestion accordingly.
diff = tcnt - tj
tgnz = tgnz-diff
tcnt = tcnt-diff

to replace while (tcnt>tj) {tgnz=tgnz-1; tcnt=tcnt-1;}
and other 3 such instances.

#2 Updated by Igor Sfiligoi almost 6 years ago

There is no standard ceiling function in awk.
ceil() definitely does not work (at least on some platforms).

#3 Updated by Igor Sfiligoi almost 6 years ago

Changed iterators into a diff.

Code now in v3/5909.

#4 Updated by Igor Sfiligoi almost 6 years ago

  • Status changed from Assigned to Resolved

Merged back into branch_v3_2 and master.

#5 Updated by Parag Mhashilkar over 5 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF