Merge ~ines-almeida/launchpad:update-ppa-charms-configuration into launchpad:master

Proposed by Ines Almeida
Status: Merged
Approved by: Ines Almeida
Approved revision: 41a749bcd8042cf4b37bd63916d949e7cb29d348
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~ines-almeida/launchpad:update-ppa-charms-configuration
Merge into: launchpad:master
Diff against target: 43 lines (+4/-4)
4 files modified
charm/launchpad-ppa-publisher/reactive/launchpad-ppa-publisher.py (+1/-1)
charm/launchpad-ppa-publisher/templates/vhosts/ppa.conf.j2 (+1/-1)
charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.buildd.conf.j2 (+1/-1)
charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.conf.j2 (+1/-1)
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+450146@code.launchpad.net

Commit message

Update the PPA publisher apache configuration to use port 80 instead of 8080

This highly simplifies the setup needed for the load balancer (HAProxy) without making much of a difference otherwise.

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/charm/launchpad-ppa-publisher/reactive/launchpad-ppa-publisher.py b/charm/launchpad-ppa-publisher/reactive/launchpad-ppa-publisher.py
index da60fee..89a4445 100644
--- a/charm/launchpad-ppa-publisher/reactive/launchpad-ppa-publisher.py
+++ b/charm/launchpad-ppa-publisher/reactive/launchpad-ppa-publisher.py
@@ -161,7 +161,7 @@ def configure_apache_website():
161 apache_website.set_remote(161 apache_website.set_remote(
162 domain=domain,162 domain=domain,
163 enabled="true",163 enabled="true",
164 ports="8080",164 ports="80",
165 site_config="\n".join(site_configs),165 site_config="\n".join(site_configs),
166 site_modules="expires headers rewrite wsgi",166 site_modules="expires headers rewrite wsgi",
167 )167 )
diff --git a/charm/launchpad-ppa-publisher/templates/vhosts/ppa.conf.j2 b/charm/launchpad-ppa-publisher/templates/vhosts/ppa.conf.j2
index 4f91629..d6c8e42 100644
--- a/charm/launchpad-ppa-publisher/templates/vhosts/ppa.conf.j2
+++ b/charm/launchpad-ppa-publisher/templates/vhosts/ppa.conf.j2
@@ -1,4 +1,4 @@
1<VirtualHost *:8080>1<VirtualHost *:80>
2 ServerName {{ domain_ppa }}2 ServerName {{ domain_ppa }}
3{%- if domain_alternate %}3{%- if domain_alternate %}
4 ServerAlias {{ domain_ppa_alternate }}4 ServerAlias {{ domain_ppa_alternate }}
diff --git a/charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.buildd.conf.j2 b/charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.buildd.conf.j2
index 90df926..7c6518d 100644
--- a/charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.buildd.conf.j2
+++ b/charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.buildd.conf.j2
@@ -1,4 +1,4 @@
1<VirtualHost *:8080>1<VirtualHost *:80>
2 #NOWEBSTATS2 #NOWEBSTATS
3 ServerName {{ domain_ppa_buildd }}3 ServerName {{ domain_ppa_buildd }}
4 DocumentRoot {{ ppa_archive_private_root }}4 DocumentRoot {{ ppa_archive_private_root }}
diff --git a/charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.conf.j2 b/charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.conf.j2
index a3f59f5..9009aed 100644
--- a/charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.conf.j2
+++ b/charm/launchpad-ppa-publisher/templates/vhosts/private-ppa.conf.j2
@@ -1,4 +1,4 @@
1<VirtualHost *:8080>1<VirtualHost *:80>
2 ServerName {{ domain_ppa_private }}2 ServerName {{ domain_ppa_private }}
3{%- if domain_alternate %}3{%- if domain_alternate %}
4 ServerAlias {{ domain_ppa_private_alternate }}4 ServerAlias {{ domain_ppa_private_alternate }}

Subscribers

People subscribed via source and target branches

to status/vote changes: