Skip to content

[DEBUG] Investigate leak_check test failures#182

Closed
aaronj0 wants to merge 2 commits intocompiler-research:masterfrom
aaronj0:leakchecincr
Closed

[DEBUG] Investigate leak_check test failures#182
aaronj0 wants to merge 2 commits intocompiler-research:masterfrom
aaronj0:leakchecincr

Conversation

@aaronj0
Copy link
Collaborator

@aaronj0 aaronj0 commented Feb 10, 2026

to investigate the sudden leakcheck failures on our CI (which are not reproducible with py3.12)

@aaronj0 aaronj0 changed the title [test] Increase number of runs for leakcheck tests [DEBUG] Investigate leak_check test failures Feb 10, 2026
@aaronj0 aaronj0 force-pushed the leakchecincr branch 3 times, most recently from a08c749 to 3e0f3e4 Compare February 10, 2026 13:24
@aaronj0
Copy link
Collaborator Author

aaronj0 commented Feb 10, 2026

Okay, this behaviour comes from compiler-research/CppInterOp@76d0600 as confirmed by the CI.

Not necessarily a memory leak, delta rss checks have always been unreliable:

      # The use of arena's, free-lists, etc. means that checking rss remains
      # unreliable, unless looking for consistent jumps, so the leak check will
      # be run M times and only considered failed if it "leaks" every time. In
      # actual practice, the number of fails is 0, 1, or M. Note that the total
      # number of gc objects tracked is always required to remain the same.

Debugging further...

On a separate note, I discovered that currently, the leakcheck tests are only run on the cppyy repo. I will open PR's rectifying this situation. I propose using a requirements.txt file and installing psutil everywhere.

Edit: Still unsure why this is not reproduceable

@aaronj0 aaronj0 force-pushed the leakchecincr branch 4 times, most recently from 3649b6d to 6a385a6 Compare March 10, 2026 11:05
@aaronj0
Copy link
Collaborator Author

aaronj0 commented Mar 11, 2026

Closing as this debug activity is complete

@aaronj0 aaronj0 closed this Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant