Skip to content
This repository was archived by the owner on Sep 12, 2024. It is now read-only.

Commit 169e65c

Browse files
committed
apply Sharad and Jean-Roch suggestions
1 parent 9cdb28f commit 169e65c

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

Unified/assignor.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -219,6 +219,7 @@ def rank( wfn ):
219219
for prim in list(primary):
220220
set_lfn = getLFNbase( prim )
221221
## if they are requested for processing, they should bbe all closed already
222+
# FIXME: remove this closeAllBlocks
222223
closeAllBlocks(url, prim, blocks)
223224

224225
## should be 2 but for the time-being let's lower it to get things going
@@ -400,6 +401,8 @@ def pick_campaign( assign_parameters, parameters):
400401
parameters['EventsPerJob'] = eventsPerJob
401402
else:
402403
spl = wfh.getSplittings()[0]
404+
# FIXME: decide which of the lines below needs to remain...
405+
eventsPerJobEstimated = spl['events_per_job'] if 'events_per_job' in spl else None
403406
eventsPerJobEstimated = spl['avg_events_per_job'] if 'avg_events_per_job' in spl else None
404407
if eventsPerJobEstimated and eventsPerJobEstimated > eventsPerJob:
405408
#sendEmail("setting lumi splitting for run-dependent MC","%s was assigned with %s lumis/job"%( wfo.name, lumisPerJob))

Unified/injector.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,7 @@ def injector(url, options, specific):
222222
parser = optparse.OptionParser()
223223
parser.add_option('-i','--invalidate',help="fetch invalidations from mcm",default=False,action='store_true')
224224
parser.add_option('-w','--wmstatus',help="from which status in req-mgr",default="assignment-approved")
225-
parser.add_option('-s','--setstatus',help="What status to set locally",default="considered")
225+
parser.add_option('-s','--setstatus',help="What status to set locally",default="staged")
226226
parser.add_option('-u','--user',help="What user to fetch workflow from",default="pdmvserv")
227227
parser.add_option('-r','--replace',help="the workflow name that should be used for replacement",default=None)
228228
parser.add_option('--user_relval',help="The user that can inject workflows for relvals", default=None)

0 commit comments

Comments
 (0)