cloud-bulldozer / benchmark-operator

The Chuck Norris of cloud benchmarks
Apache License 2.0
285 stars 129 forks source link

Enable multiple client/server pairs with NodePort uperf #755

Closed smalleni closed 2 years ago

smalleni commented 2 years ago

Signed-off-by: Sai Sindhur Malleni smalleni@redhat.com

Description

Currently uperf with NodePort servicetype only works with pairs: 1. This PR fixes the issue with running multiple pairs.

Fixes

757

Thanks to Murali for the initial feedback and testing the PR to find issues.

mohit-sheth commented 2 years ago

/rerun all

smalleni commented 2 years ago

Hey @smalleni, can you add a test case and update docs for this?

Hey @rsevilla87 docs already exist and I just modified an existing test case to run with 2 pairs. Hopefully that's good.

smalleni commented 2 years ago

@rsevilla87 @jtaleric should be good now (defaulting to 2 pairs for nodeport and fixed the location of the test).