-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add DQMOnlineBeamspot streamer files for run 370580 #6
Add DQMOnlineBeamspot streamer files for run 370580 #6
Conversation
A new Pull Request was created by @francescobrivio for branch main. @cmsbuild, @nothingface0, @rvenditti, @syuvivida, @iarspider, @antoniovagnerini, @tjavaid, @smuzaffar, @aandvalenzuela can you please review it and eventually sign? Thanks. |
test parameters:
|
@cmsbuild please test |
-1 Failed Tests: UnitTests The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Unit TestsI found 1 errors in the following unit tests: ---> test testTauEmbeddingProducers had ERRORS Comparison SummarySummary:
|
Failure in TauEmbedding unitTest is unrelated to this PR. |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-81faa6/35537/summary.html Comparison SummarySummary:
|
+1 |
+externals this is needed by cms-sw/cmssw#43110 |
This pull request is fully signed and it will be integrated in one of the next main IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @rappoccio, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Addresses #3
PR Description
streamDQMOnlineBeamspot
used in the unitTest of thebeamhlt
clientTestDQMOnlineClient-beamhlt_dqm_sourceclient
silently fails in IBs cms-sw/cmssw#43108PR Validation
Successfully ran locally the
beamhlt
unitTest with the new streamer files