launchpad-reviewers team mailing list archive

[Raphaël Badin <raphael.badin@xxxxxxxxxxxxx>]

Raphaël Badin has proposed merging lp:~rvb/maas/packaging.fix-longpoll into lp:~maas-maintainers/maas/packaging.

Commit message:
Enable the proxy_http module that is required by longpoll.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #1064960 in maas (Ubuntu): "Longpoll is broken (requests to /MAAS/longpoll return 404)."
  https://bugs.launchpad.net/ubuntu/+source/maas/+bug/1064960

For more details, see:
https://code.launchpad.net/~rvb/maas/packaging.fix-longpoll/+merge/128904

Enable the proxy_http module that is required by longpoll.

That module is shipped with apache:
$ dpkg -S /etc/apache2/mods-available/proxy_http.load
apache2.2-common: /etc/apache2/mods-available/proxy_http.load

I've tested that this fixes bug 1064960.
-- 
https://code.launchpad.net/~rvb/maas/packaging.fix-longpoll/+merge/128904
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~rvb/maas/packaging.fix-longpoll into lp:~maas-maintainers/maas/packaging.

=== modified file 'debian/changelog'
--- debian/changelog	2012-10-09 16:19:24 +0000
+++ debian/changelog	2012-10-10 11:07:20 +0000
@@ -4,7 +4,10 @@
   * debian/patches/99-temporary-fix-constraints.patch: Dropped.
   * debian/control: ${python:Depends} for maas-common.
 
- -- Andres Rodriguez <andreserl@xxxxxxxxxx>  Mon, 08 Oct 2012 17:02:56 -0400
+  [ Raphaël Badin ]
+  * debian/maas-region-controller.postinst: Enable proxy_http module.
+
+  -- Andres Rodriguez <andreserl@xxxxxxxxxx>  Mon, 08 Oct 2012 17:02:56 -0400
 
 maas (0.1+bzr1223+dfsg-0ubuntu1) quantal; urgency=low
 

=== modified file 'debian/maas-region-controller.postinst'
--- debian/maas-region-controller.postinst	2012-10-05 04:26:58 +0000
+++ debian/maas-region-controller.postinst	2012-10-10 11:07:20 +0000
@@ -177,6 +177,7 @@
 	fi
 
 	# enable apache modules needed
+	a2enmod proxy_http
 	a2enmod expires
 	a2enmod wsgi