Open Bug 2044025 Opened 2 days ago Updated 1 day ago

Import script failing to auto ignore gkrust leaks

Categories

(Testing :: web-platform-tests, defect)

defect

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: mccr8, Assigned: mccr8)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

As seen in bug 2034815 and bug 2044021, two directories of WPTs are experiencing frequent leaks like this:
INFO - TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at gkrust.ebd94ecb2afbeb2-cgu.0, gkrust.ebd94ecb2afbeb2-cgu.0, gkrust.ebd94ecb2afbeb2-cgu.0, gkrust.ebd94ecb2afbeb2-cgu.0

The problem is that the sequence of weird characters keeps changing causing large spikes in leaks until the import script runs again.

James, I don't remember exactly how the lsan-allowed matching works. If gkrust gets added, will it match frames like gkrust.ebd94ecb2afbeb2-cgu.0? These doesn't seem like a very frequent problem so I'm glad to go through and clean up the 2 directories for this specific issue but I can't tell offhand if it'll work or not. Thanks.

Flags: needinfo?(james)

Looks like mozleak interprets the allow list as a set of regexp: https://searchfox.org/firefox-main/source/testing/mozbase/mozleak/mozleak/lsan.py#94-96. The sync bot won't ever create anything other than static strings, but I think something like gkrust\..*-cgu\.0 would work.

Flags: needinfo?(james)
Assignee: nobody → continuation
Status: NEW → ASSIGNED

(In reply to James Graham [:jgraham] from comment #2)

Looks like mozleak interprets the allow list as a set of regexp: https://searchfox.org/firefox-main/source/testing/mozbase/mozleak/mozleak/lsan.py#94-96. The sync bot won't ever create anything other than static strings, but I think something like gkrust\..*-cgu\.0 would work.

I think the re.escape will prevent you from passing in an actual regexp. But it looks like it uses self.allowedRegexp.match(frame) eventually, so I think it'll match the prefix just fine, fortunately.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: