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.

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

修复:同一个实例注册到两个consul客户端上,从consul的一个客户端下线实例,ngx_http_upsync_server_t根据i… #253

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

muzhao1
Copy link

@muzhao1 muzhao1 commented May 15, 2019

…p与port删除实例,会把两个实例都删除(删除逻辑有bug),导致后续不进行服务发现

…p与port删除实例,会把两个实例都删除(删除逻辑有bug),导致后续不进行服务发现
@muzhao1 muzhao1 closed this May 15, 2019
@muzhao1
Copy link
Author

muzhao1 commented May 15, 2019

如同一个实例注册到两个consul客户端上,访问upstream显示如下:
Upstream name: marketing; Backend server count: 2
server 10.100.23.12:8073 weight=11 max_fails=2 fail_timeout=10s;
server 10.100.23.12:8073 weight=11 max_fails=2 fail_timeout=10s;
从其中一个consul客户端上下线实例,访问upstream_show显示如下:
Upstream name: marketing; Backend server count: 1
在重新注册实例后,访问upstream_show依旧显示的跟上面一样,没有继续进行服务发现,一直假死,只能nginx reload,才能恢复

@muzhao1 muzhao1 reopened this May 15, 2019
@xiaokai-wang
Copy link
Member

@muzhao1 为什么要注册两个相同的server 到consul?有什么特殊场景吗

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants