-
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
ECAL Phase 2 - Skip new ALCA step for Phase 2 ECAL development WFs #40378
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40378/33496
|
A new Pull Request was created by @thomreis (Thomas Reis) for master. It involves the following packages:
@bbilin, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen, @sunilUIET can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
type ecal |
test parameters:
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1c591a/29727/summary.html Comparison SummaryThere are some workflows for which there are errors in the baseline: Summary:
|
+Upgrade Technical PR to disable ALCA step for special ECAL workflow. |
+pdmv |
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. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
After the ALCA step was introduced for Phase 2 WFs with #39858 the
ecalDevel
workflows with suffix.61
and.612
failed during the new step5 since the track collections were not produced in the reco step. Since these WFs are intended for ECAL reconstruction developments, the reconstruction of other subdetector collections is not complete.This PR skips the ALCA step for the ecalDevel WFs.
PR validation:
WF 20034.61 now runs to completion with four steps only.