-
Notifications
You must be signed in to change notification settings - Fork 160
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
Rebuild discovery-first config server #737
Conversation
- use private ref to Discovery.ClientBase - add ability to swap with DI-provided discovery client and logging add some missing services during bootstrap
Codecov Report
@@ Coverage Diff @@
## main #737 +/- ##
==========================================
+ Coverage 74.22% 74.26% +0.04%
==========================================
Files 1312 1313 +1
Lines 45593 45667 +74
Branches 8957 8955 -2
==========================================
+ Hits 33840 33916 +76
Misses 9246 9246
+ Partials 2507 2505 -2
Continue to review full report at Codecov.
|
Respect Eureka:Client:Enabled and Spring:Cloud:Discovery:Enabled
{ | ||
break; | ||
} | ||
|
||
attempts++; | ||
if (attempts < _settings.RetryAttempts) | ||
if (attempts <= _settings.RetryAttempts) |
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.
Previously setting RetryAttempts
to 1 would result in a single total call rather than 1 call with 1 retry
SonarCloud Quality Gate failed. |
Following up on #660, Rebuild discovery-first config server:
Also adds:
spring:cloud:discovery:enabled
andeureka:client:enabled