-
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
[Backport] Add TrackerAdditionalParametersPerDet_cfi to prevent exception in Tracker test script #35200
Conversation
A new Pull Request was created by @cvuosalo (Carl Vuosalo) for CMSSW_12_0_X. It involves the following packages:
@civanch, @Dr15Jones, @makortel, @cvuosalo, @ianna, @mdhildreth, @cmsbuild can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
type bugfix |
backport of #35183 |
@cmsbuild please test |
please abort |
Tests seem to be stuck |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2654fd/18467/summary.html Comparison SummarySummary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_1_X is complete. 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 |
Backport of #35183 from 12_1. Adding a line to a config to prevent an exception occurring. See #35183 for more details.
The config in this PR is used to help validate geometry DB payloads, so it will be needed when the new DDD 12_0 geometry DB payloads are made.
The changed config in this PR was run for both Run 1 and Run 3 to check that it runs successfully with no exception.