-
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
fix process name for TIER0RAWSIPIXELCAL
step
#44925
Conversation
type bug-fix |
urgent
|
cms-bot internal usage |
test parameters:
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44925/40200
|
A new Pull Request was created by @mmusich for master. It involves the following packages:
@sunilUIET, @miquork, @subirsarkar, @AdrianoDee, @cmsbuild, @srimanob can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1ae36a/39294/summary.html Comparison SummarySummary:
|
+pdmv |
+1 |
merge
|
hold
|
Pull request has been put on hold by @mmusich |
come on |
@mmusich |
let's make another PR on top... |
... |
Really sorry: I overlooked :( |
28205aa will fix that.
@antoniovilela after this PR, the workflow in question is not failing anymore but not doing anything useful. There's no need to worry. |
Ok, thanks. |
#44932 fixes the issue at #44925 (comment), sorry for the (too) many iterations. |
PR description:
This is a follow-up to #44914 where, when changing the
triggerResultsProcess
parameter ofALCASPLITSIPIXELCAL
to match theprocess
name ofTIER0EXPSIPIXELCAL
, I accidentally put it out of synch with the process name inTIER0RAWSIPIXELCAL
, resulting in IB relval failures, see e.g. this logPR validation:
runTheMatrix.py -l 1040,1040.1 -t 4 -j 8 --ibeos
runs OKIf 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:
Will be backported to CMSSW_14_0_X in #44918