-
Notifications
You must be signed in to change notification settings - Fork 347
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
[WIP] Enable IPv6 first cluster test #4590
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4590 +/- ##
==========================================
+ Coverage 65.47% 65.48% +0.01%
==========================================
Files 211 212 +1
Lines 31854 31918 +64
==========================================
+ Hits 20855 20903 +48
- Misses 9754 9770 +16
Partials 1245 1245 ☔ View full report in Codecov by Sentry. |
Signed-off-by: zirain <zirain2009@gmail.com>
@zirain should this be in v1.2.0 ? |
therer're a lot of pending PRs that block this one. |
Signed-off-by: zirain <zirain2009@gmail.com>
IPFamily: net.PreferIPFamily(ipv6First, infra.Config), | ||
} | ||
|
||
// TODO: this's a temporary solution to support IPv6 first cluster for Envoy Gateway. |
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.
Remind myself not to forget to send a separate PR with modification here.
WIP, use CI to test some pending PRs together.