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

Health check dynamicly config #81

Closed
wants to merge 7 commits into from
Closed

Health check dynamicly config #81

wants to merge 7 commits into from

Conversation

noname007
Copy link
Collaborator

implement the dynamic config in the dashboard

import： not finished

  1. need to implment the view
  2. patch this left a interface , modify the health check conf dynamicly openresty/lua-resty-upstream-healthcheck#37

@sumory
Copy link
Collaborator

sumory commented Mar 15, 2017

  • handler的处理现在是写死的只检查default_upstream? 可以将这块做成配置
  • hc模块应该用的是ngx.timer来定时check的吧？要注意新加和删除check时这一块的处理
  • 需要接入dashboard页面
  • API的暴露也要完善下

如果这四块都做好了，那这个插件就很用处了

@sumory sumory closed this Mar 20, 2017
@noname007
Copy link
Collaborator Author

饭总，你不要关闭好不好，我在靠这个issue 开着，等前端的人来，找我 😆

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@noname007 @sumory