Add timeout to abort joining of seed nodes

[ismaelhamed]

Port #23551

[ismaelhamed]

@Aaronontheweb there's some documentation related to these new settings, but the Akka.NET documentation is so different in this topic (cluster) that I'm not sure where to put it.

[Aaronontheweb]

@ismaelhamed do the best you can to add it to this PR, and I'll help you out if need be

[ismaelhamed]

@Aaronontheweb it's part of the Joining To Seed Nodes section in the Cluster Usage document. But the whole thing is missing. Maybe we're better off adding another PR with all the missing documentation related to the cluster later on.

[ismaelhamed]

@Aaronontheweb not sure where all those errors are coming from, because the unit tests are passing just fine in my machine.

[Aaronontheweb]

@ismaelhamed it's usually due to all of the blocking calls inside the TestKit and that we're running many different specs concurrently. It's a regular nuisance

[ismaelhamed]

@Aaronontheweb LOL... I thought I'd never see this PR green.

[Aaronontheweb]

weird, got a compilation failure this time? wtf

[ismaelhamed]

@ismaelhamed it's usually due to all of the blocking calls inside the TestKit and that we're running many different specs concurrently. It's a regular nuisance

Would automerge be an option for approved pull requests?

[Aaronontheweb]

@ismaelhamed I have to review stuff sometimes when these specs fail just to be sure we didn't make the problem worse. We have one PR open where the MNTR has failed in the same location every time we've run it, even though there's nothing obvious in the PR that would have affected it. That's why we're a but judicious about combing through these before merging.

[ismaelhamed]

@ismaelhamed I have to review stuff sometimes when these specs fail just to be sure we didn't make the problem worse. We have one PR open where the MNTR has failed in the same location every time we've run it, even though there's nothing obvious in the PR that would have affected it. That's why we're a but judicious about combing through these before merging.

Agree. It's just that the build system seems to frequently fail (regular nuisance like you said), and that makes some PR stale for a long time --I opened this PR in July and have force-pushed 8 times since then.

[Aaronontheweb]

Yeah, that's a nuisance - we need to get some changes implemented in the TestKit and in the way our test suites themselves run in order to alleviate this.