Monitoring JSON changes in DAQ

[smorovic]

Changes:

• migrating to JSON based format for module legend used by f3mon
• adding HLTErrorEvents field which will used to distinguish between losses caused by crashes in CMSSW and other reasons, for example hardware failures.

[cmsbuild]

A new Pull Request was created by @smorovic (Srecko Morovic) for CMSSW_7_5_X.

Monitoring JSON changes in DAQ

It involves the following packages:

DQMServices/Components
EventFilter/Utilities
HLTrigger/JSONMonitoring

@perrotta, @cmsbuild, @cvuosalo, @fwyzard, @emeschi, @Martin-Grunewald, @deguio, @slava77, @mommsen, @vanbesien, @danduggan can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @barvic this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[mommsen]

+1

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/9257/console

[fwyzard]

+1

[fwyzard]

HLT change looks reasonable

[cmsbuild]

+1
Tested at: b9ccbe9
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12136/9257/summary.html

[cmsbuild]

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

[deguio]

+1

[cvuosalo]

+1

For #12136 b9ccbe9

Changes in the monitoring JSON for the DAQ. #12137 and #12138 are the 76X and 80X versions of this PR.

The code changes are OK for 75X. The 80X version of this PR will require some small code changes for maintainability. Jenkins tests against baseline CMSSW_7_5_X_2015-10-27-2300 show no significant differences.

[cmsbuild]

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

[davidlange6]

+1