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

关于ngx_http_dyups_module与nginx_upstream_check_module的更新合并问题 #1231

Closed
IYism opened this issue Apr 8, 2019 · 4 comments
Closed
Labels

Comments

@IYism
Copy link
Contributor

IYism commented Apr 8, 2019

目前这2个模块原作者的master分支已经修复了模块存在的bug及新特性的支持,怎么没见tengine官方也纳入进来呢?

https://github.com/yzprofile/ngx_http_dyups_module
https://github.com/yaoweibin/nginx_upstream_check_module

@wangfakang
Copy link
Collaborator

@IYism 多谢反馈,关于dyups模块的相关fix已经有对应的PR #1133 ,,另外如有其它遗漏的非常欢迎贡献PR.

@wangfakang wangfakang added the TODO label Apr 8, 2019
@IYism
Copy link
Contributor Author

IYism commented Apr 9, 2019

@wangfakang Hi,关于dyups模块以下pulls是tengine没有更新的。
yzprofile/ngx_http_dyups_module#97
yzprofile/ngx_http_dyups_module#104
yzprofile/ngx_http_dyups_module#109

关于check模块以下pulls是tengine没有更新的。
yaoweibin/nginx_upstream_check_module#146

@chobits
Copy link
Member

chobits commented May 6, 2019

最新功能(所有的dyups更新)已经提到这个pr:https://github.com/alibaba/tengine/pull/1255,很快会合并。

所以我现在正在关闭改issue,如果有其他问题可以reopen

@chobits chobits closed this as completed May 6, 2019
@yunoasgit
Copy link

Tengine users find it difficult to use dyups module
#1230
It is recommended that documents be updated consistently or linked to the latest URL

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

No branches or pull requests

4 participants