Merge lp:~flepied/charms/precise/mongodb/trunk into lp:charms/mongodb

Proposed by Frederic Lepied
Status: Merged
Approved by: Clint Byrum
Approved revision: 18
Merged at revision: 18
Proposed branch: lp:~flepied/charms/precise/mongodb/trunk
Merge into: lp:charms/mongodb
Diff against target: 23 lines (+4/-4)
1 file modified
hooks/install (+4/-4)
To merge this branch: bzr merge lp:~flepied/charms/precise/mongodb/trunk
Reviewer Review Type Date Requested Status
Juan L. Negron (community) Approve
Review via email: mp+104058@code.launchpad.net

Description of the change

fix hardcoded DEFAULT_PORT in install hook

To post a comment you must log in.
Revision history for this message
Juan L. Negron (negronjl) wrote :

Thanks for the updates Fred.

Approved.

-Juan

review: Approve
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Pushed into precise, this should also be pushed to oneiric but this bug is blocking that:

https://bugs.launchpad.net/charms/+bug/991980

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'hooks/install'
--- hooks/install 2012-01-27 17:32:55 +0000
+++ hooks/install 2012-04-30 06:10:22 +0000
@@ -39,7 +39,7 @@
39# set. In our case, we have defaulted to "myset".39# set. In our case, we have defaulted to "myset".
40############################################################################################################40############################################################################################################
41# Web Admin UI41# Web Admin UI
42if [ "$WEB_ADMIN_UI" == "True" ]; then42if [ "$WEB_ADMIN_UI" == "yes" ]; then
43 sed -i -e "s/ -- / -- --rest /" /etc/init/mongodb.conf43 sed -i -e "s/ -- / -- --rest /" /etc/init/mongodb.conf
44fi44fi
4545
@@ -75,7 +75,7 @@
75############################################################################################################75############################################################################################################
76# Register the port76# Register the port
77############################################################################################################77############################################################################################################
78[ -x /usr/bin/open-port ] && open-port 27017/TCP78[ -x /usr/bin/open-port ] && open-port ${DEFAULT_PORT}/TCP
79if [ "$WEB_ADMIN_UI" == "True" ]; then79if [ "$WEB_ADMIN_UI" == "yes" ]; then
80 [ -x /usr/bin/open-port ] && open-port 28017/TCP80 [ -x /usr/bin/open-port ] && open-port $((${DEFAULT_PORT} + 1000))/TCP
81fi81fi

Subscribers

People subscribed via source and target branches