Merge lp:~pavel-pachkovskij/charms/precise/redis-master/redis-charms-fix-lxc-support into lp:charms/redis-master

Proposed by Pavel Pachkovskij
Status: Merged
Approved by: Mark Mims
Approved revision: 5
Merge reported by: Mark Mims
Merged at revision: not available
Proposed branch: lp:~pavel-pachkovskij/charms/precise/redis-master/redis-charms-fix-lxc-support
Merge into: lp:charms/redis-master
Diff against target: 38 lines (+3/-3)
3 files modified
hooks/install (+1/-1)
hooks/redis-admin-relation-joined (+1/-1)
hooks/redis-master-relation-joined (+1/-1)
To merge this branch: bzr merge lp:~pavel-pachkovskij/charms/precise/redis-master/redis-charms-fix-lxc-support
Reviewer Review Type Date Requested Status
Mark Mims (community) Approve
Review via email: mp+147335@code.launchpad.net

Description of the change

Wasn't discoverable in local environment due to
MY_HOSTNAME=`hostname -f`

To post a comment you must log in.
Revision history for this message
Mark Mims (mark-mims) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'hooks/install'
--- hooks/install 2013-02-06 18:14:14 +0000
+++ hooks/install 2013-02-08 10:04:30 +0000
@@ -16,7 +16,7 @@
16# Set some variables16# Set some variables
17############################################################################################################17############################################################################################################
1818
19MY_HOSTNAME=`hostname -f`19MY_HOSTNAME=`unit-get public-address`
2020
21############################################################################################################21############################################################################################################
22# Modify the redis-server configuration file22# Modify the redis-server configuration file
2323
=== modified file 'hooks/redis-admin-relation-joined'
--- hooks/redis-admin-relation-joined 2011-10-13 08:12:21 +0000
+++ hooks/redis-admin-relation-joined 2013-02-08 10:04:30 +0000
@@ -10,7 +10,7 @@
10# Set global variables10# Set global variables
11############################################################################################################11############################################################################################################
1212
13MY_HOSTNAME=`hostname -f`13MY_HOSTNAME=`unit-get public-address`
1414
15############################################################################################################15############################################################################################################
16# Set the hostname in the relation so it can be picked up by the _other_ side16# Set the hostname in the relation so it can be picked up by the _other_ side
1717
=== modified file 'hooks/redis-master-relation-joined'
--- hooks/redis-master-relation-joined 2011-10-13 08:12:21 +0000
+++ hooks/redis-master-relation-joined 2013-02-08 10:04:30 +0000
@@ -10,7 +10,7 @@
10# Set global variables10# Set global variables
11############################################################################################################11############################################################################################################
1212
13MY_HOSTNAME=`hostname -f`13MY_HOSTNAME=`unit-get public-address`
14MY_PORT="6379"14MY_PORT="6379"
1515
16############################################################################################################16############################################################################################################

Subscribers

People subscribed via source and target branches

to all changes: