-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
test_dumpRecoGeometry
test should be split into multiple scripts
#46053
Comments
cms-bot internal usage |
A new Issue was created by @makortel. @Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
assign Fireworks/Geometry |
New categories assigned: visualization @Dr15Jones,@alja,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks |
A simple fix would be to split the test for each tag in https://github.com/cms-sw/cmssw/blob/master/Fireworks/Geometry/test/test_dumpRecoGeometry.sh#L11 in a new script. Let me know if there is any more sophisticated solution. |
The
test_dumpRecoGeometry
test script runs manycmsRun
jobs and can time out in PR tests (see e.g. #45999 (comment)). It would be good to split the script so that (at least some) these jobs could be run in parallel.The text was updated successfully, but these errors were encountered: