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

openstack: throttle job scheduling by default #618

Merged
merged 3 commits into from Sep 8, 2015
Merged

openstack: throttle job scheduling by default #618

merged 3 commits into from Sep 8, 2015

Conversation

ghost
Copy link

@ghost ghost commented Sep 7, 2015

The test failed because it looked for upload= instead of clone=

Signed-off-by: Loic Dachary <ldachary@dachary.org>
@ghost ghost assigned zmc Sep 7, 2015
@@ -618,6 +624,9 @@ def schedule_suite(job_config,
verbose=verbose,
log_prefix=log_prefix,
)
if throttle:
log.info("pause between jobs : --throttle " + str(throttle))
sleep(int(throttle))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably no-op if --dry-run is passed

When scheduling, wait SLEEP seconds between jobs.  Useful to avoid
bursts that may be too hard on the underlying infrastructure or exceed
OpenStack API limits (server creation per minute for instance).

http://tracker.ceph.com/issues/12977 Refs: #12977

Signed-off-by: Loic Dachary <loic@dachary.org>
When running a suite with 50 jobs, it will schedule about 100 instance
creation within less than a minute. It is likely to exceed the API
quotas of the OpenStack provider (number of instance creation per
minute) and lead to instance creation failures.

Set the teuthology-suite to be called with --throttle 15 by default,
that is about 60 / 15 = 4 * 2 = ~8 server creation per minute.

http://tracker.ceph.com/issues/12977 Fixes: #12977

Signed-off-by: Loic Dachary <loic@dachary.org>
@ghost
Copy link
Author

ghost commented Sep 8, 2015

@zmc updated to not wait if dry_run, thanks for catching this :-)

@ceph-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1844/
Test PASSed.

zmc added a commit that referenced this pull request Sep 8, 2015
openstack: throttle job scheduling by default
@zmc zmc merged commit f9a1cb4 into ceph:master Sep 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants