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

Runner v2.276.0 fails to open some stderr/stdout streams #927

Closed
thboop opened this issue Jan 21, 2021 · 6 comments
Closed

Runner v2.276.0 fails to open some stderr/stdout streams #927

thboop opened this issue Jan 21, 2021 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@thboop
Copy link
Collaborator

thboop commented Jan 21, 2021

Describe the bug
The newest runner version may fail to open some stderr/stdout streams. You can't typically open() a socket, and it looks like the stderr stream may be getting kept as a socket now.

To Reproduce
Run a ubuntu runner with the following yaml

name: CI test

on: push

jobs:
  build:
    name: test
    runs-on: self-hosted
    steps:
    - name: test tee /dev/stderr
      run : |
        set -x
        set -euo pipefail
        seq 0 10 | tee /dev/stderr | xargs echo

It will fail with tee: /dev/stderr: No such device or address
Expected behavior
The stderr stream should open and this should not fail.

Runner Version and Platform

v2.276.0 on Ubuntu.

Other information

Looks like it may be related to the .net upgrade
dotnet/runtime#46469

@thboop thboop added the bug Something isn't working label Jan 21, 2021
@zen0wu
Copy link

zen0wu commented Jan 21, 2021

Got bitten by this! @thboop Is there any way we could quickly fix this to unblock our CI?

@thboop
Copy link
Collaborator Author

thboop commented Jan 21, 2021

Hey @zen0wu , I'm working on rolling a new version right now, hope to have it available to all users in the next day or so!

Sorry to hear this is impacting your builds, I hope to have this ready asap!

@zen0wu
Copy link

zen0wu commented Jan 21, 2021

@thboop Thank you!

What's broken for us is when we do echo foo > /dev/stderr, I wonder if just removing removing > /dev/stderr and do a normal echo would fix this? I can try myself, but figured it'd be quicker to check with you.

@TingluoHuang
Copy link
Member

@zen0wu echo foo >&2 still work, only /dev/stderr is broken

@thboop thboop assigned thboop and unassigned TingluoHuang Jan 21, 2021
@thboop
Copy link
Collaborator Author

thboop commented Jan 21, 2021

This is in the process of rolling out and all runners should be upgraded in the next day or so.

@thboop
Copy link
Collaborator Author

thboop commented Jan 22, 2021

Should be fixed for everyone now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants