slime-io / slime

An intelligent ServiceMesh manager based on Istio
https://slime-io.github.io/
Other
424 stars 78 forks source link

meshregistry:nacos/eureka source fetch from registry fails will build delete event #389

Open believening opened 1 year ago

believening commented 1 year ago
          这里可能导致单次请求失败就删除了这个server的所有服务、实例,带来的后果比较严重。

可以考虑:

  1. 保留上一次cache,失败时沿用;

  2. 单个失败升级为整体失败,跳过本次更新;

这个可以单独建issue来优化

_Originally posted by @YonkaFang in https://github.com/slime-io/slime/pull/382#discussion_r1226126068_

believening commented 1 year ago

关于对接 source 策略的提议:


上述策略在当前实现中的匹配度较低的地方:

一个潜在的约束是:一个 se 只会来自于某一个 service。在不考虑 regroup 时,对于 List instances by service 失败时不更新 se 有帮助。


当前 meshreg 模块对接多注册中心时,存在两种模式:

zk 模式和对接策略提议更加接近。