#19: Finalizing upgrade to PHP 7.1.

docker/assets-control/boot-application.sh

@@ -13,7 +13,7 @@ sudo -u www-data php bin/console doctrine:migrations:migrate --no-interaction
 
 /etc/init.d/rsyslog start
 /usr/sbin/cron
-/etc/init.d/php5-fpm start
+/etc/init.d/php7.1-fpm start
 /etc/init.d/nginx start
 
 echo "All done."

docker/assets-monitor/boot-application.sh

@@ -18,7 +18,7 @@ JAVA_HOME=/usr/lib/jvm/java-7-openjdk-amd64 JRE_HOME=/usr/lib/jvm/java-7-openjdk
         --ttl 3600 \
     >> /var/log/browsermob-proxy.log 2>&1 &
 
-/etc/init.d/php5-fpm start
+/etc/init.d/php7.1-fpm start
 /etc/init.d/nginx start
 
 echo "All done."

puppet/hieradata/role/control-and-monitor-app-server.yaml

@@ -3,7 +3,7 @@ classes:
   - base
   - selenium
   - browsermob-proxy
-  - php5
+  - php7_1
   - composer
   - user-journeymonitor
   - nodejs

puppet/hieradata/role/control-app-container.yaml

@@ -2,7 +2,7 @@
 classes:
   - base-minimal
   - base
-  - php5
+  - php7_1
   - composer
   - user-journeymonitor
   - nodejs

puppet/hieradata/role/control-app-server.yaml

@@ -1,7 +1,7 @@
 ---
 classes:
   - base
-  - php5
+  - php7_1
   - composer
   - user-journeymonitor
   - nodejs

puppet/hieradata/role/monitor-app-container.yaml

@@ -3,7 +3,7 @@ classes:
   - base-minimal
   - selenium
   - browsermob-proxy
-  - php5
+  - php7_1
   - composer
   - user-journeymonitor
   - cronjobs

puppet/hieradata/role/monitor-app-server.yaml

@@ -3,7 +3,7 @@ classes:
   - base
   - selenium
   - browsermob-proxy
-  - php5
+  - php7_1
   - composer
   - user-journeymonitor
   - nodejs

puppet/modules/composer/manifests/init.pp

@@ -1,17 +1,17 @@
 class composer {
 
-  file { "/usr/bin/composer-1.0.0.phar":
+  file { "/usr/bin/composer-1.3.1.phar":
     owner   => "root",
     group   => "root",
     mode    => 06755,
-    source  => "puppet:///modules/composer/usr/bin/composer-1.0.0.phar",
+    source  => "puppet:///modules/composer/usr/bin/composer-1.3.1.phar",
     require => [ Class["php7_1"] ],
   }
 
   file { "/usr/bin/composer":
     ensure  => "link",
-    target  => "/usr/bin/composer-1.0.0.phar",
-    require => File["/usr/bin/composer-1.0.0.phar"],
+    target  => "/usr/bin/composer-1.3.1.phar",
+    require => File["/usr/bin/composer-1.3.1.phar"],
   }
 
 }

puppet/modules/jdk8/manifests/init.pp

@@ -5,13 +5,18 @@
     creates => "/etc/apt/sources.list.d/openjdk-r-ppa-trusty.list"
   }
 
+  exec { "apt-get update after adding openjdk-8-jdk ppa for jdk8":
+    command => "/usr/bin/apt-get update",
+    require => [ Exec["add openjdk-8-jdk ppa for jdk8"] ]
+  }
+
   package { ["openjdk-7-jdk"]:
     ensure  => "purged",
   }
 
   package { ["openjdk-8-jdk"]:
     ensure  => "installed",
-    require => [ Exec["add openjdk-8-jdk ppa for jdk8"], Exec["apt-get update"] ]
+    require => [ Exec["apt-get update after adding openjdk-8-jdk ppa for jdk8"] ]
   }
 
 }

puppet/modules/nginx/manifests/init.pp

@@ -19,20 +19,20 @@
     group   => "root",
     mode    => 0644,
     content => template("nginx/etc/nginx/sites-available/journeymonitor.com.erb"),
-    require => [ Package["nginx"], Package["php7_1-fpm"] ],
+    require => [ Package["nginx"], Package["php7.1-fpm"] ],
     notify  => Service["nginx"],
   }
 
   file { "/etc/nginx/sites-enabled/default":
     ensure  => "absent",
-    require => [ Package["nginx"], Package["php7_1-fpm"] ],
+    require => [ Package["nginx"], Package["php7.1-fpm"] ],
     notify  => Service["nginx"],
   }
 
   file { "/etc/nginx/sites-enabled/journeymonitor.com":
     ensure  => "link",
     target  => "/etc/nginx/sites-available/journeymonitor.com",
-    require => [ Package["nginx"], Package["php7_1-fpm"] ],
+    require => [ Package["nginx"], Package["php7.1-fpm"] ],
     notify  => Service["nginx"],
   }
 
@@ -42,7 +42,7 @@
     hasrestart => true,
     restart    => "/etc/init.d/nginx configtest && /etc/init.d/nginx reload", # Only restart if configuration is okay
     enable     => true,
-    require    => [ File["/etc/nginx/sites-enabled/journeymonitor.com"], Package["nginx"], Package["php7_1-fpm"] ],
+    require    => [ File["/etc/nginx/sites-enabled/journeymonitor.com"], Package["nginx"], Package["php7.1-fpm"] ],
   }
 
 }

puppet/modules/php7_1/manifests/init.pp

@@ -1,22 +1,41 @@
 class php7_1 ($fpm_user = "www-data") {
 
   # We want a PHP 7.1-only system as far as possible
-  package { ["php5-cli", "php5-sqlite", "php5-curl", "php5-fpm", "php5-common", "php5-json", "php5-readline"]:
+  package { [
+    "php5-cli",
+    "php5-sqlite",
+    "php5-curl",
+    "php5-fpm",
+    "php5-common",
+    "php5-json",
+    "php5-readline"
+  ]:
     ensure  => absent,
     require => Exec["apt-get update"],
   }
 
   exec { "add ondrej/php ppa":
-    command => "LC_ALL=C.UTF-8 /usr/bin/add-apt-repository --yes ppa:ondrej/php",
-    creates => "/etc/apt/sources.list.d/ondrej-php-trusty.list"
+    environment => [
+      "LC_ALL=C.UTF-8",
+    ],
+    command => "/usr/bin/add-apt-repository --yes ppa:ondrej/php",
+    creates => "/etc/apt/sources.list.d/ondrej-php-trusty.list",
+    require => Package[ "software-properties-common" ]
   }
 
   exec { "apt-get update after adding ondrej/php ppa":
     command => "/usr/bin/apt-get update",
     require => [ Exec["add ondrej/php ppa"] ]
   }
 
-  package { ["php7.1-cli", "php7.1-sqlite3", "php7.1-curl", "php7.1-fpm"]:
+  package { [
+    "php7.1-cli",
+    "php7.1-sqlite3",
+    "php7.1-curl",
+    "php7.1-fpm",
+    "php7.1-xml",
+    "php7.1-mbstring"
+  ]:
     ensure  => present,
     require => Exec["apt-get update after adding ondrej/php ppa"],
   }

puppet/modules/php7_1/templates/etc/php/7.1/fpm/pool.d/www.conf.erb

@@ -1,4 +1,4 @@
-# managed by puppet
+; managed by puppet
 ; Start a new pool named 'www'.
 ; the variable $pool can we used in any directive and will be replaced by the
 ; pool name ('www' here)