I'll have a PR shortly which I'd like to get into production ASAP. This changes some append/skiplist settings to include a few more variants of windows functions, mainly for better hang processing.
I am confused. RealMsgWaitFor.* belongs to irrelevantSignatureRegEx and prefixSignatureRegEx (under the .*WaitFor.* pattern).
Yes, but we process irrelevant first and then prefix, AIUI, so this should still be correct.
Commit pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/63861cb298762a3f4edd462eb3925c49283f339f Bug 781317 - Put all "WaitFor" functions and "BaseGetNamedObjectDirectory" into the append list, and add a few additional windows functions to the skiplist.
Lonnen, this has been pushed, right? In that case, I guess we can mark it fixed. (I hope this is on master and the branch used for prod right now.)
(In reply to Robert Kaiser (:firstname.lastname@example.org) from comment #5) > (I hope this is on master and the branch used for prod right now.) It is on prod according to crash stats.
(In reply to Scoobidiver from comment #6) > (In reply to Robert Kaiser (:email@example.com) from comment #5) > > (I hope this is on master and the branch used for prod right now.) > It is on prod according to crash stats. I knew it had been pushed to production. There was a reason why I didn't just mark fuixed but asked instead. Please don't just set them as fixed when questions asked are open, like if this was pushed to both master and production, as I asked above. I'll leave it resolved though, as there is *probably* no harm done.
Through some kind of gitfail the thing which was pushed only accounted for the WaitFor parts of this patch: I've put up https://github.com/mozilla/socorro/pull/784 for the BaseGetNamedObjectDirectory change.
Commit pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/47573de947b0af52feea1f9097600f0129f7ab8a Merge pull request #784 from bsmedberg/BaseGetNamedObjectDirectory Bug 781317 - Add BaseGetNamedObjectDirectory to the append list for better signatures from bug 763896
This is on master and stage, but has not been pushed to prod yet since it's not needed tonight and we're anticipating more changes to follow shortly. @:bsmedberg, in the future it may be best to avoid putting other bug numbers in the summary. Bug 763896 also got a comment about a Socorro master push.