Merge lp:~caio1982/charms/trusty/jenkins/trunk into lp:charms/trusty/jenkins
Proposed by
Caio Begotti
Status: | Merged |
---|---|
Merged at revision: | 33 |
Proposed branch: | lp:~caio1982/charms/trusty/jenkins/trunk |
Merge into: | lp:charms/trusty/jenkins |
Diff against target: |
27 lines (+12/-0) 2 files modified
hooks/api-relation-changed (+9/-0) metadata.yaml (+3/-0) |
To merge this branch: | bzr merge lp:~caio1982/charms/trusty/jenkins/trunk |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Marco Ceppi (community) | Approve | ||
Review Queue (community) | automated testing | Needs Fixing | |
Review via email: mp+242803@code.launchpad.net |
Description of the change
This is a fix for https:/
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 :-)
To post a comment you must log in.
This items has failed automated testing! Results available here http:// reports. vapour. ws/charm- tests/charm- bundle- test-10468- results