CTPPS: bug fix #2 in run ranges for FEDs channels mapping (9_1_X)

[forthommel]

Forward-port of #18732 onto 9_1_X.

Proper handling of the begin/end conditions through the usage of a edm::contains (instead of a home-brewed check).

runTheMatrix.py -l limited on 0dbffaa yielded 17 17 15 13 6 0 0 0 tests passed, 1 0 1 0 0 0 0 0 failedwith the usual step0 DAS error on 4.22, and a return code 35072 for step2 of 1001.0

[cmsbuild]

A new Pull Request was created by @forthommel (Laurent Forthomme) for CMSSW_9_1_X.

It involves the following packages:

CondFormats/CTPPSReadoutObjects

@ghellwig, @arunhep, @cerminar, @cmsbuild, @franzoni, @ggovi, @mmusich, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @tocheng this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/19856/console Started: 2017/05/16 00:31

[cmsbuild]

+1
Tested at: 0dbffaa
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18733/19856/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• You potentially added 63 lines to the logs
• Reco comparison results: 1634 differences found in the comparisons
• DQMHistoTests: Total files compared: 24
• DQMHistoTests: Total histograms compared: 1831084
• DQMHistoTests: Total failures: 25142
• DQMHistoTests: Total nulls: 1
• DQMHistoTests: Total successes: 1805761
• DQMHistoTests: Total skipped: 180
• DQMHistoTests: Total Missing objects: 0
• Checked 98 log files, 14 edm output root files, 24 DQM output files

[ggovi]

+1

[davidlange6]

Closing PR. CMSSW_9_1_X is for muon TDR upgrade changes only (CMSSW_9_2_X for data taking)