Merge lp:~allanlesage/uci-engine/coverage-retriever-juju-deployment into lp:uci-engine

[Allan LeSage]

Juju deployment for coverage retriever.

Initial stab at a coverage-retriever juju deployment, mostly to get some tutorializing on how to do such :) . . . . Francis feel free to pass me along if I need too much instruction.

[Francis Ginther (fginther)]

This should be enough to deploy the coverage retriever. There is additional work to be done to add relations to other parts of the ci-engine deployment, but that can be done later.

This presents a small problem in that we need to be more flexible in how components are deployed. For example, the default behavior of deploy.py is to deploy everything needed for the ci-engine deployment. Does that include the coverage components or the components for some other future add on service? Just something that needs to be addressed. I think we look for ways to solve this problem at the next face to face sprint.

review: Needs Fixing

[Allan LeSage (allanlesage)]

Thanks Francis, yes I'll be learning more about deployment as we move forward with our planned lander changes, etc.

[PS Jenkins bot (ps-jenkins)]

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

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

review: Approve (continuous-integration)

[Francis Ginther (fginther)]

This looks good. Do you know if this instance really needs mem=4096M or was this just a copy from another file? We're looking at trimming down the size of each instance in a very near MP.

review: Approve

[Francis Ginther (fginther)]

FYI, the MP to reduce the branch-source-builders memory size is:
https://code.launchpad.net/~ev/uci-engine/drop-bsb-4g-ram/+merge/238135

It should be landing today. I suspect this needs to match, but you can address in a follow up MP.

[lp:~allanlesage/uci-engine/coverage-retriever-juju-deployment]

842. By Allan LeSage on 2014-10-13

Remove 4096M mem constraint per fginther's advice.

843. By Allan LeSage on 2014-10-13

Merge trunk.

[Allan LeSage (allanlesage)]

Francis, removed that mem constraint per your rec. I'll top-approve unless you object!

[PS Jenkins bot (ps-jenkins)]

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

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

review: Approve (continuous-integration)

[Francis Ginther (fginther)]

Thanks for the additional update.

review: Approve

[Evan (ev)]

Small problem. Once that's fixed, consider this a +1

review: Needs Fixing

[lp:~allanlesage/uci-engine/coverage-retriever-juju-deployment]

844. By Allan LeSage on 2014-10-14

Restore mem constraint at 1024M, remove python-gnupg from charm.

[Allan LeSage (allanlesage)]

Top-approving having adopted ev's recs.

=== added file 'juju-deployer/coverage-retriever.yaml.tmpl'
--- juju-deployer/coverage-retriever.yaml.tmpl	1970-01-01 00:00:00 +0000
+++ juju-deployer/coverage-retriever.yaml.tmpl	2014-10-14 15:41:09 +0000
@@ -0,0 +1,29 @@
+ci-airline:
+    series: precise
+    services:
+        ci-airline-coverage-retriever-worker:
+            charm: rabbitmq-worker
+            constraints: "mem=1024M"
+            options:
+                main: ./run-python ./coverage-retriever/coverageretriever/run_worker.py
+                current_code: ${CI_PAYLOAD_URL}
+                available_code: ${CI_PAYLOAD_URL}
+                unit-config: include-base64://configs/unit_config.yaml
+                packages: "python-swiftclient"
+                install_sources: |
+                    - ${CI_PPA}
+                    - "cloud:precise-icehouse"
+                install_keys: |
+                    - null
+                    - null
+                nagios_context: ci-airline-staging
+                nagios_check_http_params: -H coverageretriever:8080 -I 127.0.0.1 -e '200 OK' --url='/api/v1/'
+        ci-airline-rabbit:
+            branch: lp:charms/precise/rabbitmq-server@55
+            charm: rabbitmq
+            num_units: 1
+            options:
+                management_plugin: true
+                source: cloud:precise-icehouse
+    relations:
+        - [ci-airline-coverage-retriever-worker, ci-airline-rabbit]