Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

runtests: internal IPC fixes & test update #11160

wants to merge 3 commits into from


Copy link

  • use the correct fd after select
  • handle interrupted reads from IPC pipes
  • fix the log directory in test425

These can be interrupted by signals, especially SIGINT to shut down, and
must be restarted so the IPC call arrives correctly. If the read just
returns an error instead, the IPC calling state will go out of sync and
a proper shutdown won't happen.

Ref: #10818
This must be %LOGDIR to let it work with parallel tests.

Ref: #10969
The code was using the wrong fd when determining which runner was ready
with a response.

Ref: #10818
Closes #11160
@github-actions github-actions bot added the tests label May 19, 2023
@curl curl deleted a comment from Sharlo3 May 20, 2023
@dfandrich dfandrich closed this in c95ca8d May 21, 2023
@dfandrich dfandrich deleted the dfandrich/pr11160 branch May 21, 2023 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet

Successfully merging this pull request may close these issues.

None yet

1 participant