--- FAIL: TestBroker_RemovePeer: race detected during execution of test

[davecheney]

http://paste.ubuntu.com/23985334/

[travisjeffery]

Seems to be fixed after merging a change that split the serf and raft pkgs from the broker's pkg—because they don't access the same peers map anymore. I was able to reproduce it before but can't now. Thanks for making the issue.

• 8145ad3
• 65595df

[davecheney]

I don't believe this issue is fixed, I am using a revision newer than both you quoted.

…

On Mon, 13 Feb 2017, 14:56 Travis Jeffery ***@***.***> wrote: Closed #27 <#27>. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#27 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAcA60sQGU-8jLAYWBf07dvF5OgcYS3ks5rb9R3gaJpZM4L-q50> .

[travisjeffery]

So you pulled and reproduced it after I left that comment? Just checking in case you're looking at the dates of when the commits were made rather than when they were merged into master.

[davecheney]

I'm currently at fee05bf. Would you accept a PR to add -race detection to your CI?

…

On Mon, Feb 13, 2017 at 3:50 PM, Travis Jeffery ***@***.***> wrote: So you pulled and reproduced it after I left that comment? Just checking in case you're looking at the dates of when the commits were made rather than when they were merged into master. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#27 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAcA4yVtLKu4tgeRtcBLc8P6Oqf8w3qks5rb-EGgaJpZM4L-q50> .

[travisjeffery]

Yeah that commit's behind. See where it in relation to the other commits here: https://github.com/travisjeffery/jocko/commits/master, or the compare: fee05bf...master

Absolutely. I was gonna add it, but it'd cool if you wanted to do it.