Benjamin Wang
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