Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ contents:
inline: |
. {
errors
bufsize 512
health :18080
forward . {{`{{- range $upstream := .DNSUpstreams}} {{$upstream}}{{- end}}`}} {
policy sequential
force_tcp
Copy link
Contributor

Choose a reason for hiding this comment

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

@sadasu Your PR description says "prefer udp", is that out of date and you ended up moving to force_tcp? Or should this be prefer_udp?

prefer_udp makes more sense to me (and gemini mentioned it is recommended on aws eks!) but I haven't been following this pr closely

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, prefer_udp was one of the first options we tried because that was the preferred solution during high load conditions. Testing revealed that the upstream servers reduced load by not responding to UDP requests.
We tried setting the max_concurrent to a value < 1000 which is the default value as a way to circumvent i/o timeouts due to UDP port exhaustion. But, during our testing we found that udp port exhaustion was not the cause of our i/o timeouts.
force_tcp was providing us with the best results.

}
cache 30
reload
Expand Down