-
Notifications
You must be signed in to change notification settings - Fork 338
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
fix: use independent dns service for UDP e2e test #753
Conversation
…rce in kube-system namespace
@gxthrj ping |
test/e2e/scaffold/k8s.go
Outdated
if readyNum == desired { | ||
return "Service is now available", nil | ||
} | ||
return "Endpoints not ready yet", nil |
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.
Add the expected number and the actual number to the message, it's useful to troubleshoot issues.
Codecov Report
@@ Coverage Diff @@
## master #753 +/- ##
==========================================
+ Coverage 31.69% 31.70% +0.01%
==========================================
Files 65 66 +1
Lines 6639 6640 +1
==========================================
+ Hits 2104 2105 +1
Misses 4280 4280
Partials 255 255
Continue to review full report at Codecov.
|
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.
Good job
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.
👍
wait for CI
CI 🟢 |
Use independent dns service for UDP e2e test, avoid create resource in kube-system namespace
Please answer these questions before submitting a pull request
Why submit this pull request?
Bugfix
New feature provided
Improve performance
Backport patches
Related issues
bug: e2e test will fail randomly #751