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
Migrate SiPixelGenErrorDBObjectESProducer and SiPixelTemplateDBObjectESProducer to EventSetup consumes #28142
Migrate SiPixelGenErrorDBObjectESProducer and SiPixelTemplateDBObjectESProducer to EventSetup consumes #28142
Conversation
6d32e7c
to
b6b48ea
Compare
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28142/12188
|
A new Pull Request was created by @makortel (Matti Kortelainen) for master. It involves the following packages: CalibTracker/SiPixelESProducers @christopheralanwest, @tocheng, @cmsbuild, @franzoni, @tlampen, @pohsun can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild, please test |
The tests are being triggered in jenkins. |
I couldn't avoid noticing that the two files are almost exact copies differing only with the product and record types. The copy paste could be easily avoided with templates. If the DPG likes, I can do that. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@mmusich I believe the comment #28142 (comment) by @makortel is for you @christopheralanwest @pohsun could you please check the PR? |
+1 |
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. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR is an updated version of #27043 to migrate SiPixelGenErrorDBObjectESProducer and SiPixelTemplateDBObjectESProducer to EventSetup consumes (as part of #26748) using the recently-introduced "mayConsume" functionality.
PR validation:
Code compiles, limited matrix runs.
@Dr15Jones