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
[HLTrigger/JSONMonitoring] updated sanity check in L1TriggerJSONMonitoring #28797
[HLTrigger/JSONMonitoring] updated sanity check in L1TriggerJSONMonitoring #28797
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28797/13490
|
A new Pull Request was created by @missirol (Marino Missiroli) for master. It involves the following packages: HLTrigger/JSONMonitoring @cmsbuild, @Martin-Grunewald, @fwyzard can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild, please test |
The tests are being triggered in jenkins. |
@missirol could you prepare a backport to CMSSW_11_0_X ? |
type bugfix |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @silviodonato, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR improves one sanity check done in the plugin
L1TriggerJSONMonitoring
, requiring that theGlobalAlgBlk
object associated to BX=0 be not empty, in order to avoid a crash here.This kind of crash occurred a few times during the last MWGR (see for example this elog); it was caused by the fact that the L1 uGT FED had not been included in the run.
If accepted, this should probably be backported to earlier releases that might be used in future global-runs.
Thanks to @fwyzard for suggesting the fix.
PR validation:
Validated with internal HLT tests (rerunning the HLT menu on the relevant error streams).