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

Support UNIX domain socket in the example servers #609

Merged
merged 2 commits into from
Apr 22, 2018

Conversation

yugui
Copy link
Member

@yugui yugui commented Apr 20, 2018

c.f. #451

@yugui yugui requested a review from tmc April 20, 2018 03:20
defer cancel()

// Waiting for the server's getting available.
// TODO(yugui) find a better way to wait
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/healthz ;-)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:-)

True. But let me work on it in another PR.

@yugui yugui force-pushed the example/unix-domain-socket branch 2 times, most recently from 470620c to eed05d1 Compare April 22, 2018 01:22
@yugui yugui force-pushed the example/unix-domain-socket branch from eed05d1 to a31262a Compare April 22, 2018 01:22
@yugui
Copy link
Member Author

yugui commented Apr 22, 2018

Build fixed. Could you take another look?

@achew22 achew22 merged commit e8bd377 into master Apr 22, 2018
@yugui yugui deleted the example/unix-domain-socket branch April 24, 2018 00:42
yugui added a commit that referenced this pull request Apr 27, 2018
Wait for the gateway server to get ready.
#609 (comment)
yugui added a commit that referenced this pull request Apr 27, 2018
Wait for the gateway server to get ready.
#609 (comment)
yugui added a commit that referenced this pull request Apr 27, 2018
Wait for the gateway server to get ready.
#609 (comment)
yugui added a commit that referenced this pull request Apr 27, 2018
Wait for the gateway server to get ready.
#609 (comment)
achew22 pushed a commit that referenced this pull request Apr 27, 2018
* Avoid timing issues in the integration tests

Wait for the gateway server to get ready.
#609 (comment)
adasari pushed a commit to adasari/grpc-gateway that referenced this pull request Apr 9, 2020
* Lets example servers gracefully shutdown
* Support UNIX domain sockets in the example servers
adasari pushed a commit to adasari/grpc-gateway that referenced this pull request Apr 9, 2020
* Avoid timing issues in the integration tests

Wait for the gateway server to get ready.
grpc-ecosystem#609 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants