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

Investigate buffering issues on GitHub runners #1301

Open
0x2b3bfa0 opened this issue Dec 30, 2022 · 5 comments
Open

Investigate buffering issues on GitHub runners #1301

0x2b3bfa0 opened this issue Dec 30, 2022 · 5 comments
Assignees
Labels
technical-debt Refactoring, linting & tidying

Comments

@0x2b3bfa0
Copy link
Member

0x2b3bfa0 commented Dec 30, 2022

#1299 is a duct tape workaround to what seems to be a buffering issue on the GitHub self-hosted runner output, which prevents our logic from detecting when the runner transitions to a busy state in a timely manner; see #1255 (comment) for context.

@0x2b3bfa0 0x2b3bfa0 added the technical-debt Refactoring, linting & tidying label Dec 30, 2022
@dacbd
Copy link
Contributor

dacbd commented Dec 30, 2022

hmm I suspect this to be the source of my failed termination issues, on cml-playground, I say that I have observed the opposite though... the runner detects the first job and enters its busy state but fails to detect the job finishing and never enters an idle state.

@tasdomas
Copy link
Contributor

tasdomas commented Dec 30, 2022 via email

@dacbd
Copy link
Contributor

dacbd commented Dec 30, 2022

We also call the GitHub API, but that also has its own issues, and it's only used as final validation.

@0x2b3bfa0
Copy link
Member Author

the runner detects the first job and enters its busy state but fails to detect the job finishing and never enters an idle state

Sounds, indeed, like a buffering issue. 🤔 Node.js treatment of file descriptors is, indeed, cough very particular, and anything is possible.

@0x2b3bfa0
Copy link
Member Author

Is there no better way of detecting the runner state, other than parsing logs?

At least, none known to me. 😰

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
technical-debt Refactoring, linting & tidying
Projects
None yet
Development

No branches or pull requests

3 participants