[Backport to 9_4_X] Allow to suppress ExternalLHEProducer exception if not all events were processed

[justinasr]

PR description:

This PR enables to suppress an exception thrown by ExternalLHEProducer using a VALIDATION_RUN environment variable.

PR validation:

• scram b runtests tests

If this PR is a backport please specify the original PR and why you need to backport that PR:

Backport of #33624 including suggestion added with #33674.
Backport is needed in all releases that are still used in Monte Carlo production.

[cmsbuild]

A new Pull Request was created by @justinasr (Justinas Rumševičius) for CMSSW_9_4_X.

It involves the following packages:

GeneratorInterface/LHEInterface

@SiewYan, @mkirsano, @cmsbuild, @GurpreetSinghChahal, @agrohsje, @alberto-sanchez can you please review it and eventually sign? Thanks.
@alberto-sanchez, @mkirsano this is something you requested to watch as well.
@silviodonato, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

[agrohsje]

please test

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b1a59c/16528/summary.html
COMMIT: 5bc451e
CMSSW: CMSSW_9_4_X_2021-07-04-0000/slc6_amd64_gcc630
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/34359/16528/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 27
• DQMHistoTests: Total histograms compared: 2721493
• DQMHistoTests: Total failures: 105
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2721226
• DQMHistoTests: Total skipped: 162
• DQMHistoTests: Total Missing objects: 0
• TriggerResults: no differences found

[agrohsje]

@qliphy @silviodonato there are 105 failures. The new if will not be entered in the above tests so everything should be identical as the tests of 10_6 showed. So I am happy to sign. But let me know if I am missing something.

[qliphy]

@agrohsje The failures seem to be fake similarly as reported in #29365

[agrohsje]

Thanks @qliphy . Same is also true for 8_0:
#34361 (comment)

[agrohsje]

+generators

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_9_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_0_X is complete. This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2)

[qliphy]

+1