Merge lp:~caio1982/charms/trusty/jenkins/trunk into lp:charms/trusty/jenkins

[Caio Begotti]

This is a fix for https://bugs.launchpad.net/capomastro/+bug/1387601

The idea is having the necessary variables set in the relation so services which know how to talk to Jenkins via its API can easily do so without changing the charm by branching it locally. The change is in fact quite small but helpful.

We've been using this API relation in lp:capomastro for some time now, it's a major charm component for our deployments of build services at Canonical and we believe this can benefit more users :-)

[Review Queue (review-queue)]

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-10468-results

review: Needs Fixing (automated testing)

[Caio Begotti (caio1982)]

Hey hello there Marco! I'm not sure I understand how to interpret these
test results. I mean, I haven't touch the master relation so I wonder why
it failed? Also, we don't use Jenkins on Precise in the PES team, so I
can't vouch for this test. What exactly I should do?

Cheers!

On Fri, Nov 28, 2014 at 2:32 PM, Review Queue <
<email address hidden>> wrote:

> Review: Needs Fixing automated testing
>
> This items has failed automated testing! Results available here
> http://reports.vapour.ws/charm-tests/charm-bundle-test-10468-results
> --
>
> https://code.launchpad.net/~caio1982/charms/trusty/jenkins/trunk/+merge/242803
> You are the owner of lp:~caio1982/charms/trusty/jenkins/trunk.
>

[Review Queue (review-queue)]

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-10852-results

review: Needs Fixing (automated testing)

[Marco Ceppi (marcoceppi)]

LGTM, +1

review: Approve

=== added file 'hooks/api-relation-changed'
--- hooks/api-relation-changed	1970-01-01 00:00:00 +0000
+++ hooks/api-relation-changed	2014-11-25 16:16:16 +0000
@@ -0,0 +1,9 @@
+#!/bin/bash
+
+set -ue
+ 
+juju-log "Updating API interface with up-to-date Jenkins information."
+ 
+relation-set admin_username="$(config-get username)"               \
+             admin_password="$(config-get password)"               \
+             jenkins_url="http://$(unit-get private-address):8080" \

=== modified file 'metadata.yaml'
--- metadata.yaml	2014-01-24 21:48:52 +0000
+++ metadata.yaml	2014-11-25 16:16:16 +0000
@@ -21,6 +21,9 @@
     interface: jenkins-extension
     scope: container
     optional: true
+  api:
+    interface: jenkins-api
+    optional: true
 requires:
   master:
     interface: jenkins-slave