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

fix unstable ci test #51

Closed
utam0k opened this issue May 31, 2021 · 3 comments
Closed

fix unstable ci test #51

utam0k opened this issue May 31, 2021 · 3 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@utam0k
Copy link
Member

utam0k commented May 31, 2021

e.g. https://github.com/utam0k/youki/runs/2706688448

@utam0k
Copy link
Member Author

utam0k commented May 31, 2021

This issue may be caused by the uds.
https://github.com/utam0k/youki/runs/2714118427

@utam0k utam0k added enhancement New feature or request help wanted Extra attention is needed labels Jun 2, 2021
@utam0k
Copy link
Member Author

utam0k commented Jun 2, 2021

I'll try to implement pipe insted of using unix domain socket.

@utam0k
Copy link
Member Author

utam0k commented Jun 5, 2021

I found the solution for this issue. Youki needs to handle ErrorKind::WouldBlock.
https://github.com/jaemk/mini_http/blob/cb0301c92ebe1c2d6f284c877561bb1e92206b29/src/main.rs#L107

I will try fixing it comming soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant