Fix tracked nEvents parameter in ExternalLHEProducer (73x)

[bendavid]

This is the proper solution for the issue discussed in
https://hypernews.cern.ch/HyperNews/CMS/get/dataopsrequests/5669.html

By making the nEvents paramter untracked, the spurious run transitions should be avoided.

[bendavid]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes or unless it breaks tests. @nclopezo, @ktf can you please take care of it?

[cmsbuild]

+1
Tested at: fdd05cb
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5818/72/summary.html

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5818/72/summary.html

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?

[franzoni]

Hello @bendavid ,

since we don't have an ETA yet for the changes in wmagent which will allow this transition on the CMSSW side (https://hypernews.cern.ch/HyperNews/CMS/get/wmDevelopment/642/1/1/1/1/1.html)
can I suggest (as we discussed briefly in person) that we revert this change for nEvents and defer it to the right moment ?

While there's no plans of GEN-SIM with 73x,
this detail may get forgotten,
land one day in production,
and create a big mess :-)

Ciao,
Giovanni

[franzoni]

Things have been actually set in motion on the wma side: https://hypernews.cern.ch/HyperNews/CMS/get/wmDevelopment/642/1/1/1/1/1/1.html

At some point (~ next week) we'll receive a call from @amaltaro /ops to place a request with nEvents set to untracked.

@bendavid, @srimanob @boudoul : can we think of using 73x for a GEN-SIM test request injection ?
That would cost setting up a dummy campaign.
The alternative would be moving forward and changing the nEvents type in 53x, but that would mean testing with a heavily-production release ...

[bendavid]

@franzoni, since we said we wanted this fixed before we start the 71x wmlhe campaign, an alternative could be to backport it to 71x, and create the RunIIFall14wmLHE campaign which will anyways be needed.

It is sufficient to test this with only a wmLHE request btw, so we can also just create the dummy campaign (or move one of the existing wmlhe test campaigns to 7_3_0_pre1). wmLHE production has been privately tested in 73x with existing gridpacks, so we could quickly submit a test request there.

[bendavid]

We should also tested an old/unfixed release on the testbed to make sure it is really working with both the tracked and untracked parameter.

[amaltaro]

nEvents changes were applied to ReqMgr in cmsweb-testbed, could you guys please inject a couple of requests? Should we follow up it here or in the HN thread? Thanks