Add revision to reads in TXN to robustness test and add regression test
What would you like to be added?
Issue https://github.com/etcd-io/etcd/issues/18667 was not detected by robustness test because TXN we generate don't set revision in range. We should fix this gap.
TODO:
- Add randomly adding revision to generating multiTXNOps https://github.com/etcd-io/etcd/blob/448fb7e36c11971d610bef241f2ffb7ddf39788d/tests/robustness/traffic/etcd.go#L301-L332
- Add regression test
make test-robustness-issue18667like in https://github.com/etcd-io/etcd/pull/17680/files
Note, until fix for https://github.com/etcd-io/etcd/issues/18667 is merged, we cannot enable running this test by default. Might need to make setting revision optional and only run it as regression test, only later enabling it by default.
As discussed on Slack @henrybear327 volunteered to work on it.
Why is this needed?
Cover detect and prevent https://github.com/etcd-io/etcd/issues/18667
Any progress?
ping @henrybear327
@serathius isn't https://github.com/etcd-io/etcd/issues/19009 a duplicated issue of this one?
I am working on this issue this week.
@ArkaSaha30 are you still working on https://github.com/etcd-io/etcd/issues/18667? Or should I take over the task there? :)
That's a separate issue about implementing an e2e tests, while this one is about robustness test.
@ArkaSaha30 are you still working on https://github.com/etcd-io/etcd/issues/18667? Or should I take over the task there? :)
Sorry for the delay, I'll be taking it up this week.
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.
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.
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.
https://github.com/etcd-io/etcd/pull/20454
This PR should have resolved this issue.