-
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
[12_6_X] RPCOccupancyTest crash protection for 0 LS #39559
Conversation
@pmandrik, CMSSW_12_6_X branch is closed for direct updates. cms-bot is going to move this PR to master branch. |
please test |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39559/32349
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39559/32366
|
A new Pull Request was created by @pmandrik (Petr Mandrik) for master. It involves the following packages:
@emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @syuvivida, @pmandrik, @micsucmed, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a995b0/27960/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
@cms-sw/dqm-l2 signing a backport PR implies, amongst other thimgs, that you verified that the master version of the PR run without errors and behaved as expected. You already signed the backports of this PR: either sign also this one, or remove the signatures in the two backport ones while waiting for evaluating this one in the master |
+1 |
1 similar comment
+1 |
ping bot |
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 be automatically merged. |
PR description:
See #39558