Photon monitoring in 92x release

[gomber]

@fwyzard , @mtosi Here is the PR for the single photon paths corresponding to #18971 in CMSSW_9_2_X release

[cmsbuild]

A new Pull Request was created by @gomber for CMSSW_9_2_X.

It involves the following packages:

DQMOffline/Trigger

@vazzolini, @kmaeshima, @dmitrijus, @cmsbuild, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@battibass, @mtosi, @jhgoh, @calderona, @HuguesBrun, @trocino, @rociovilar this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

[fwyzard]

@cmsbuild, please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/21098/console Started: 2017/07/03 16:21

[cmsbuild]

+1
Tested at: 73991ae
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19520/21098/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
fc51b5b
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19520/21098/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19520/21098/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• You potentially added 23 lines to the logs
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 22
• DQMHistoTests: Total histograms compared: 1813798
• DQMHistoTests: Total failures: 29843
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 1783789
• DQMHistoTests: Total skipped: 166
• DQMHistoTests: Total Missing objects: 0
• Checked 90 log files, 14 edm output root files, 22 DQM output files

[gomber]

@davidlange6 it is the same code as #18971 but in CMSSW_9_2_X, since that is merged, could you please take care of this too?

[dmitrijus]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_9_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_9_3_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar

[fwyzard]

hold

[fwyzard]

@gomber can you check if this PR is included #19961 ?

[cmsbuild]

Pull request has been put on hold by @fwyzard
They need to issue an unhold command to remove the hold state or L1 can unhold it for all

[gomber]

@fwyzard as you know, I am away, so checking mails now.
If my answer is still needed, I just checked this PR is already included in the 19961.

[fwyzard]

Thanks - can you close this PR, when you have a chance ?