-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Reapply "udp: set Don't Fragment(DF) bit in IP packet header (#36341)" #36437
base: main
Are you sure you want to change the base?
Conversation
…oxy#36341)" This reverts commit ea1d345. Signed-off-by: Dan Zhang <danzh@google.com>
Signed-off-by: Dan Zhang <danzh@google.com>
Signed-off-by: Dan Zhang <danzh@google.com>
@@ -1330,7 +1330,7 @@ TEST_P(ClientIntegrationTest, TestProxyResolutionApi) { | |||
|
|||
// This test is simply to test the IPv6 connectivity check and DNS refresh and make sure the code | |||
// doesn't crash. It doesn't really test the actual network change event. | |||
TEST_P(ClientIntegrationTest, OnNetworkChanged) { | |||
TEST_P(ClientIntegrationTest, OnNetworkChanged1) { |
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.
nit: Revert this change?
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 is to trigger mobile tests. Looks like the tests are still failing due to bulk transfer error: https://github.com/envoyproxy/envoy/actions/runs/11166604324/job/31041663403
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.
Gotcha! It's all good then :)
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.
In fact, those failures look weird, can you take a look?
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.
I believe it's related to #36426 (the issue, not the PR). The flakiness is still there caused by AndroidEngineExplicitFlowTest.post_multipleRequests_randomBehavior
. Let me take a closer look at it further. For the time being, you just may need to retest.
/retest |
/retest |
Commit Message: removed the static_assert as it is expected that some platform, especially some iOS versions supports neither of the socket options. In this case, Envoy wont' set DF bit.
Additional Description: reland #36341
Risk Level: low
Testing: new unit tests
Docs Changes: N/A
Release Notes: Yes
Platform Specific Features: N/A
Runtime guard: envoy.reloadable_features.udp_set_do_not_fragment