Phase2-hgx86 Fix the bug for overwriting (backport from 9_2_X #19198)

[bsunanda]

Use correct # for maximum wafers

[bsunanda]

@cmsbuild Please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/20550/console Started: 2017/06/13 18:44

[cmsbuild]

A new Pull Request was created by @bsunanda for CMSSW_9_1_X.

It involves the following packages:

RecoLocalCalo/HGCalRecAlgos

@perrotta, @cmsbuild, @slava77, @kpedro88, @davidlange6 can you please review it and eventually sign? Thanks.
@vandreev11, @sethzenz, @kpedro88, @argiro, @cseez, @pfs, @lgray this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

[kpedro88]

+1

[cmsbuild]

+1
Tested at: caae427
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19209/20550/summary.html

[cmsbuild]

Comparison job queued.

[kpedro88]

Backport of #19198

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 1681 differences found in the comparisons
• DQMHistoTests: Total files compared: 24
• DQMHistoTests: Total histograms compared: 1834259
• DQMHistoTests: Total failures: 8530
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 1825549
• DQMHistoTests: Total skipped: 180
• DQMHistoTests: Total Missing objects: 0
• Checked 98 log files, 14 edm output root files, 24 DQM output files

[cmsbuild]

Pull request #19209 was updated. @perrotta, @cmsbuild, @slava77, @kpedro88, @davidlange6 can you please check and sign again.

[bsunanda]

@cmsbuild Please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/20568/console Started: 2017/06/14 16:44

[cmsbuild]

+1
Tested at: 9b8b88a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19209/20568/summary.html

[cmsbuild]

Comparison job queued.

[kpedro88]

+1

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 1695 differences found in the comparisons
• DQMHistoTests: Total files compared: 24
• DQMHistoTests: Total histograms compared: 1834225
• DQMHistoTests: Total failures: 27227
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 1806818
• DQMHistoTests: Total skipped: 180
• DQMHistoTests: Total Missing objects: 0
• Checked 98 log files, 14 edm output root files, 24 DQM output files

[slava77]

+1

for #19209 9b8b88a

• the same change (ignoring whitespaces) as in 92X version Phase2-hgx85 Correct for overwriting (as reported in PR #19179) #19198 84a5122
• jenkins tests pass and comparisons show no differences

[cmsbuild]

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

[davidlange6]

+1