A fix for the DQMStreamerReader (76x) #11552
Conversation
DQMStreamerReader would incorrectly set matchTriggerSel_ to false if the last component didn't match. (it should be *or*). Minor fixes for esMonitoring script: - added a few more fields to the report document - fixed 'stdlog*' fields - added command line parameter 'path' (previously it was hardcoded into /tmp/dqm_monitoring)
|
A new Pull Request was created by @dmitrijus (Dmitrijus) for CMSSW_7_6_X. A fix for the DQMStreamerReader (76x) It involves the following packages: DQMServices/StreamerIO @cmsbuild, @danduggan, @vanbesien, @deguio can you please review it and eventually sign? Thanks. |
|
please test |
|
+1 |
|
The tests are being triggered in jenkins. |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
cmsbuild
added
dqm-approved
fully-signed
and removed
dqm-pending
pending-signatures
labels
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
|
+1 |
A fix for the DQMStreamerReader (76x)
(discussion: #11551)
DQMStreamerReader would incorrectly set matchTriggerSel_ to false
if the last component didn't match. (it should be or).
Minor fixes for esMonitoring script:
/tmp/dqm_monitoring)