Update Run3 GEM to use the eMap from the DB

[watson-ij]

PR description:

Preemptive backport of #29194 into 11_0_X, fix for gem in run3 crashing due to not reading the emap from the db.

PR validation:

[cmsbuild]

A new Pull Request was created by @watson-ij (Ian J. Watson) for CMSSW_11_0_X.

It involves the following packages:

EventFilter/GEMRawToDigi

@perrotta, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks.
@jshlee, @Martin-Grunewald this is something you requested to watch as well.
@davidlange6, @silviodonato, @fabiocos you are the release manager for this.

cms-bot commands are listed here

• Backported from Update Run3 GEM to use the eMap from the DB #29194

[silviodonato]

@watson-ij why do you want to backport this fix to CMSSW_11_0_X ?

[watson-ij]

Hi Silvio, Slava asked me to also make a backport request at the same time as #29194, as this should fix issues with CMSSW crashing while GEM is running in MGWR#1 data.

[cmsbuild]

Pull request #29195 was updated. @perrotta, @silviodonato, @cmsbuild, @franzoni, @slava77, @fabiocos, @davidlange6 can you please check and sign again.

[silviodonato]

Hi Ian, ok, indeed the MWGR#1 will be in May and then they will continue to use 11_0_X

[slava77]

@christopheralanwest @tocheng
please clarify if we already have a 11_0 PR with GEM eMap in MC for Run-3 (a backport of #29210 )
Thank you.

[cmsbuild]

The tests are being triggered in jenkins.
Tested with other pull request(s) #29277
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/5333/console Started: 2020/03/23 20:54

[cmsbuild]

+1
Tested at: 8fc8b69
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6c68e2/5333/summary.html
CMSSW: CMSSW_11_0_X_2020-03-23-1100
SCRAM_ARCH: slc7_amd64_gcc820

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 34
• DQMHistoTests: Total histograms compared: 2793840
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2793498
• DQMHistoTests: Total skipped: 341
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 33 files compared)
• Checked 147 log files, 16 edm output root files, 34 DQM output files

[slava77]

backport of #29194

[slava77]

+1

for #29195 8fc8b69

• the backport looks correct; the code changes are the same as in the master version Update Run3 GEM to use the eMap from the DB #29194
• jenkins tests pass (the run3 MC wf is exercised and shows no difference as expected)

[silviodonato]

+operations

[cmsbuild]

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

[silviodonato]

+1