Yiyiyimu commented on a change in pull request #4191:
URL: https://github.com/apache/apisix/pull/4191#discussion_r645176342
##########
File path: apisix/core/config_etcd.lua
##########
@@ -529,6 +531,17 @@ local function _automatic_fetch(premature, self)
return
end
+ if not health_check.conf then
Review comment:
In lua-resty-etcd
https://github.com/api7/lua-resty-etcd/blob/1fe01f77dda252f2d95eed0a7635f3479097b8ed/lib/resty/etcd/health_check.lua#L70-L83
And I think I didn't configure it correctly but failed to find the reason.
The health check would renew its value in shared dict for every loop in
`automatic_fetch`, so etcd health check would keep refreshing the tainted node
status but not directly jump over it. Removing the error check imported in this
PR still got the same problem.
--
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]
