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

fix (rare) crash in DYT muon reco #6428

Merged
merged 1 commit into from Nov 16, 2014

Conversation

slava77
Copy link
Contributor

@slava77 slava77 commented Nov 15, 2014

Fixes the crash reported in
https://hypernews.cern.ch/HyperNews/CMS/get/relval/3239.html

no changes expected in regular tests and none are observed (the condition for the crash is fairly rare)

@jhgoh please check the changes with the main developers.
My fix is just technical to avoid the crash, but it would be good to check in more details why
this condition happens.
The issue can be reproduced from http://cms-project-relval.web.cern.ch/cms-project-relval/failures/CMSSW_7_3_0_pre2/12Nov2014_1/JpsiMuMu_Pt-15_Job1.tar.gz
(dump the PSet.py there to see the expanded config details)

@cmsbuild cmsbuild added this to the Next CMSSW_7_3_X milestone Nov 15, 2014
@slava77
Copy link
Contributor Author

slava77 commented Nov 15, 2014

+1

for #6428 5dfaa9e

tested in CMSSW_7_3_0_pre2 (test area crash-dyt).
No changes are observed in the short matrix tests.
The crash in the pre2 relval job is not happening anymore.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @slava77 (Slava Krutelyov) for CMSSW_7_3_X.

fix (rare) crash in DYT muon reco

It involves the following packages:

RecoMuon/GlobalTrackingTools

@cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks.
@battibass, @abbiendi, @bellan, @trocino, @bachtis, @rociovilar this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes or unless it breaks tests.

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes (tests are also fine). This pull request will be automatically merged.

cmsbuild added a commit that referenced this pull request Nov 16, 2014
@cmsbuild cmsbuild merged commit ef6ec98 into cms-sw:CMSSW_7_3_X Nov 16, 2014
@cmsbuild
Copy link
Contributor

@Dr15Jones
Copy link
Contributor

@slava77 Thanks for the fix. I saw the exact same crash in the threaded build today and was worried it was a threading issue. Glad to see it is not :).

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

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

Successfully merging this pull request may close these issues.

None yet

4 participants