Merge lp:~allanlesage/uci-engine/coverage-extractor-plus-nfss-juju-deployment into lp:uci-engine

[Allan LeSage]

Deploy coverage-extractor against NFSS.

Initial stab at juju deployment of coverage-extractor, linking to NFSS.

[PS Jenkins bot (ps-jenkins)]

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

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

review: Approve (continuous-integration)

[lp:~allanlesage/uci-engine/coverage-extractor-plus-nfss-juju-deployment]

860. By Allan LeSage on 2014-10-22

Remove NFSS deployment; we're not using juju relations to communicat with.

[PS Jenkins bot (ps-jenkins)]

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

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

review: Approve (continuous-integration)

[Francis Ginther (fginther)]

Comments inline.

review: Needs Fixing

[lp:~allanlesage/uci-engine/coverage-extractor-plus-nfss-juju-deployment]

861. By Allan LeSage on 2014-10-22

Coverage-extraction deployment details amend for fginther's comments.

[Allan LeSage (allanlesage)]

Updated, thanks!

[Francis Ginther (fginther)]

The indentation is off and needs to be fixed. While that's going on, I can do a test upgrade of my current deployment.

review: Needs Fixing

[lp:~allanlesage/uci-engine/coverage-extractor-plus-nfss-juju-deployment]

862. By Allan LeSage on 2014-10-23

Correct tabination.

[PS Jenkins bot (ps-jenkins)]

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

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

review: Approve (continuous-integration)

[Francis Ginther (fginther)]

As discussed face to face, I think we should change the deployment name in this file from 'ci-airline' to 'ci-engine-coverage' (or something similar). This change will remove this component from the default set deployed with the other engine components and give us the opportunity to land this changes and continue to iterate toward the right solution for the nfss_insert problem.

review: Needs Fixing

[lp:~allanlesage/uci-engine/coverage-extractor-plus-nfss-juju-deployment]

863. By Allan LeSage on 2014-10-24

Adjusting deployment name to ci-engine-coverage per fginther's advice.

[PS Jenkins bot (ps-jenkins)]

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

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

review: Approve (continuous-integration)

[Francis Ginther (fginther)]

This solves the immediate problem we saw earlier of not being able to deploy the other ci-engine components. And this allows us to land the deployer file so that it is visible to others and we can consider it in any future deployment changes. The deployment still needs work, but it is not going to block any other work.

review: Approve

[Allan LeSage (allanlesage)]

Top-approved given a visual across-the-table thumbs-up from Francis.

Unmerged revisions

863. By Allan LeSage on 2014-10-24

Adjusting deployment name to ci-engine-coverage per fginther's advice.

862. By Allan LeSage on 2014-10-23

Correct tabination.

861. By Allan LeSage on 2014-10-22

Coverage-extraction deployment details amend for fginther's comments.

860. By Allan LeSage on 2014-10-22

Remove NFSS deployment; we're not using juju relations to communicat with.

859. By Allan LeSage on 2014-10-21

Add requests-oauthlib, add quotation marks to coverage-extractor deployment.

858. By Allan LeSage on 2014-10-20

Initial stab at a juju deployment of coverage-extractor with NFSS.

857. By Allan LeSage on 2014-10-18

Merge trunk.

856. By Allan LeSage on 2014-10-18

Extract line and branch coverage from a coverage.xml artifact in swift, post to NFSS.

=== added file 'juju-deployer/coverage-extractor.yaml.tmpl'
--- juju-deployer/coverage-extractor.yaml.tmpl	1970-01-01 00:00:00 +0000
+++ juju-deployer/coverage-extractor.yaml.tmpl	2014-10-24 13:06:51 +0000
@@ -0,0 +1,31 @@
+ci-engine-coverage:
+    services:
+        ci-airline-coverage-extractor-worker:
+            series: trusty
+            charm: rabbitmq-worker
+            constraints: "mem=1024M"
+            options:
+                main: ./run-python ./coverage-extractor/coverageextractor/run_worker.py
+                current_code: ${CI_PAYLOAD_URL}
+                available_code: ${CI_PAYLOAD_URL}
+                unit-config: include-base64://configs/unit_config.yaml
+                packages: "python-swiftclient python-requests-oauthlib"
+                install_sources: |
+                    - ${CI_PPA}
+                    - "cloud:precise-icehouse"
+                install_keys: |
+                    - null
+                    - null
+                nagios_context: ci-airline-staging
+                nagios_check_http_params: -H coverageextractor:8080 -I 127.0.0.1 -e '200 OK' --url='/api/v1/'
+        ci-airline-rabbit:
+            series: precise
+            branch: lp:~canonical-ci-engineering/charms/precise/uci-engine/rabbitmq-server-trunk-transition
+            charm: rabbitmq
+            constraints: "mem=1024M"
+            num_units: 1
+            options:
+                management_plugin: true
+                source: cloud:precise-icehouse
+    relations:
+        - ["ci-airline-coverage-extractor-worker", "ci-airline-rabbit"]