Skip to content

Conversation

@bklang
Copy link
Member

@bklang bklang commented Oct 6, 2014

This PR allows scaling up the CPS rate over time, linearly.

However, I'm not quite happy with it because the increment interval is hard-coded at 1 second. This means going from 5CPS to 35CPS can only happen in 30 seconds, which isn't terribly useful to watch the impact of growing CPS. The problem is SIPp, which re-uses the stats interval (-fd) setting to determine the CPS increase interval. The correct fix for this would be to create a separate interval argument to SIPp and use that. TBD.

@bklang
Copy link
Member Author

bklang commented Oct 6, 2014

See SIPp/sipp#107

@wdrexler
Copy link
Contributor

wdrexler commented Oct 6, 2014

Could you not just specify your CPS increase speed by using SIPp's -rate_increase option? "-rate_increase 30" would get you from 5 to 35 CPS in a second with -fd 1

@bklang
Copy link
Member Author

bklang commented Oct 6, 2014

@wdrexler The problem is that I want to increase more slowly. For example, I want to go from 10CPS to 20CPS over a period of 10 minutes, with 1 CPS increase each minute. To do that, I would have to set -fd 60s, which has the negative side effect of lowering the stats collection interval, since they share the setting.

@bklang bklang force-pushed the feature/linear_cps_increase branch 5 times, most recently from b3282a1 to 98c10e3 Compare October 19, 2014 02:57
@bklang
Copy link
Member Author

bklang commented Mar 13, 2015

@benlangfeld

@benlangfeld benlangfeld force-pushed the feature/linear_cps_increase branch 2 times, most recently from e3687b8 to 27f6034 Compare March 17, 2015 14:44
@benlangfeld benlangfeld force-pushed the feature/linear_cps_increase branch from 27f6034 to 64d23ae Compare March 17, 2015 22:49
@benlangfeld
Copy link
Member

So SippyCup is now released with our usage of the SIPp API for this, and SIPp has merged it. We're now simply waiting for a new SIPp package to be released at https://launchpad.net/~taisph/+archive/ubuntu/sipp; I've contacted the PPA maintainer about this to check it will happen.

We're also waiting on #31 so that this PR can introduce changes necessary to ensure the correct version of the SIPp package gets installed.

Tick tock....

@benlangfeld benlangfeld force-pushed the feature/linear_cps_increase branch from 64d23ae to f367a10 Compare March 18, 2015 14:00
benlangfeld added a commit that referenced this pull request Mar 18, 2015
@benlangfeld benlangfeld merged commit 48b61cb into develop Mar 18, 2015
@benlangfeld benlangfeld deleted the feature/linear_cps_increase branch March 18, 2015 14: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.

4 participants