/npipe

WIP: Handle mutex patch from gsamfira. #20

Closed
wants to merge 1 commit into
from

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@cmars @natefinch
@cmars

cmars commented Jun 3, 2016

No description provided.

@cmars

cmars commented Jun 5, 2016

Ran cmd/jujud/agent tests 300 times with this patch: https://gist.github.com/anonymous/11d575817b068f20c994816ed332b5c4
@cmars

cmars commented Jun 6, 2016

Without this patch: https://gist.github.com/anonymous/37c739b824809093a97e84bd3d1d3fb3
@natefinch
Owner

natefinch commented Jun 18, 2016

This is not really landable as-is... even the basic premise gives me the willies... a package-global lock? That's just asking for trouble. I'll try to nail down a better fix for this, even if it's just a timeout and/or a "don't do that".
@natefinch
Owner

natefinch commented Jun 21, 2016

This is fixed by #21

@natefinch natefinch closed this Jun 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment