Skip to content

CI: Wait Only on Server#232

Merged
ric-evans merged 1 commit intomainfrom
allow-one-failed-client
Nov 3, 2023
Merged

CI: Wait Only on Server#232
ric-evans merged 1 commit intomainfrom
allow-one-failed-client

Conversation

@ric-evans
Copy link
Copy Markdown
Member

@ric-evans ric-evans commented Nov 3, 2023

Wait only on the server. We don't actually care about the clients, if they fail or not. Now, if all the clients fail, then the sever times out, the test fails, and we can look at client logs.

This has the effect of allowing one failed client, which forgives some troublesome issues seen occasionally.

@ric-evans ric-evans added enhancement New feature or request CI / Testing About CI and/or testing labels Nov 3, 2023
@ric-evans ric-evans self-assigned this Nov 3, 2023
@ric-evans ric-evans merged commit 639c86e into main Nov 3, 2023
@ric-evans ric-evans deleted the allow-one-failed-client branch November 3, 2023 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI / Testing About CI and/or testing enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant