-
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
Feature: compatible with acm #2699
Conversation
因为 我们要从自己搭的nacos环境切换到阿里云上的 MSE+ACM,但是ACM(配置中心)使用nacos api只能使用endpoint不能使用serverAddr,所以在pr中增加了使用endpoint的兼容 |
Codecov Report
@@ Coverage Diff @@
## develop #2699 +/- ##
=============================================
+ Coverage 51.12% 51.13% +0.01%
- Complexity 2847 2849 +2
=============================================
Files 566 566
Lines 18072 18072
Branches 2141 2141
=============================================
+ Hits 9240 9242 +2
Misses 7944 7944
+ Partials 888 886 -2
|
change the title to feature: compatible acm |
已经测试通过 seata server能注册上,client 能注册到server |
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
seata config compatible aliyun acm endpoint