-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[13_2_X] Online DQM fed client: use rawDataRepacker for HCAL FEDIntegrity Task in HI runs (backport of 43120) #43121
Conversation
A new Pull Request was created by @syuvivida for CMSSW_13_2_X. It involves the following packages:
@tjavaid, @cmsbuild, @antoniovagnerini, @syuvivida, @rvenditti, @nothingface0 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
backport of #43120 |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b07829/35452/summary.html Comparison SummarySummary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_13_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_13_3_X is complete. This pull request will now be reviewed by the release team before it's merged. @antoniovilela, @rappoccio, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
As suggested by @mmusich, in the current heavy ion HLT menu,
Dataset_HIOnlineMonitor
still keeps the event productFEDRawDataCollection_rawDataCollector_*_*
. This product includes non-zero suppressed Tracker FEDs and may be the cause why in some HI runs, DQM streamer files are way over the 2GB limit. Therefore, we tested the DQM client codes with a test streamer that does not containFEDRawDataCollection_rawDataCollector_*_*
and see if any DQM client crashes. We found the crash of this clientfed_dqm_sourceclient-live
could be easily fixed with the change of inputTag forhcalFEDIntegrityTask.tagFEDs
.With this update, the client
fed_dqm_sourceclient-live
could run on streamers that either include rawDataCollector or not. The PR does not rely on other PRs.PR validation:
More information could be found in PR 43120
If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
backport of 43120
This PR could be applied on top of current existing online DQM release.