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 the connection leak problem of app callback gRPC client #6274
Conversation
Signed-off-by: gaofeihang <gaofeihang@gmail.com>
/ok-to-test |
2 similar comments
/ok-to-test |
/ok-to-test |
Dapr E2E testCommit ref: 9a2c82b ✅ Build succeeded for linux/amd64
✅ Build succeeded for linux/arm64
✅ Infrastructure deployed
|
Dapr E2E testCommit ref: 9a2c82b ✅ Build succeeded for linux/amd64
✅ Infrastructure deployed
✅ Build succeeded for linux/arm64
✅ Build succeeded for windows/amd64
|
Dapr E2E testCommit ref: 9a2c82b ✅ Build succeeded for linux/amd64
✅ Infrastructure deployed
✅ Build succeeded for windows/amd64
✅ Build succeeded for linux/arm64
|
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.
This makes sense, thanks for catching this
/ok-to-test |
Dapr E2E testCommit ref: 9a2c82b ✅ Build succeeded for linux/amd64
✅ Infrastructure deployed
✅ Build succeeded for linux/arm64
✅ Build succeeded for windows/amd64
|
/ok-to-test |
1 similar comment
/ok-to-test |
Dapr E2E testCommit ref: 3860898 ✅ Build succeeded for linux/amd64
✅ Infrastructure deployed
✅ Build succeeded for linux/arm64
✅ Build succeeded for windows/amd64
|
Dapr E2E testCommit ref: 3860898 ✅ Build succeeded for linux/amd64
✅ Infrastructure deployed
✅ Build succeeded for linux/arm64
✅ Build succeeded for windows/amd64
|
Codecov Report
@@ Coverage Diff @@
## master #6274 +/- ##
==========================================
- Coverage 64.45% 64.44% -0.02%
==========================================
Files 189 189
Lines 18885 18891 +6
==========================================
+ Hits 12173 12174 +1
- Misses 5744 5751 +7
+ Partials 968 966 -2
|
/ok-to-perf |
1 similar comment
/ok-to-perf |
Dapr perf testCommit ref: b001582 ✅ Infrastructure deployed
✅ Build succeeded
❌ Perf tests failedPlease check the logs for details on the error. |
Dapr perf testCommit ref: 46bcf53 ✅ Build succeeded
✅ Infrastructure deployed
❌ Perf tests failedPlease check the logs for details on the error. |
related to #6301 |
Description
Issue reference
Fix #6273
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: