-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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 customisePixelLocalReconstruction() customize function #35722
Fix customisePixelLocalReconstruction() customize function #35722
Conversation
…v1 in HLTSchedule
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35722/26047
|
A new Pull Request was created by @makortel (Matti Kortelainen) for master. It involves the following packages:
@cmsbuild, @missirol, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
@cmsbuild, please test |
Tests are stuck? |
@cmsbuild, please abort |
@cmsbuild, please test |
@smuzaffar Any idea why the tests are stuck in "Waiting for tests to start"? |
@cmsbuild ping |
The Jenkins build seems to be sussecfully ran |
the gpu relval comparison job failed as the baseline failed to run any of the gpus relval ( https://cmssdt.cern.ch/SDT/jenkins-artifacts/ib-baseline-tests/CMSSW_12_1_X_2021-10-19-1100/slc7_amd64_gcc900/-GenuineIntel/matrixgpu-results/runall-report-step123-.log ). I will fix the bot to not just fail but to report back that it is not able to run gpu comparison |
-1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4f0f34/19734/summary.html Comparison SummarySummary:
|
Just to clarify (to myself) that with this PR all the tested GPU workflows succeeded
|
correct @makortel |
+hlt |
This pull request is fully signed and it will be integrated in one of the next master IBs (but tests are reportedly failing). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
merge |
PR description:
This PR adds explicit manipulation of
HLTSchedule
when the Paths whose customization was added in #35566 are replaced inProcessFragment
. Fixes #35624.PR validation:
Tested on CPU-only machine that workflow 11634.502 works.