Skip to content

Commit

Permalink
fqdn: Fix goroutine leak in transparent-mode
Browse files Browse the repository at this point in the history
When number of concurrent dns requests was moderately high, there was a
chance that some of the gorutines would get stuck waiting for response.
Contains fix from cilium/dns#10

Signed-off-by: Marcel Zieba <marcel.zieba@isovalent.com>
  • Loading branch information
marseel authored and rolinh committed Apr 16, 2024
1 parent 14b6145 commit 804d5f0
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 6 deletions.
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ require (
github.com/cilium/checkmate v1.0.3
github.com/cilium/coverbee v0.3.2
github.com/cilium/deepequal-gen v0.0.0-20231116094812-0d6c075c335f
github.com/cilium/dns v1.1.51-0.20231120140355-729345173dc3
github.com/cilium/dns v1.1.51-0.20240411200813-4e6b438d9e05
github.com/cilium/ebpf v0.14.0
github.com/cilium/endpointslice-controller v0.0.0-20240409203012-75cb5d61db1b
github.com/cilium/fake v0.6.1
Expand Down
4 changes: 2 additions & 2 deletions go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

20 changes: 18 additions & 2 deletions vendor/github.com/cilium/dns/shared_client.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion vendor/modules.txt
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ github.com/cilium/coverbee/pkg/verifierlog
## explicit; go 1.20
github.com/cilium/deepequal-gen
github.com/cilium/deepequal-gen/generators
# github.com/cilium/dns v1.1.51-0.20231120140355-729345173dc3
# github.com/cilium/dns v1.1.51-0.20240411200813-4e6b438d9e05
## explicit; go 1.18
github.com/cilium/dns
# github.com/cilium/ebpf v0.14.0
Expand Down

0 comments on commit 804d5f0

Please sign in to comment.