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

flux-shell: support stdin flow control to subprocesses #6413

Open
chu11 opened this issue Nov 4, 2024 · 0 comments
Open

flux-shell: support stdin flow control to subprocesses #6413

chu11 opened this issue Nov 4, 2024 · 0 comments

Comments

@chu11
Copy link
Member

chu11 commented Nov 4, 2024

libsubprocess now supports flow control on writes to ensure no buffer overflow

This is supported in flux-exec (#6370) but not in the flux-shell when data is read from the KVS. #6292 is probably a necessity to get this to work.

Side note, stdin via file in flux-shell simply dups the file descriptor to the subprocesse's stdin. So buffer overflow should not be possible when using file input for stdin. So there is a decent workaround to buffer overflow in flux-shell that can be handled right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant