Properly ignore threshold when requested

[Dr15Jones]

The default value of 0 for threshold will now properly stop the threshold checking.

[Dr15Jones]

@slava77 please double check the logic.

[cmsbuild]

A new Pull Request was created by @Dr15Jones (Chris Jones) for master.

It involves the following packages:

FWCore/Services

@cmsbuild, @smuzaffar, @Dr15Jones, @davidlange6 can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @wddgit this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

[Dr15Jones]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/21106/console Started: 2017/07/03 22:55
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/21107/console Started: 2017/07/03 23:14

[slava77]

On 7/3/17 1:53 PM, Chris Jones wrote: @slava77 <https://github.com/slava77> please double check the logic.

looks fine, visually. Thank you for the quick fix.

…

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#19531 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEdcbkJhe7121WT_9f7p69HX2mPXya1Zks5sKVTPgaJpZM4OMrQ->.

[Dr15Jones]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar

[cmsbuild]

+1
Tested at: 3b53b09
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19531/21107/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
d5de801
4cac99e
236f7c2
b0da358
34923e5
514d35a
aee2fec
526d318
ccc6917
b1580cd
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19531/21107/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19531/21107/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• You potentially added 12 lines to the logs
• Reco comparison results: 1773 differences found in the comparisons
• DQMHistoTests: Total files compared: 22
• DQMHistoTests: Total histograms compared: 1756015
• DQMHistoTests: Total failures: 46458
• DQMHistoTests: Total nulls: 35
• DQMHistoTests: Total successes: 1709356
• DQMHistoTests: Total skipped: 166
• DQMHistoTests: Total Missing objects: 0
• Checked 90 log files, 14 edm output root files, 22 DQM output files

[davidlange6]

+1