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
runtime: support unix domain socket #788
Conversation
48a3a00
to
dd73656
Compare
Codecov Report
@@ Coverage Diff @@
## master #788 +/- ##
==========================================
+ Coverage 20.85% 21.82% +0.97%
==========================================
Files 29 29
Lines 1482 1535 +53
==========================================
+ Hits 309 335 +26
- Misses 1136 1159 +23
- Partials 37 41 +4
Continue to review full report at Codecov.
|
291901a
to
8f63990
Compare
ping @mkosieradzki @artursouza |
d1c71fa
to
a50dc76
Compare
@daixiang0 please fix the failing tests. @mukundansundar can you please review? |
doc pr: dapr/docs#1730 |
65db46a
to
0457af2
Compare
@mukundansundar @yaron2 all tests pass, please review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a small change to fail on Windows instead of fallback.
@msfussell Could you review this PR? |
@daixiang0 please update the README with an example for how to use the new parameter. |
11a0c84
to
0ef564b
Compare
@yaron2 done. |
Signed-off-by: Long <long.dai@intel.com>
Signed-off-by: Long <long.dai@intel.com>
@yaron2 @mukundansundar could we merge it? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Great PR @daixiang0 |
* runtime: support unix domain socket Signed-off-by: Long <long.dai@intel.com> * feedback Signed-off-by: Long <long.dai@intel.com> * update example Signed-off-by: Long <long.dai@intel.com>
* runtime: support unix domain socket Signed-off-by: Long <long.dai@intel.com> * feedback Signed-off-by: Long <long.dai@intel.com> * update example Signed-off-by: Long <long.dai@intel.com> Signed-off-by: Andy Le <anhldbk@gmail.com>
* runtime: support unix domain socket Signed-off-by: Long <long.dai@intel.com> * feedback Signed-off-by: Long <long.dai@intel.com> * update example Signed-off-by: Long <long.dai@intel.com>
Description
Ref dapr/dapr#2864
Since go-sdk still does not support it, no e2e tests added.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #785
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: