ES consume migration of LumiAlCaRecoProducer#36095
Conversation
|
@cmsbuild , please test |
|
+code-checks
|
|
A new Pull Request was created by @tvami (Tamas Vami) for master. It involves the following packages:
@malbouis, @tvami, @yuanchao, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5ef42c/20475/summary.html Comparison SummarySummary:
|
|
+alca
|
|
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) |
|
+1 |
|
@tvami There is a IB issue appearing after merging this PR, which seems to be related: eos/cms/store/user/cmsbuild/store/data/Run2016H/AlCaLumiPixels1/RAW/v1/000/283/383/00000/BAF61397-DA95-E611-B879-02163E0145CC.root Would you please have a check? |
PR description:
ES consume migration of LumiAlCaRecoProducer
PR validation:
I tried to run the unit tests but got file not found error :(
Maybe it will work for Jenkins
Anyway the code compiles
if this PR is a backport please specify the original PR and why you need to backport that PR:
N/A