cms-sw / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
1.07k stars 4.28k forks source link

LHESource does not support xrootd fallback #28241

Open Dr15Jones opened 4 years ago

Dr15Jones commented 4 years ago

As brought up on the Workflow Management talk in the Fall O&C Week, not having xrootd fallback causes difficulty for WM.

cmsbuild commented 4 years ago

A new Issue was created by @Dr15Jones Chris Jones.

@davidlange6, @Dr15Jones, @smuzaffar, @fabiocos, @kpedro88 can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

Dr15Jones commented 4 years ago

FYI @bbockelm @dan131riley

Dr15Jones commented 4 years ago

assign core, generator

cmsbuild commented 4 years ago

New categories assigned: core

@Dr15Jones,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks

Dr15Jones commented 4 years ago

assign generators

cmsbuild commented 4 years ago

New categories assigned: generators

@alberto-sanchez,@SiewYan,@qliphy,@efeyazgan,@mkirsano,@agrohsje you have been requested to review this Pull request/Issue and eventually sign? Thanks

dan131riley commented 4 years ago

LHEReader just calls StorageFactory::open()

https://github.com/cms-sw/cmssw/blob/35176b9685b0361db09ba627df8cde4e14d73d09/GeneratorInterface/LHEInterface/src/LHEReader.cc#L52

If I understand correctly, that doesn't use the InputFileCatalog at all, so I guess they must be currently using PFNs, not LFNs? If that's correct, there'd be two parts to adding AAA fallback--part 1 would be to add an InputFileCatalog to the LHESource and add fallback logic, and part 2 would be to switch to using LFNs in the workflows.