Benjamin Wang

Results 627 comments of Benjamin Wang

OOPS, I misread the comment above, I closed the PR instead of this issue. I am good as long as eventually we can integrate the line chart. Please feel free...

Please read https://github.com/etcd-io/etcd/issues/17247#issuecomment-1895725142

Overall it seems not a good idea to verify etcd's correctness depending on WAL data. It's a little strange that you verify etcd's correctness, but you trust the WAL data...

OpenBSD has no user-settable per-socket TCP keepalive options. [tcpsockopt_openbsd.go#L13-L15](https://github.com/golang/go/blob/53279a6af372e3708afe8eaf618d56ee98edf045/src/net/tcpsockopt_openbsd.go#L13-L15)

Followup https://github.com/etcd-io/etcd/pull/15812#issuecomment-1929504990 cc @Elbehery

We also need to add a changelog item, but it's OK to add it in a separate PR. cc @jmhbnz @ivanvc @ArkaSaha30 to take a second look.

This is the draft PR per our discussion in the [doc](https://docs.google.com/document/d/1U9hAcZQp3Y36q_JFiw2VBJXVAo2dK2a-8Rsbqv3GgDo/edit) @mitake @ptabor @serathius @chaochn47 @fuweid Please let me know if you have any immediate comment or concern, before I...

Thanks all for the feedback, which basically makes sense to me. The overall idea is basically coming from one of @ptabor 's comments in the doc. Will update & cleanup...

Please also see the updated doc https://docs.google.com/document/d/1U9hAcZQp3Y36q_JFiw2VBJXVAo2dK2a-8Rsbqv3GgDo/edit#heading=h.3oeryohw1c9o

https://github.com/etcd-io/etcd/issues/15477#issuecomment-1518568351