believening
believening
**Please provide a description of this PR:** an attempted fix for #38444
https://github.com/kubernetes/client-go/blob/1ac8d459351e21458fd1041f41e43403eadcbdba/tools/leaderelection/leaderelection.go#L197-L213 `OnStopedLeading` and `OnStopedElecting` are different, should we move the call of `OnStopedLeading` back? ``` if !le.acquire(ctx) { return // ctx signalled done } defer func() { le.config.Callbacks.OnStoppedLeading() }() ```
**问题描述** 注册仓库模块支持提取 pod 标签到服务实例元数据上,基本原理是将注册中心的实例 ip 和 pod ip 做匹配,两者相同时,执行添加。 这里要求一个前提是,pod 信息在从注册中心拉取数据转换成 istio se 前已经准备好了。slime 使用 pod informer 获取 pod 信息,这个 informer 的启动和不同注册中心 source 启动实际上是异步进行的,可能导致无法将 pod 标签正确的添加到服务实例元数据上,对于依赖这个特性的场景造成错误。 **影响的子模块(在下面列表中打'X')** - [ ]...
这里的转换过程历史上缺少 UT 覆盖,逻辑变动后转换结果的确认依赖 review,包括 nacos 及 zookeeper 均是如此,作为后续的工作项。 @believening _Originally posted by @believening in https://github.com/slime-io/slime/pull/486#discussion_r1468995677_
We have not used meshconfig historically or currently, and need to check if it can be removed. https://github.com/slime-io/slime/blob/0bdcb6c9baa5afa6709f03557a1d65254b0d3b4e/framework/bootstrap/adsc/adsc.go#L10-L12
- [ ] 伴随 libistio 引入了 istio client-go lib 等, 用以逐步替换 slime 对于 istio 以及 k8s 对象的数据结构定义(framework 中仍存在为替换的部分) _Originally posted by @believening in https://github.com/slime-io/slime/issues/468#issuecomment-1873917981_
这里可能导致单次请求失败就删除了这个server的所有服务、实例,带来的后果比较严重。 可以考虑: 1. 保留上一次cache,失败时沿用; 2. 单个失败升级为整体失败,跳过本次更新; > 这个可以单独建issue来优化 _Originally posted by @YonkaFang in https://github.com/slime-io/slime/pull/382#discussion_r1226126068_
**Is your feature request related to a problem? Please describe.** A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] After lazyload restarts, it...
https://github.com/slime-io/slime/blob/461f58eac63873c5f66bbbbf5ed56b8b4c07b77f/staging/src/slime.io/slime/modules/meshregistry/pkg/source/eureka/source.go#L87 https://github.com/slime-io/slime/blob/461f58eac63873c5f66bbbbf5ed56b8b4c07b77f/staging/src/slime.io/slime/modules/meshregistry/pkg/source/nacos/source.go#L116