Prints "Could not find any config for exited child..." upon upgrade

[contribsys]

Prints "Could not find any config for exited child..." upon upgrade

[gdb]

(As a note, looks like this can also be triggered when can_safely_reload? returns false.)

[ebroder]

Hmm. So my recollection is that I wrote the code in #39 because our einhorn worker monitoring would fire because einhorn would keep the state passer in its state variables indefinitely, thus looking like a very old worker process.

But in any case, it feels like the code in #40 should safely handle all possible cases of this race - it's basically a more generalized and better version of the same issues as #39. Maybe we can revert 673f5a8 now that #40 is in place? We should make sure we think through the state upgrade path before we do, though.

I'm not totally confident all of that is right, though. But intuitively, it seems like a better fix than making the communication with the state passer more complicated.

[gdb]

Ah yeah, I like that much better. I'll poke at it.

• gdb

On Sun, Apr 12, 2015 at 11:41 PM, Evan Broder [email protected]
wrote:

Hmm. So my recollection is that I wrote the code in #39
#39 because our einhorn worker
monitoring would fire because einhorn would keep the state passer in its
state variables indefinitely, thus looking like a very old worker process.

But in any case, it feels like the code in #40
#40 should safely handle all
possible cases of this race - it's basically a more generalized and better
version of the same issues as #39
#39. Maybe we can revert 673f5a8
673f5a8
now that #40 #40 is in place? We
should make sure we think through the state upgrade path before we do,
though.

I'm not totally confident all of that is right, though. But intuitively,
it seems like a better fix than making the communication with the state
passer more complicated.

—
Reply to this email directly or view it on GitHub
#46 (comment).

[adambrakhane]

Did you guys ever figure out how to resolve this?