Juju Charms Collection
wordpress package

Merge lp:~jamesbeedy/charms/trusty/wordpress/apache2_trusty_fix into lp:charms/trusty/wordpress

  1. Trusty Tahr (14.04)
  2. apache2_trusty_fix
  3. Merge into trunk
Proposed by james beedy
Status: Merged
Merged at revision: 87
Proposed branch: lp:~jamesbeedy/charms/trusty/wordpress/apache2_trusty_fix
Merge into: lp:charms/trusty/wordpress
Diff against target: 44 lines (+17/-1)
2 files modified
files/charm/apache/etc_apache2_conf-d_php5-fpm.conf (+4/-0)
hooks/config-changed (+13/-1)
To merge this branch: bzr merge lp:~jamesbeedy/charms/trusty/wordpress/apache2_trusty_fix
Reviewer Review Type Date Requested Status
Review Queue (community) automated testing Approve
Pen Gale (community) Approve
Marco Ceppi Pending
charmers Pending
Review via email: mp+297720@code.launchpad.net

Description of the change

Modified to include support for > precise.

Fixes Bug #1593498

To post a comment you must log in.
lp:~jamesbeedy/charms/trusty/wordpress/apache2_trusty_fix updated
87. By james beedy

Removed unneeded sources modifications

88. By james beedy

Add closing fi

89. By james beedy

Re-add sources sed cmd and apache2-mpm-worker

Revision history for this message
Pen Gale (pengale) wrote :

The code makes sense, and the charm deploys successfully and tests pass. +1

(It looks like there might be an issue with the tests when run via bundletester -- they pass, but leave the wordpress machine in a "hook failed: cache-relation-changed" for memcached:cache" state. The precise tests in trunk do the same thing, however, so I don't believe that it is related to this code, or should block this PR from being merged.)

review: Approve
Revision history for this message
james beedy (jamesbeedy) wrote :

@petevg thanks for the review! Yes, the cache-relation-changed fails for me to ... prior to this change.

Revision history for this message
Kevin W Monroe (kwmonroe) wrote :

Hey Marco, will you pull this into your namespace (or a wp-charmers group, if such a thing exists)?

Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-aws/5015/

review: Approve (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

This item has failed automated testing! Results available here http://juju-ci.vapour.ws/job/charm-bundle-test-lxc/4812/

review: Needs Fixing (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-aws/5023/

review: Approve (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-aws/5031/

review: Approve (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

This item has failed automated testing! Results available here http://juju-ci.vapour.ws/job/charm-bundle-test-lxc/4819/

review: Needs Fixing (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-lxc/4824/

review: Approve (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-aws/5038/

review: Approve (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-lxc/4828/

review: Approve (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-aws/5042/

review: Approve (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-aws/5045/

review: Approve (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-lxc/4832/

review: Approve (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

The results (PASS) are in and available here: http://juju-ci.vapour.ws/job/charm-bundle-test-lxc/4835/

review: Approve (automated testing)
Revision history for this message
Cory Johns (johnsca) wrote :

This has been merged and released as cs:trusty/wordpress-5

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'files/charm/apache/etc_apache2_conf-d_php5-fpm.conf'
--- files/charm/apache/etc_apache2_conf-d_php5-fpm.conf 2012-11-21 20:12:24 +0000
+++ files/charm/apache/etc_apache2_conf-d_php5-fpm.conf 2016-06-17 03:22:30 +0000
@@ -3,4 +3,8 @@
3 Action php5-fcgi /php5-fcgi3 Action php5-fcgi /php5-fcgi
4 Alias /php5-fcgi /usr/lib/cgi-bin/php5-fcgi4 Alias /php5-fcgi /usr/lib/cgi-bin/php5-fcgi
5 FastCgiExternalServer /usr/lib/cgi-bin/php5-fcgi -socket /var/run/php5-fpm.sock -pass-header Authorization5 FastCgiExternalServer /usr/lib/cgi-bin/php5-fcgi -socket /var/run/php5-fpm.sock -pass-header Authorization
6
7 <Directory /usr/lib/cgi-bin>
8 Require all granted
9 </Directory>
6</IfModule>10</IfModule>
711
=== modified file 'hooks/config-changed'
--- hooks/config-changed 2014-10-29 13:10:45 +0000
+++ hooks/config-changed 2016-06-17 03:22:30 +0000
@@ -3,6 +3,7 @@
3set -ue3set -ue
44
5source inc/common5source inc/common
6source /etc/lsb-release
67
7##8##
8# This is where things can get a bit hectic. So this long blog is just to9# This is where things can get a bit hectic. So this long blog is just to
@@ -65,7 +66,18 @@
65 rm -f /etc/apache2/sites-enabled/*66 rm -f /etc/apache2/sites-enabled/*
66 a2enmod actions fastcgi alias proxy_balancer proxy_http headers67 a2enmod actions fastcgi alias proxy_balancer proxy_http headers
6768
68 install -o root -g root -m 0644 files/charm/apache/etc_apache2_conf-d_php5-fpm.conf /etc/apache2/conf.d/php5-fpm.conf69 if ["$DISTRIB_CODENAME" == "precise" ]; then
70 install -o root -g root -m 0644 files/charm/apache/etc_apache2_conf-d_php5-fpm.conf /etc/apache2/conf.d/php5-fpm.conf
71 else
72 install -o root -g root -m 0644 files/charm/apache/etc_apache2_conf-d_php5-fpm.conf /etc/apache2/conf-available/php5-fpm.conf
73
74 # Check for symlink pre-exist
75 if [ -h /etc/apache2/conf-enabled/php5-fpm.conf ]; then
76 echo "Symlink pre-exists"
77 else
78 ln -s /etc/apache2/conf-available/php5-fpm.conf /etc/apache2/conf-enabled/php5-fpm.conf
79 fi
80 fi
6981
70 juju-log "Installing Apache loadbal config..."82 juju-log "Installing Apache loadbal config..."
71 install -o root -g root -m 0644 files/charm/apache/etc_apache2_sites-enabled_loadbalancer /etc/apache2/sites-available/loadbalancer.conf83 install -o root -g root -m 0644 files/charm/apache/etc_apache2_sites-enabled_loadbalancer /etc/apache2/sites-available/loadbalancer.conf

Subscribers

People subscribed via source and target branches

to all changes: