Merge lp:~psivaa/ubuntu-ci-services-itself/speedup-imagebuild-tr into lp:ubuntu-ci-services-itself

Proposed by Para Siva
Status: Merged
Approved by: Chris Johnston
Approved revision: 377
Merged at revision: 377
Proposed branch: lp:~psivaa/ubuntu-ci-services-itself/speedup-imagebuild-tr
Merge into: lp:ubuntu-ci-services-itself
Diff against target: 22 lines (+2/-2)
2 files modified
juju-deployer/configs/unit_config.yaml.tmpl (+1/-1)
juju-deployer/image-builder.yaml.tmpl (+1/-1)
To merge this branch: bzr merge lp:~psivaa/ubuntu-ci-services-itself/speedup-imagebuild-tr
Reviewer Review Type Date Requested Status
Chris Johnston (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+210785@code.launchpad.net

Commit message

Based on the consecutive runs given in https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AvOb-Wd__AD8dEhkeGRWUzBnSmZ5SWhOT0RlQ0tyWmc&usp=drive_web#gid=0

In short, Increasing imagebuilder RAM to 4G and making test-runner testbed to use standard.medium improves the timing considerably for bug 1290716.

Note: Increasing the RAM further to 8G did not yield a lot of timing improvement for imagebuilder and hence stopped at 4G.

Description of the change

Based on the consecutive runs given in https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AvOb-Wd__AD8dEhkeGRWUzBnSmZ5SWhOT0RlQ0tyWmc&usp=drive_web#gid=0

In short, Increasing imagebuilder RAM to 4G and making test-runner testbed to use standard.medium improves the timing considerably for bug 1290716.

Note: Increasing the RAM further to 8G did not yield a lot of timing improvement for imagebuilder and hence stopped at 4G.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:377
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/401/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/401/rebuild

review: Approve (continuous-integration)
Revision history for this message
Chris Johnston (cjohnston) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'juju-deployer/configs/unit_config.yaml.tmpl'
2--- juju-deployer/configs/unit_config.yaml.tmpl 2014-03-12 01:35:01 +0000
3+++ juju-deployer/configs/unit_config.yaml.tmpl 2014-03-13 12:44:20 +0000
4@@ -23,4 +23,4 @@
5 master_ppa: $CI_MASTER_PPA
6 tr_flavors:
7 - m1.small
8- - standard.small
9+ - standard.medium
10
11=== modified file 'juju-deployer/image-builder.yaml.tmpl'
12--- juju-deployer/image-builder.yaml.tmpl 2014-03-10 22:25:00 +0000
13+++ juju-deployer/image-builder.yaml.tmpl 2014-03-13 12:44:20 +0000
14@@ -22,7 +22,7 @@
15 wsgi_wsgi_file: imagebuilder.wsgi:app
16 imagebuild-worker:
17 charm: rabbitmq-worker
18- constraints: "root-disk=8192M"
19+ constraints: "mem=4096M root-disk=8192M"
20 options:
21 main: ./run-python ./image-builder/imagebuilder/run_worker.py
22 vcs: ${CI_CODE_SOURCE}

Subscribers

People subscribed via source and target branches