-
Notifications
You must be signed in to change notification settings - Fork 24
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
Debug #334 #592
base: main
Are you sure you want to change the base?
Debug #334 #592
Conversation
Klaim and I are iterating here. Found one more segfault and scheduled a few more CI runs. Let's see what we can fish. For now we have this stack:
Highlights:
|
The latest libmambapy patch are causing the conda-libmamba-solver tests to run too slow to fit even in the 6h, maybe something to do with the extra output. Disabling for now because they don't usually segfault anyway. |
I think it's because I enabled assertions in one of the translation unit... I'll remove that for next test binaries. |
I think something in the latest binaries has fixed or hidden the segfault, because we haven't obtained one in like 300 cumulative runs. |
Had to post that message to finally get one 😂 Similar to the other one:
|
That last method is calling two functions:
The good thing is that even if we don't find this segfault, this could be considered optional because we toposort the tree in the conda side anyway. So we could ask for a flag or boolean somewhere to omit this step 🤞 But let's see if we can find the offending code first. |
One more segfault at https://github.com/conda/conda-libmamba-solver/actions/runs/12466302016/job/34797855960?pr=592#step:12:318
|
Description
See #334 (comment)
Checklist - did you ...
news
directory (using the template) for the next release's release notes?