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
Changes from scram build code-format-all #26807
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26807/9816
|
please test |
The tests are being triggered in jenkins. |
A new Pull Request was created by @wddgit (W. David Dagenhart) for master. It involves the following packages: HeterogeneousCore/CUDAServices @cmsbuild, @smuzaffar, @Dr15Jones can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
-1 Tested at: 8551c2e You can see the results of the tests here: I found follow errors while testing this PR Failed tests: UnitTests RelVals
I found errors in the following unit tests: ---> test runtestPhysicsToolsPatAlgos had ERRORS
When I ran the RelVals I found an error in the following workflows: runTheMatrix-results/136.731_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2/step2_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2.log4.22 step2 runTheMatrix-results/4.22_RunCosmics2011A+RunCosmics2011A+RECOCOSD+ALCACOSD+SKIMCOSD+HARVESTDC/step2_RunCosmics2011A+RunCosmics2011A+RECOCOSD+ALCACOSD+SKIMCOSD+HARVESTDC.log |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
EOS is giving problems again? |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+core |
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:
Includes only changes from scram build code-format-all (clang format) and nothing else. This PR includes the last 2 Core packages. These are the only Core packages where clang format was not already run and the results already in a merged or pending PR. The one exception is the Skeletons package, where there is in an issue to remind us of the problems that need to be resolved.
PR validation:
The only test I performed was checking that the revised packages compile and build.