Yiyiyimu opened a new pull request #4191:
URL: https://github.com/apache/apisix/pull/4191


   ### What this PR does / why we need it:
   fix #3673 #3937.
   
   Import health check of lua-resty-etcd, so 
   1. when one etcd node disconnected, apisix would try to connect another one 
and perform as normal, while health-check would continue print warning log 
about disconnected etcd node
   2. when all etcd nodes disconnected, apisix would try to re-connect with 
etcd, with a cyclic backoff interval for 1s, 2s, 4s, ..., 512s, 1s, 2s ... so 
the error log would not been flushed when etcd are all down.
   
   TODO:
   - [ ] for etcd node deployed in kubernetes, since nodes are behind the same 
domain, we could not know if one of them is taint, as talked in 
https://github.com/apache/apisix/issues/3692
   - [ ] add test, but since the current health check does not work for 
kubernetes, need to implement a new CI pipeline for multiple etcd nodes
   
   ### Pre-submission checklist:
   
   * [x] Did you explain what problem does this PR solve? Or what new features 
have been added?
   * [ ] Have you added corresponding test cases?
   * [ ] Have you modified the corresponding document?
   * [ ] Is this PR backward compatible? **If it is not backward compatible, 
please discuss on the [mailing 
list](https://github.com/apache/apisix/tree/master#community) first**
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to