Skip to content
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

Allow backoff to not penalize all workloads in mixed mode #4564

Merged
merged 1 commit into from
Sep 13, 2022

Conversation

sadhansood
Copy link
Contributor

@sadhansood sadhansood commented Sep 12, 2022

In the current testnet setup, we noticed a couple times when shared counter workload completely stop making progress that it affected the transaction throughout of transfer object workload. This is because the backoff in load gen considers total pending requests across all workloads before deciding to send a request. We aim to fix this behavior by adding a new mode (disjoint_mode) where different workloads run completely independently such that even if one is completely dead, the other keeps making progress as expected.

@sadhansood sadhansood marked this pull request as ready for review September 12, 2022 07:11
@sadhansood sadhansood force-pushed the sadhan/workload_disjoint_mode branch from ba61fa6 to 6dccdf7 Compare September 12, 2022 22:49
@sadhansood sadhansood force-pushed the sadhan/workload_disjoint_mode branch from 6dccdf7 to 0bd4dfe Compare September 13, 2022 00:52
* Fix metric collection for network client in stress test

* Cleanup old benchmark code
@sadhansood sadhansood force-pushed the sadhan/workload_disjoint_mode branch from 0bd4dfe to 8580145 Compare September 13, 2022 00:54
@sadhansood sadhansood enabled auto-merge (squash) September 13, 2022 01:02
@sadhansood sadhansood merged commit b85bd34 into main Sep 13, 2022
@sadhansood sadhansood deleted the sadhan/workload_disjoint_mode branch September 13, 2022 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants