You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@fzakaria would you be interested to take a look at this?
So far I know that Thread creation, Mutex, Queue cause problems and at the moment I do not have more ideas how to get around this.
The change basically removed creating new threads in the finalizers since the finalizers may be called during shutdown & threads cannot be started.
This bug involves changes post the submission of the PR above; where the segfault is occurring within Queue#push
To me that means either the Queue or the @index have been garbage collected already.
Apologies for the delay - I've been on vacation the last week. I'll see if I can run that PR through our CI systems, we've had quite a high hit-rate on the segfault.
Activity
pitr-ch commentedon Feb 16, 2020
Thanks for the report. Few notes:
Further investigation is needed.
pitr-ch commentedon Feb 17, 2020
@fzakaria would you be interested to take a look at this?
So far I know that Thread creation, Mutex, Queue cause problems and at the moment I do not have more ideas how to get around this.
fzakaria commentedon Feb 17, 2020
Sure thing.
I'll build 2.51 from source & run the tests and see what I can uncover.
fzakaria commentedon Feb 17, 2020
Few notes from looking into it:
Queue#pushTo me that means either the
Queueor the@indexhave been garbage collected already.Update to latest ruby version supported by the agent
Avoid concurrent-ruby 1.1.6 which is triggering MRI segfaults
pitr-ch commentedon Mar 1, 2020
Thanks for looking into this. I got an idea how to deal with it, please have a look #856
pitr-ch commentedon Mar 1, 2020
@DavidS could you try the PR if it works for you?
DavidS commentedon Mar 9, 2020
Apologies for the delay - I've been on vacation the last week. I'll see if I can run that PR through our CI systems, we've had quite a high hit-rate on the segfault.
pitr-ch commentedon Mar 9, 2020
@DavidS No problem, thank you for giving it a try.
Update to latest ruby version supported by the agent
schuylr commentedon May 11, 2020
Just want to confirm I'm also seeing this on Ruby 2.5.3 with Version 1.1.6 of the gem.