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

Expand on endpoint selection strategy #31

Merged
merged 1 commit into from Sep 11, 2019
Merged

Conversation

thanethomson
Copy link
Member

This commit allows for a broader endpoint selection strategy for load testing. Specifically, there are 3 possible strategies for selecting endpoints for load testing:

  1. "supplied" - use only supplied endpoints for load testing.
  2. "discovered" - use only discovered endpoints for load testing (and none of the supplied ones).
  3. "any" - use both supplied and discovered endpoints for testing.

This allows for permutations such as when one supplies a seed node as an endpoint, but one does not want to submit transactions to that seed node.

This necessarily introduces another command line parameter --max-endpoints, which allows one to restrict the maximum number of endpoints returned for load testing after applying the selection
strategy.

This commit allows for a broader endpoint selection strategy for load
testing. Specifically, there are 3 possible strategies for selecting
endpoints for load testing:

1. "supplied" - use only supplied endpoints for load testing.
2. "discovered" - use only discovered endpoints for load testing (and
   none of the supplied ones).
3. "any" - use both supplied and discovered endpoints for testing.

This allows for permutations such as when one supplies a seed node as an
endpoint, but one does not want to submit transactions to that seed
node.

This necessarily introduces another command line parameter
`--max-endpoints`, which allows one to restrict the maximum number of
endpoints returned for load testing after applying the selection
strategy.
@thanethomson thanethomson merged commit d2a9f0b into master Sep 11, 2019
@thanethomson thanethomson deleted the fix/endpoint-selection branch September 11, 2019 13:12
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.

None yet

1 participant