Closed farquet closed 5 years ago
Hi @farquet, thanks for the PR.
Question: There's already an existing totalRequests
flag that governs how long slow_cooker runs. Do you have a use case where that flag is not sufficient, necessitating this new iterations
flag?
For _slowcooker I agree. To get iterations
iterations printed, one can set totalRequests
to iterations * concurrency * qps * interval in seconds
.
But there is no such mapping for strest-grpc. This PR is here for consistency.
Following on the remark from https://github.com/BuoyantIO/strest-grpc/pull/55 ,
changed cleanup := make(chan bool, 2)
into cleanup := make(chan bool, 3)
since a third place in code can trigger a cleanup.
Thanks @farquet!
If
iterations
is greater than 0, the experiment will not be infinite, but print N reports and exit automatically afterwards.