Merge lp:~doanac/charms/precise/ubuntu-ci-services-itself/lander-image-builder-start into lp:~canonical-ci-engineering/charms/precise/ubuntu-ci-services-itself/lander

Proposed by Andy Doan
Status: Merged
Approved by: Francis Ginther
Approved revision: 30
Merged at revision: 28
Proposed branch: lp:~doanac/charms/precise/ubuntu-ci-services-itself/lander-image-builder-start
Merge into: lp:~canonical-ci-engineering/charms/precise/ubuntu-ci-services-itself/lander
Diff against target: 43 lines (+19/-0)
1 file modified
templates/jobs/lander_master.xml (+19/-0)
To merge this branch: bzr merge lp:~doanac/charms/precise/ubuntu-ci-services-itself/lander-image-builder-start
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Review via email: mp+202759@code.launchpad.net

Description of the change

This adds relations for the image builder and provides some scaffolding so it actually builds something.

To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

I'm not too familiar with what lander_merge_parameters does, but if this is just supposed to be some sane values that should work as a quick test, that seems fine to me. Does it need to wait and see the end status for it? Does it need to do cleanup or anything? Every time you run this, you'll get a new copy of the image out in glance. I ask, because at least my personal account seems to have some limitations on the space, I got an error today that it was out of space.

Revision history for this message
Andy Doan (doanac) wrote :

On 01/27/2014 09:04 AM, Paul Larson wrote:
> I'm not too familiar with what lander_merge_parameters does, but if
> this is just supposed to be some sane values that should work as a
> quick test, that seems fine to me.

that was my idea. once the bsbuilder starts creating something real,
we'll be able to back out the hard-coded portion.

> Does it need to wait and see the
> end status for it?

yeah - we have to make the jenkins job listen to the progress-trigger so
it will know what to pass downstream to the next job.

> Does it need to do cleanup or anything? Every time
> you run this, you'll get a new copy of the image out in glance. I
> ask, because at least my personal account seems to have some
> limitations on the space, I got an error today that it was out of
> space.

i'm guessing we'll need to clean up these objects every so often during
development

30. By Andy Doan

add ticket-system information into params

Revision history for this message
Francis Ginther (fginther) wrote :

approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'templates/jobs/lander_master.xml'
2--- templates/jobs/lander_master.xml 2014-01-15 18:27:36 +0000
3+++ templates/jobs/lander_master.xml 2014-01-27 21:53:47 +0000
4@@ -33,6 +33,8 @@
5
6 ppa_ip=$(grep ppa-django /srv/lander_jenkins/juju-relations/* | awk '{print $NF}' | head -n1)
7 bsb_ip=$(grep bsb-restish /srv/lander_jenkins/juju-relations/* | awk '{print $NF}' | head -n1)
8+img_ip=$(grep imagebuild-restish /srv/lander_jenkins/juju-relations/* | awk '{print $NF}' | head -n1)
9+ts_ip=$(grep ts-django /srv/lander_jenkins/juju-relations/* | awk '{print $NF}' | head -n1)
10
11 # Convert to input file
12 request_parameters=$(echo $request_parameters | sed -e 's/^"//' -e 's/"$//')
13@@ -42,6 +44,16 @@
14 "progress_trigger": "${JOB_NAME}-${BUILD_NUMBER}",
15 "ppa_assigner_url": "http://${ppa_ip}:8080",
16 "bsb_url": "http://${bsb_ip}:8080",
17+ "imgbuilder_url": "http://${img_ip}:8080",
18+ "ts_url": "http://${ts_ip}:8080",
19+ "package_list": ["phablet-tools"],
20+ "base_image": {
21+ "image_type": "cloud",
22+ "url_list": [
23+ "http://cloud-images.ubuntu.com/trusty/current/trusty-server-cloudimg-amd64-disk1.img"
24+ ],
25+ "series": "trusty"
26+ },
27 "request_parameters": $request_parameters
28 }
29 EOF
30@@ -153,6 +165,13 @@
31 child_number=$TRIGGERED_BUILD_NUMBER_lander_branch_source_builder
32 wget -O params.json "${JENKINS_URL}job/${LAST_TRIGGERED_JOB_NAME}/${child_number}/artifact/results/params.json"
33
34+# TODO - temporary until the bsbuilder works properly
35+cat > params.json <<EOF
36+{
37+ "ppa": "ppa:phablet-team/tools"
38+}
39+EOF
40+
41 /srv/lander_jenkins/lander/bin/lander_merge_parameters.py --result-file params.json --service bsbuilder --output-file all.json --prior-file all.json
42
43 # Convert to a format that can be passed to the child jobs

Subscribers

People subscribed via source and target branches