Merge lp:~jjo/ubuntu-api-website/add-api-website-relations into lp:~mhall119/ubuntu-api-website/canonical-is-charm

Proposed by JuanJo Ciarlante
Status: Merged
Approved by: Michael Hall
Approved revision: no longer in the source branch.
Merged at revision: 5
Proposed branch: lp:~jjo/ubuntu-api-website/add-api-website-relations
Merge into: lp:~mhall119/ubuntu-api-website/canonical-is-charm
Diff against target: 15 lines (+9/-0)
1 file modified
hooks/api-website-relation-joined (+9/-0)
To merge this branch: bzr merge lp:~jjo/ubuntu-api-website/add-api-website-relations
Reviewer Review Type Date Requested Status
Michael Hall Approve
Review via email: mp+187873@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Michael Hall (mhall119) :
review: Approve
5. By JuanJo Ciarlante

[jjo, r=mhall119] Add a basic api-website-relation-{joined,changed} hook, need as per provides (for reverseproxy-like FE services in-front) - requires setting {hostname,port} properly

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added symlink 'hooks/api-website-relation-changed'
2=== target is u'api-website-relation-joined'
3=== added file 'hooks/api-website-relation-joined'
4--- hooks/api-website-relation-joined 1970-01-01 00:00:00 +0000
5+++ hooks/api-website-relation-joined 2013-09-26 18:33:53 +0000
6@@ -0,0 +1,9 @@
7+#!/bin/bash
8+set -x
9+. hooks/common.sh
10+RID=$(relation-ids reverseproxy)
11+if [[ -n ${RID} ]]; then
12+ OPTION="-r $RID"
13+fi
14+relation-set $OPTION port="${LISTEN_PORT?}"
15+relation-set $OPTION hostname="$(unit-get private-address)"

Subscribers

People subscribed via source and target branches

to all changes: