Skip to content

Don't unset actor global on root teardown#347

Draft
goodboy wants to merge 2 commits intomainfrom
dun_unset_current_actor
Draft

Don't unset actor global on root teardown#347
goodboy wants to merge 2 commits intomainfrom
dun_unset_current_actor

Conversation

@goodboy
Copy link
Owner

@goodboy goodboy commented Dec 12, 2022

Originally changed to address in an issue discovered (at least i thought) in dev of pikers/piker#420, but now not so sure we ever needed it and it was some other IPC drops-while-streaming issue?

Anyway, sticking the commit here as a reminder in case the issue exhibits again in piker.

Removing this commit seems to fix #345

@goodboy goodboy mentioned this pull request Dec 13, 2022
9 tasks
@goodboy
Copy link
Owner Author

goodboy commented Jan 13, 2023

Yeah pretty sure we don't need this any more, the original piker teardown bug was some other thing likely to do with our service manager..

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.

OSError: [Errno 9] Bad file descriptor in cluster test

1 participant