Include EndLuminosityBlock transition in the esConsumes for LumiAlCaRecoProducer

[tvami]

PR description:

Resolves the issue mentioned here
#36095 (comment)
by including EndLuminosityBlock transition in the esConsumes for LumiAlCaRecoProducer

PR validation:

The previously offended wf,
runTheMatrix.py -l 1020.0
runs fine now.

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

N/A

[tvami]

type bug-fix

[tvami]

test parameters:

• workflows = 1020.0

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36114/26616

• This PR adds an extra 12KB to repository

[cmsbuild]

A new Pull Request was created by @tvami (Tamas Vami) for master.

It involves the following packages:

• Calibration/LumiAlCaRecoProducers (alca)

@cmsbuild, @malbouis, @tvami, @yuanchao, @francescobrivio can you please review it and eventually sign? Thanks.
@mmusich, @tocheng this is something you requested to watch as well.
@perrotta, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

[tvami]

@cmsbuild , please test

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-701241/20517/summary.html
COMMIT: e6f27f9
CMSSW: CMSSW_12_2_X_2021-11-12-2300/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/36114/20517/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

• @fwyzard scripts for running MPI jobs in the CMSSW environment #36109
• @slava77 speedup: use cached RhoEtaPhi from caloCellGeometry in EgammaHcalIsolation #36108
• @francescobrivio Migrate BeamSpot codes to new PoolDBOutputService methods #36077
• @cvuosalo esConsumes migration of CondTools/Geometry/test/GeometryTester.cc  #36107
• @makortel Modernize several modules in TopQuarkAnalysis #36098
• @swagata87 Appropriately fill PFCandidate::dnn_gamma_ data member in the constructors and copy operator #36096
• @Dr15Jones Allow fast merge for file created using concurrent LuminosityBlocks #36026
• @fwyzard Simplify the ReusableObjectHolder #36068
• @francescobrivio Migrate some ALCA code to new PoolDBOutputService methods #36090

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-701241/20517/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-701241/20517/git-merge-result

Comparison Summary

@slava77 comparisons for the following workflows were not done due to missing matrix map:

• /data/cmsbld/jenkins/workspace/compare-root-files-short-matrix/data/PR-701241/1020.0_AlCaLumiPixels2016H+AlCaLumiPixels2016H+TIER0EXPLP+ALCAEXPLP+ALCAHARVLP+TIER0PROMPTLP

Summary:

• No significant changes to the logs found
• Reco comparison results: 8 differences found in the comparisons
• DQMHistoTests: Total files compared: 42
• DQMHistoTests: Total histograms compared: 2902040
• DQMHistoTests: Total failures: 11
• DQMHistoTests: Total nulls: 1
• DQMHistoTests: Total successes: 2902006
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: -0.004 KiB( 41 files compared)
• DQMHistoSizes: changed ( 312.0 ): -0.004 KiB MessageLogger/Warnings
• Checked 177 log files, 37 edm output root files, 42 DQM output files
• TriggerResults: no differences found

[tvami]

wf 1020 had no problems this time
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-701241/20517/runTheMatrixINPUT-results/1020.0_AlCaLumiPixels2016H+AlCaLumiPixels2016H+TIER0EXPLP+ALCAEXPLP+ALCAHARVLP+TIER0PROMPTLP/

[tvami]

+alca

[cmsbuild]

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

[tvami]

@qliphy I'm not sure if this can still reach the 2300 IB

[qliphy]

+1

@qliphy I'm not sure if this can still reach the 2300 IB
Thanks!