-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
bugfix: registration service configuration missing and inconsistent #2524
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2524 +/- ##
=============================================
+ Coverage 51.05% 51.06% +0.01%
- Complexity 2762 2766 +4
=============================================
Files 550 550
Lines 17525 17529 +4
Branches 2064 2064
=============================================
+ Hits 8947 8951 +4
Misses 7735 7735
Partials 843 843
|
server/src/main/resources/registry.conf are the password and timeout configuration missing in the file? need to add it? |
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
Agree with @a364176773
explanation: the local clusterName variable needs to be used here, because the configuration properties of the cluster may not exist on the client side but only on the server side. |
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
done |
retry ci |
Ⅰ. Describe what this PR did
inconsistent Redis and Etcd3 publish and subscribe service
SOFA property name setting error
Nacos and SOFA missing client configuration
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews