mrb_load_file panics due to an expired syscall about go-mruby HOT 7 OPEN

this ruby code in a file that is loaded with LoadFile will break it:

sleep 600

from go-mruby.

@erikh Do you have exact code & steps we can use to reproduce this?

I pulled your load file patch and tried with implementations of sleep using both golang and mruby-sleep mgem but could not reproduce. Is there any chance that other goroutines could be sharing the mruby interpreter?

I did find this which may be relevant.

from go-mruby.

let me see if I can find the test case I used, but IIRC, I converted
github.com/erikh/box to use load_file instead of load_string (which is what
already exists). this is what caused the behavior.

box's tests build for about 20 minutes now, so it tripped in there
somewhere.

I'll try to investigate soon. If you are feeling confident in the patch,
I'm not going to object to it being merged if my case was an edge one.

On Fri, Nov 11, 2016 at 5:16 AM, Mike Simons [email protected]
wrote:

@erikh https://github.com/erikh Do you have exact code & steps we can
use to reproduce this?

I pulled your load file patch and tried with implementations of sleep
using both golang and mruby-sleep mgem but could not reproduce. Is there
any chance that other goroutines could be sharing the mruby interpreter?

I did find this
golang/go#15639 (comment) which
may be relevant.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#40 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABJ6w8cJFKjZurvW7HPEAKSB6FqmP2iks5q9GqYgaJpZM4KjGyY
.

from go-mruby.

My gut feel is that this is an issue albeit one that is triggered with a very specific, possibly compound set of steps. I haven't been able to replicate it though, even with your patch applied to box (and modified to use LoadFile).

This is the kind of tricky issue that is going to crop up and put folks off using it for stability concerns so I think it is important to understand the reasons behind it so we can either mitigate or inform potential users.

from go-mruby.

Sorry I was a little short, basically, what I did was make a long running
open mrb_load_file (running ruby program), at which point I got this error.
maybe a platform issue?

On Thu, Nov 17, 2016 at 4:44 AM, Mike Simons [email protected]
wrote:

My gut feel is that this is an issue albeit one that is triggered with
a very specific, possibly compound set of steps. I haven't been able to
replicate it though, even with your patch applied to box (and modified to
use LoadFile).

This is the kind of tricky issue that is going to crop up and put folks
off using it for stability concerns so I think it is important to
understand the reasons behind it so we can either mitigate or inform
potential users.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#40 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABJ67hW3kRkvntr5BNJIwSx_SjDCQvDks5q_EwRgaJpZM4KjGyY
.

from go-mruby.

for now, the code is reverted, I'm hoping I can resolve this soon.

On Thu, Nov 17, 2016 at 8:28 AM, Erik Hollensbe [email protected] wrote:

Sorry I was a little short, basically, what I did was make a long running
open mrb_load_file (running ruby program), at which point I got this error.
maybe a platform issue?

On Thu, Nov 17, 2016 at 4:44 AM, Mike Simons [email protected]
wrote:

My gut feel is that this is an issue albeit one that is triggered with
a very specific, possibly compound set of steps. I haven't been able to
replicate it though, even with your patch applied to box (and modified to
use LoadFile).

This is the kind of tricky issue that is going to crop up and put folks
off using it for stability concerns so I think it is important to
understand the reasons behind it so we can either mitigate or inform
potential users.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#40 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABJ67hW3kRkvntr5BNJIwSx_SjDCQvDks5q_EwRgaJpZM4KjGyY
.

from go-mruby.

@mikesimons ar eyou still using this project? do you still see this on recent versions? I have run a lot of builds through box now -- almost 10k/day now -- and I never see it.

from go-mruby.