Merge lp:~billy-olsen/maas/lp1484696-proxypass-websockets-1.8 into lp:maas/1.8

Proposed by Billy Olsen on 2015-12-22
Status: Merged
Approved by: Mike Pontillo on 2015-12-22
Approved revision: 4058
Merged at revision: 4058
Proposed branch: lp:~billy-olsen/maas/lp1484696-proxypass-websockets-1.8
Merge into: lp:maas/1.8
Diff against target: 12 lines (+1/-1)
1 file modified
contrib/maas-http.conf (+1/-1)
To merge this branch: bzr merge lp:~billy-olsen/maas/lp1484696-proxypass-websockets-1.8
Reviewer Review Type Date Requested Status
Mike Pontillo (community) 2015-12-22 Approve on 2015-12-22
Review via email:

Commit message

Specify the disablereuse=on for the websockets ProxyPass configuration to workaround upstream bug

Description of the change

Specify the disablereuse=on for the websockets ProxyPass configuration to workaround upstream bug

To post a comment you must log in.
Mike Pontillo (mpontillo) wrote :

Looks good. Thanks for the fixes! I'll discuss when to land this fix at the next team meeting.

review: Approve
Mike Pontillo (mpontillo) wrote :

Landing this for 1.8.4.

Holding off on the 1.9 branch until 1.9.0 is out.

MAAS Lander (maas-lander) wrote :
Download full text (83.7 KiB)

The attempt to merge lp:~billy-olsen/maas/lp1484696-proxypass-websockets-1.8 into lp:maas/1.8 failed. Below is the output from the failed tests.

Hit trusty-security InRelease
Ign trusty InRelease
Hit trusty-updates InRelease
Hit trusty-backports InRelease
Hit trusty Release.gpg
Hit trusty Release
Hit trusty-security/main Sources
Hit trusty-security/universe Sources
Hit trusty-security/main amd64 Packages
Hit trusty-security/universe amd64 Packages
Hit trusty-security/main Translation-en
Hit trusty-security/universe Translation-en
Hit trusty-updates/main Sources
Hit trusty-updates/restricted Sources
Hit trusty-updates/universe Sources
Hit trusty-updates/multiverse Sources
Hit trusty-updates/main amd64 Packages
Hit trusty-updates/restricted amd64 Packages
Hit trusty-updates/universe amd64 Packages
Hit trusty-updates/multiverse amd64 Packages
Hit trusty-updates/main Translation-en
Hit trusty-updates/multiverse Translation-en
Hit trusty-updates/restricted Translation-en
Hit trusty-updates/universe Translation-en
Hit trusty-backports/main Sources
Hit trusty-backports/restricted Sources
Hit trusty-backports/universe Sources
Hit trusty-backports/multiverse Sources
Hit trusty-backports/main amd64 Packages
Hit trusty-backports/restricted amd64 Packages
Hit trusty-backports/universe amd64 Packages
Hit trusty-backports/multiverse amd64 Packages
Hit trusty-backports/main Translation-en
Hit trusty-backports/multiverse Translation-en
Hit trusty-backports/restricted Translation-en
Hit trusty-backports/universe Translation-en
Hit http://prodstack-zon...

Mike Pontillo (mpontillo) wrote :

Re-trying. Random test failure; couldn't be related to this branch...

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'contrib/maas-http.conf'
2--- contrib/maas-http.conf 2015-06-26 14:25:46 +0000
3+++ contrib/maas-http.conf 2015-12-22 06:12:06 +0000
4@@ -26,7 +26,7 @@
6 <IfModule proxy_module>
7 ProxyPreserveHost on
8- ProxyPass /MAAS/ws "ws://localhost:5240/MAAS/ws"
9+ ProxyPass /MAAS/ws "ws://localhost:5240/MAAS/ws" disablereuse=on
10 ProxyPass /MAAS/static/ !
11 ProxyPass /MAAS/ http://localhost:5240/MAAS/
12 ProxyPass /MAAS http://localhost:5240/MAAS/


People subscribed via source and target branches

to all changes: