Skip to content
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

Closed
makortel opened this issue Sep 19, 2024 · 5 comments · Fixed by #46080
Closed

test_dumpRecoGeometry test should be split into multiple scripts #46053

makortel opened this issue Sep 19, 2024 · 5 comments · Fixed by #46080

Comments

@makortel
Copy link
Contributor

The test_dumpRecoGeometry test script runs many cmsRun 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.

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 19, 2024

cms-bot internal usage

@cmsbuild
Copy link
Contributor

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
Copy link
Contributor Author

assign Fireworks/Geometry

@cmsbuild
Copy link
Contributor

New categories assigned: visualization

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

@alja
Copy link
Contributor

alja commented Sep 19, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants