Merge ~andreserl/maas:disable_nginx_on_install into maas:master

[Andres Rodriguez]

debian/maas-rack-controller.postinst,postrm: Disable/enable nginx

MAAS now depends on nginx. Since the package automatically binds to port 80, MAAS enables/disables it on install/remove accordingly.

[Andres Rodriguez (andreserl)]

selfie!

review: Approve

diff --git a/debian/maas-rack-controller.postinst b/debian/maas-rack-controller.postinst
index bec6b65..1b0b4b5 100644
--- a/debian/maas-rack-controller.postinst
+++ b/debian/maas-rack-controller.postinst
@@ -147,11 +147,17 @@ fix_dns_permissions() {
     fi
 }
 
+disable_nginx() {
+    systemctl stop nginx >/dev/null 2>&1 || true
+    systemctl disable nginx >/dev/null 2>&1 || true
+}
+
 # Unconditionally ensure that there is at least an empty configuration
 # file. This does *not* overwrite any existing configuration.
 maas-rack config
 
 if [ "$1" = "configure" ] && [ -z "$2" ]; then
+    disable_nginx
     configure_logging
     configure_libdir
     configure_maas_url
@@ -178,6 +184,7 @@ elif [ -n "$DEBCONF_RECONFIGURE" ]; then
     configure_shared_secret
 
 elif [ "$1" = "configure" ] && dpkg --compare-versions "$2" gt 0.1+bzr266+dfsg-0ubuntu1; then
+    disable_nginx
     configure_logging
     configure_libdir
     maas-rack upgrade-cluster
diff --git a/debian/maas-rack-controller.postrm b/debian/maas-rack-controller.postrm
index 6a73639..aa28f1a 100644
--- a/debian/maas-rack-controller.postrm
+++ b/debian/maas-rack-controller.postrm
@@ -48,3 +48,9 @@ esac
 if [ -L /etc/tgt/conf.d/maas.conf ]; then
     rm -rf /etc/tgt/conf.d/maas.conf
 fi
+
+# MAAS installation disables system nginx so it doesn't
+# prevent MAAS from running our own nginx daemon. As such,
+# on removal, reenable nginx.
+systemctl enable nginx >/dev/null 2>&1 || true
+systemctl start nginx >/dev/null 2>&1 || true