-
Notifications
You must be signed in to change notification settings - Fork 246
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
Set MALLOC_CONF=junk:true for PR tests #2228
base: master
Are you sure you want to change the base?
Conversation
as proposed in cms-sw/cmssw#44962
A new Pull Request was created by @smuzaffar for branch master. @smuzaffar, @iarspider, @aandvalenzuela, @cmsbuild can you please review it and eventually sign? Thanks. |
cms-bot internal usage |
test parameters:
|
please test lets build full cmssw to run all unit tests under |
-1 Failed Tests: UnitTests RelVals AddOn Unit TestsI found 2 errors in the following unit tests: ---> test testSiStripHitResolution had ERRORS ---> test testSiStripHitEfficiency had ERRORS RelVals
AddOn Tests
Expand to see more addon errors ... |
@cmsbuild please test
|
-1 Failed Tests: UnitTests Unit TestsI found 2 errors in the following unit tests: ---> test testSiStripHitResolution had ERRORS ---> test testSiStripHitEfficiency had ERRORS Comparison SummarySummary:
|
@cmsbuild, please test |
-1 Failed Tests: UnitTests Unit TestsI found 2 errors in the following unit tests: ---> test testSiStripHitResolution had ERRORS ---> test testSiStripHitEfficiency had ERRORS Comparison SummarySummary:
|
The
The
|
just for the record there is a follow up issue for that since some time cms-sw/cmssw#45084 |
Thanks for reminding! |
REMINDER @rappoccio, @mandrenguyen, @antoniovilela, @sextonkennedy: This PR was tested with cms-sw/cmssw#46111, please check if they should be merged together |
hold Do not merge this, this is just for testing |
Pull request has been put on hold by @smuzaffar |
-orp |
03f42b1
to
cf14ee3
Compare
as proposed in cms-sw/cmssw#44962