-
Notifications
You must be signed in to change notification settings - Fork 9.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
functional-tests: Provide more advanced network failure injections #5614
Comments
@chancez For the ordering, is it actually testing the TCP+HTTP stack? |
@xiang90 I think ordering makes sense among multiple connections. For example, the bridge could buffer data on each connection for a bit, then control the order that it drains the buffers into the receiver. |
@heyitsanthony Oh. Sure. You are right. |
/cc @fanminshi You might be interested in this one. |
@xiang90 cool! |
add more network failures such as packet corruption, reordering, loss, and network partition. resolve etcd-io#5614
add more network failures such as packet corruption, reordering, loss, and network partition. resolve etcd-io#5614
This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions. |
Someone in the kubernetes cluster-sig-ops group wrote the following set of scripts to do black box fauult injection against etcd: https://github.com/jdumars/etcdeath
I see we have https://github.com/coreos/etcd/blob/master/pkg/netutil/isolate_linux.go today which does total packet loss, and latency, but etcdeath provides some interesting additional failures that would be good to add to our functional tests.
In particular the following:
packet corruption:
partial packet loss (instead of complete packet loss):
packet reordering:
partitioning:
Those are just the ones I see we currently don't have, but they seem to be genuinely good tests to run, so I'm just suggesting we add these as alternative ways to stress etcd.
The text was updated successfully, but these errors were encountered: