search

cms-sw / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
1.08k stars 4.32k forks source link

`test_dumpRecoGeometry` test should be split into multiple scripts #46053

Closed makortel closed 1 month ago

makortel commented 1 month ago

The test_dumpRecoGeometry test script runs many cmsRun jobs and can time out in PR tests (see e.g. https://github.com/cms-sw/cmssw/pull/45999#issuecomment-2353723360). It would be good to split the script so that (at least some) these jobs could be run in parallel.

cmsbuild commented 1 month ago

cms-bot internal usage

cmsbuild commented 1 month ago

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

makortel commented 1 month ago

assign Fireworks/Geometry

cmsbuild commented 1 month ago

New categories assigned: visualization

@Dr15Jones,@alja,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks

alja commented 1 month ago

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.