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
ROOT 6-22 - get latest commits #6841
Conversation
we are one commit behind
A new Pull Request was created by @mrodozov (Mircho Rodozov) for branch IB/CMSSW_12_0_X/master. @cmsbuild, @smuzaffar, @mrodozov can you please review it and eventually sign? Thanks. |
please test |
-1 Failed Tests: UnitTests RelVals RelVals-INPUT AddOn Unit TestsI found errors in the following unit tests: ---> test runtestTqafTopEventProducers had ERRORS ---> test runtestTqafTopHitFit had ERRORS ---> test runtestTqafTopKinFitter had ERRORS ---> test runtestTqafTopEventSelection had ERRORS and more ... RelVals
RelVals-INPUT
Expand to see more relval errors ...AddOn Tests
|
I was running this to figure out why this tests are failing:
and the output is showing failure in CMS code:
|
Here is stack trace from the pat1 test
I have no clue on the top of my head. @Dr15Jones, any ideas? What does the added ROOT commit do? |
nothing much, I suspect this was there all along, but I got carried out the same tests fail, then I update it again before testing, test and only one unit test was failing. Bottom line - it's not due to this one commit. Also it's not one commit - it's five, but I don't think it's root - |
Maybe there is a 'runaway' exception? If this is repeatable, try running under gdb with |
I have three out of three failures + the tests. this is the catch throw result:
|
checking if ROOT master is the same |
removing cms-sw/root@8a9c9e5 commit as in 6-24 until it's fixed local run has passed
Pull request #6841 was updated. |
please test |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-516f93/14734/summary.html Comparison SummarySummary:
|
+externals |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_12_0_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-516f93/14762/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
we are one commit behind