New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HLT DQM: use global, concurrent monitor elements #21738
Conversation
The code-checks are being triggered in jenkins. |
@cmsbuild, please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-21738/2632 |
The tests are being triggered in jenkins. |
A new Pull Request was created by @fwyzard (Andrea Bocci) for master. It involves the following packages: DQM/HLTEvF @cmsbuild, @vazzolini, @kmaeshima, @dmitrijus, @Martin-Grunewald, @silviodonato, @jfernan2, @fwyzard, @vanbesien can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
bafe81d
to
b7abf13
Compare
+1 |
The code-checks are being triggered in jenkins. |
@cmsbuild, please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-21738/2634 |
-1 Tested at: 6b5bbd8 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: RelVals
When I ran the RelVals I found an error in the following worklfows: DAS Error136.7611 step1 DAS Error136.788 step1 DAS Error140.53 step1 DAS Error |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
+1 |
Update the Services and DQMEDAnalyzers in HLTrigger/Timer to make use the new concurrent-safe booking and monitor elements.
This PR builds on top of, and includes, #21652 .