weizhoublue

pending it for feature fix https://github.com/spidernet-io/spiderpool/pull/457/commits

finally, the recovery time of restarted spider-controller should be tested

1 need add some comments directly on the value.yaml and README.md @iiiceoo could handle it in August 2 it is under way

done

每个函数的健壮性不好，没有入参校验，错误了没有抛给外部 单元测试太简单，需要有正向和反向用例，甚至是并发竞争，且覆盖率100%

不让外部感知 erro 不叫 “不耦合”，外部不要 call function call 才叫 不耦合 没见过哪个 开源的 module 能自己内部消化掉 error，整个 function call 都失败了，还完全不让外部感知 应该要 传递 error 出来 ， 上层 业务 是否 忽略或处理，是上层的事，打印出错也可以上 层自己的logger 来控制 。 不需要...

有冲突了，解决下

(1)the spiderpool controller is not ready for running duplicated with https://github.com/spidernet-io/spiderpool/issues/482 (2) RestartNodeUntilReady() should change to RestartNodeUntilClusterReady() . who also wait for all pod is running . Or else later...

这个 怎么会 关了呢 ？ it 是 pending，还有 遗留 未解决

能不能关注下这个pr 的 进度 This branch has conflicts that must be resolved