-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
xdsclient: switch more transport tests to e2e style (2/N) #7693
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7693 +/- ##
==========================================
+ Coverage 81.79% 81.83% +0.03%
==========================================
Files 361 361
Lines 27822 27827 +5
==========================================
+ Hits 22756 22771 +15
+ Misses 3863 3856 -7
+ Partials 1203 1200 -3
|
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.
lgtm
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.
LGTM.
f961246
to
2604316
Compare
This PR moves more tests in the
transport
package to live at thexdsclient
package and makes them e2e style.This PR also makes the stream backoff configurable at the xDS client level. This will be used later on as well, as part of the refactor.
#a71-xds-fallback
#xdsclient-refactor
Addresses #6902
RELEASE NOTES: none