cloudsidedev / appside

Multitenant environment automation.
http://cloudside.ch
GNU Affero General Public License v3.0
38 stars 7 forks source link

Upgrade to PHP 7 #18

Closed swissspidy closed 7 years ago

swissspidy commented 7 years ago

Note that PHP 7.1 is now available and should be considered.

ivomarino commented 7 years ago

This has to be extensively tested, we check this anyway asap.

ivomarino commented 7 years ago

will be implemented in branch 18-php7-support. The basic idea is to variabilize php.yml with something like this:

    ...
    - libapache2-mod-php{{ conf_php_version_debian_family }}
    - libapache2-mod-rpaf
    - apachetop
    - php{{ conf_php_version_debian_family }}
    - php{{ conf_php_version_debian_family }}-cli
    - php{{ conf_php_version_debian_family }}-gd
    - php{{ conf_php_version_debian_family }}-json
    - php{{ conf_php_version_debian_family }}-mysql
    - php{{ conf_php_version_debian_family }}-tidy
    - php{{ conf_php_version_debian_family }}-curl
    - php{{ conf_php_version_debian_family }}-gd
    - php{{ conf_php_version_debian_family }}-imagick
    - php{{ conf_php_version_debian_family }}-imap
    - php{{ conf_php_version_debian_family }}-intl
    - php{{ conf_php_version_debian_family }}-ldap
    - php{{ conf_php_version_debian_family }}-mcrypt
    - php{{ conf_php_version_debian_family }}-gettext
    - php{{ conf_php_version_debian_family }}-xml
    - php{{ conf_php_version_debian_family }}-mbstring
    - php{{ conf_php_version_debian_family }}-soap
    - php{{ conf_php_version_debian_family }}-zip
    ...

then we can set something like this in the tenant's group_vars/webservers PHP section:

...
# conf_php_version_debian_family: 5.6
conf_php_version_debian_family: 7.1
...

for CentOS it would be conf_php_version_redhat_family and so on. This has been tested on Ubuntu 14.04 and seems to work fine:

screen shot 2017-02-24 at 13 54 11

for the PHP 5.6 -> 7.1 transition we probably need something like: appflow provision limit=atlantis tags=php,apache2.

ivomarino commented 7 years ago

@ocean90 can be closed?

ocean90 commented 7 years ago

Yep.