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

[CI] Fix SIGPIPE behavior in GH Actions default runner #357

Closed
wants to merge 4 commits into from

Conversation

itislu
Copy link
Collaborator

@itislu itislu commented Jul 6, 2024

This is a workaround for the default GitHub Actions runner which inherits to all jobs and steps a SIGPIPE handler that does nothing.
The behavior can be reproduced locally by running trap '' SIGPIPE and then commands that produce a SIGPIPE signal, like cat | ls.

The workaround is to compile a C program that resets the SIGPIPE handler and call the mstest command from it.
A shell script wrapper like trap - SIGPIPE ; exec does not work.

Reference about the behavior of GitHub Actions runner:
actions/runner#2684

This is a workaround for the default GitHub Actions runner which inherits to all jobs and steps a SIGPIPE handler that does nothing.
The behavior can be reproduced locally by running `trap '' SIGPIPE` and then commands that produce a SIGPIPE signal, like `cat | ls`.

The workaround is to compile a C program that resets the SIGPIPE handler and call the mstest command from it.
A shell script wrapper like `trap - SIGPIPE ; exec` does not work.

Reference about the behavior of GitHub Actions runner:
actions/runner#2684
@itislu itislu added development DevOps related high priority This issue must be fixed labels Jul 6, 2024
@itislu itislu removed the high priority This issue must be fixed label Jul 7, 2024
@itislu
Copy link
Collaborator Author

itislu commented Jul 10, 2024

@itislu itislu closed this Jul 10, 2024
@itislu itislu deleted the ci-sigpipe-behavior branch July 10, 2024 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development DevOps related
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant