-
Notifications
You must be signed in to change notification settings - Fork 152
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
dynamic Router and Cluster from registry center #632
Conversation
…roup change path to /ApplicationName/Interface/Method
…re-registry-center # Conflicts: # pkg/adapter/dubboregistry/registry/zookeeper/service_listener.go # pkg/adapter/dubboregistry/registrycenter.go # pkg/server/router_manager.go
Quality Gate passedIssues Measures |
升级3.1.1的时候屏蔽掉了几个UT,现在应该可以打开了 |
ci 失败,check 下看看 |
@coderabbitai review |
@CodiumAI-Agent /review |
ci失败是预期内,因为 #600 (comment) |
ci fixed in apache/dubbo-go-pixiu-samples#58 |
Quality Gate passedIssues Measures |
ignore envoy ci failed -> CI / Go Test ubuntu-latest - Go 1.18 (pull_request_target) Failing after 41s |
Good job |
What this PR does:
dynamic Router and Cluster from registry center
ref #600
Special notes for your reviewer:
support application level and interface level service discovery
support zk and nacos registry
application level service discovery use prefix url pattern to match http path,beacuse there is no methods information in the registry.
Does this PR introduce a user-facing change?:
demo project
https://github.com/FoghostCn/dubbo-go-pixiu-samples/blob/feature-registry-center/dubbogo/simple/zookeeper