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

fqdn/dnsproxy: set SO_REUSEPORT on listening socket #12728

Merged
merged 1 commit into from
Jul 31, 2020

Conversation

tklauser
Copy link
Member

Now that we start re-using the same port for the DNS proxy across
restarts (see #12718), set the SO_REUSEPORT option on the listening
port. This given the proxy a better chance to re-bind() upon restarts.

Now that we start re-using the same port for the DNS proxy across
restarts (see #12718), set the SO_REUSEPORT option on the listening
port. This given the proxy a better chance to re-bind() upon restarts.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser requested a review from jrajahalme July 31, 2020 08:43
@tklauser tklauser requested a review from a team as a code owner July 31, 2020 08:43
@maintainer-s-little-helper maintainer-s-little-helper bot added the kind/backports This PR provides functionality previously merged into master. label Jul 31, 2020
@tklauser tklauser added the area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. label Jul 31, 2020
@tklauser
Copy link
Member Author

test-backport-1.7

@tklauser tklauser added the release-note/misc This PR makes changes that have no direct user impact. label Jul 31, 2020
@qmonnet qmonnet removed the request for review from jrajahalme July 31, 2020 13:32
@nathanjsweet nathanjsweet merged commit 88174d7 into v1.7 Jul 31, 2020
@nathanjsweet nathanjsweet deleted the pr/tklauser/v1.7-proxy-reuseport branch July 31, 2020 14:25
tklauser added a commit that referenced this pull request Aug 6, 2020
Forward port of #12728

Once we start re-using the same port for the DNS proxy across
restarts (see #12794), it's useful to set the SO_REUSEPORT option on the
listening port. This given the proxy a better chance to re-bind() upon
restarts.

Updates #12794

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
tklauser added a commit that referenced this pull request Aug 7, 2020
Forward port of #12728

Once we start re-using the same port for the DNS proxy across
restarts (see #12794), it's useful to set the SO_REUSEPORT option on the
listening port. This given the proxy a better chance to re-bind() upon
restarts.

Updates #12794

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
jrajahalme pushed a commit that referenced this pull request Aug 7, 2020
[ upstream commit 3abd218 ]

Forward port of #12728

Once we start re-using the same port for the DNS proxy across
restarts (see #12794), it's useful to set the SO_REUSEPORT option on the
listening port. This given the proxy a better chance to re-bind() upon
restarts.

Updates #12794

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
tklauser added a commit that referenced this pull request Aug 10, 2020
[ upstream commit 3abd218 ]

Forward port of #12728

Once we start re-using the same port for the DNS proxy across
restarts (see #12794), it's useful to set the SO_REUSEPORT option on the
listening port. This given the proxy a better chance to re-bind() upon
restarts.

Updates #12794

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. kind/backports This PR provides functionality previously merged into master. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants