Skip to content
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.

Sign up for GitHub

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

Jump to bottom

dubbo 2.7.22 + nacos 2.1.0 使用zone不生效 #12940

Closed
Meedly opened this issue Aug 21, 2023 · 4 comments
Closed

dubbo 2.7.22 + nacos 2.1.0 使用zone不生效 #12940

Meedly opened this issue Aug 21, 2023 · 4 comments
Labels
status/waiting-for-feedback Need reporters to triage

Comments

@Meedly
Copy link

Meedly commented Aug 21, 2023 •
edited
Loading

参考了 #9679 #9673

调用链路：

consumer双发现，注册2个nacos地址

dubbo:
registries:
nacosA:
address: nacosA
zone: nacosA
group: nacosA
cluster: failsafe

nacosB:
  address: nacosB
  zone: nacosB
  group: nacosB
  cluster: failsafe

现象：
启用cluster: failsafe时，会进行AbstractClusterInvoker.doSelect进行二选一，后续无法进入ZoneAwareClusterInvoker使用zone过滤

当不使用cluster: failsafe时，会进入ZoneAwareClusterInvoker，但是clusterInovker.isAvailable均为false
@Meedly Meedly added the type/bug Bugs to being fixed label Aug 21, 2023
@AlbumenJ
Copy link
Member

Please try on 3.2.5. 2.7.x is no longer maintained.

@Meedly
Copy link
Author

Meedly commented Aug 24, 2023

经过排查，目前发现是多注册中心情况下
invoker-> cachedInvoker会被覆盖，导致实际该注册中心下的节点不属于自己，截止到2.7.22未修复

@CrazyHZM
Copy link
Member

@Meedly
Does it still exist in 3.2.x?

@CrazyHZM CrazyHZM added status/waiting-for-feedback Need reporters to triage and removed type/bug Bugs to being fixed labels Dec 27, 2023
@AlbumenJ
Copy link
Member

No news is good news. Please feel free to create a new issue if you have any question.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status/waiting-for-feedback Need reporters to triage
Projects
Dubbo Board
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@AlbumenJ @Meedly @CrazyHZM