Skip to content

Commit

Permalink
some future proofing stuff (openemr#4612)
Browse files Browse the repository at this point in the history
updated min php to 7.4
added full support in CI for 8.1
Updated to mariadb 10.6 in dev environments
updated composer and npm third party packages (lcobucci/jwt required a bit of special attention)
  • Loading branch information
bradymiller authored Nov 2, 2021
1 parent 456845f commit 01c42f5
Show file tree
Hide file tree
Showing 23 changed files with 14,665 additions and 5,364 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/syntax.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
strategy:
fail-fast: false
matrix:
php: ['7.3', '7.4', '8.0', '8.1']
php: ['7.4', '8.0', '8.1']
name: PHP ${{ matrix.php }}
steps:
- uses: actions/checkout@v2
Expand Down
215 changes: 16 additions & 199 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,103 +8,11 @@ on:

jobs:

build_apache_73_105:
name: PHP 7.3 - Apache - MariaDB 10.5
build_apache_74_106:
name: PHP 7.4 - Apache - MariaDB 10.6
runs-on: ubuntu-20.04
env:
DOCKER_DIR: apache_73_105
OPENEMR_DIR: /var/www/localhost/htdocs/openemr
CHROMIUM_INSTALL: "apk update; apk add --no-cache chromium chromium-chromedriver; export PANTHER_CHROME_DRIVER_BINARY=/usr/lib/chromium/chromedriver"
steps:
- uses: actions/checkout@v2

- name: Install PHP
uses: shivammathur/setup-php@v2
with:
php-version: '7.4'

- name: Report PHP Version
run: php -v

- name: Install npm package
uses: actions/setup-node@v1
with:
node-version: '12'

- name: Dockers environment start
run: |
source ci/ciLibrary.source
dockers_env_start
- name: Main build
run: |
source ci/ciLibrary.source
composer_github_auth
main_build
- name: CCDA build
run: |
source ci/ciLibrary.source
ccda_build
- name: Install and configure
run: |
source ci/ciLibrary.source
install_configure
- name: Unit testing
run: |
source ci/ciLibrary.source
build_test_unit
if: ${{ success() || failure() }}

- name: E2e testing
run: |
source ci/ciLibrary.source
build_test_e2e
if: ${{ success() || failure() }}

- name: Api testing
run: |
source ci/ciLibrary.source
build_test_api
if: ${{ success() || failure() }}

- name: Fixtures testing
run: |
source ci/ciLibrary.source
build_test_fixtures
if: ${{ success() || failure() }}

- name: Services testing
run: |
source ci/ciLibrary.source
build_test_services
if: ${{ success() || failure() }}

- name: Validators testing
run: |
source ci/ciLibrary.source
build_test_validators
if: ${{ success() || failure() }}

- name: Controllers testing
run: |
source ci/ciLibrary.source
build_test_controllers
if: ${{ success() || failure() }}

- name: Common testing
run: |
source ci/ciLibrary.source
build_test_common
if: ${{ success() || failure() }}

build_apache_74_105:
name: PHP 7.4 - Apache - MariaDB 10.5
runs-on: ubuntu-20.04
env:
DOCKER_DIR: apache_74_105
DOCKER_DIR: apache_74_106
OPENEMR_DIR: /var/www/localhost/htdocs/openemr
CHROMIUM_INSTALL: "apk update; apk add --no-cache chromium chromium-chromedriver; export PANTHER_CHROME_DRIVER_BINARY=/usr/lib/chromium/chromedriver"
steps:
Expand Down Expand Up @@ -836,11 +744,11 @@ jobs:
build_test_common
if: ${{ success() || failure() }}

build_nginx_73:
name: PHP 7.3 - Nginx - MariaDB 10.5
build_nginx_74:
name: PHP 7.4 - Nginx - MariaDB 10.6
runs-on: ubuntu-20.04
env:
DOCKER_DIR: nginx_73
DOCKER_DIR: nginx_74
OPENEMR_DIR: /usr/share/nginx/html/openemr
CHROMIUM_INSTALL: "apt-get update; apt-get install -y chromium-driver; export PANTHER_CHROME_DRIVER_BINARY=/usr/bin/chromedriver"
steps:
Expand All @@ -857,7 +765,7 @@ jobs:
- name: Install npm package
uses: actions/setup-node@v1
with:
node-version: '12'
node-version: '14'

- name: Dockers environment start
run: |
Expand Down Expand Up @@ -903,6 +811,7 @@ jobs:
source ci/ciLibrary.source
build_test_fixtures
if: ${{ success() || failure() }}

- name: Services testing
run: |
source ci/ciLibrary.source
Expand All @@ -927,11 +836,11 @@ jobs:
build_test_common
if: ${{ success() || failure() }}

build_nginx_74:
name: PHP 7.4 - Nginx - MariaDB 10.5
build_nginx_8:
name: PHP 8.0 - Nginx - MariaDB 10.6
runs-on: ubuntu-20.04
env:
DOCKER_DIR: nginx_74
DOCKER_DIR: nginx_80
OPENEMR_DIR: /usr/share/nginx/html/openemr
CHROMIUM_INSTALL: "apt-get update; apt-get install -y chromium-driver; export PANTHER_CHROME_DRIVER_BINARY=/usr/bin/chromedriver"
steps:
Expand All @@ -940,7 +849,7 @@ jobs:
- name: Install PHP
uses: shivammathur/setup-php@v2
with:
php-version: '7.4'
php-version: '8.0'

- name: Report PHP Version
run: php -v
Expand Down Expand Up @@ -1019,11 +928,11 @@ jobs:
build_test_common
if: ${{ success() || failure() }}

build_nginx_8:
name: PHP 8.0 - Nginx - MariaDB 10.5
build_nginx_81:
name: PHP 8.1 - Nginx - MariaDB 10.6
runs-on: ubuntu-20.04
env:
DOCKER_DIR: nginx_80
DOCKER_DIR: nginx_81
OPENEMR_DIR: /usr/share/nginx/html/openemr
CHROMIUM_INSTALL: "apt-get update; apt-get install -y chromium-driver; export PANTHER_CHROME_DRIVER_BINARY=/usr/bin/chromedriver"
steps:
Expand All @@ -1032,7 +941,7 @@ jobs:
- name: Install PHP
uses: shivammathur/setup-php@v2
with:
php-version: '8.0'
php-version: '8.1'

- name: Report PHP Version
run: php -v
Expand Down Expand Up @@ -1110,95 +1019,3 @@ jobs:
source ci/ciLibrary.source
build_test_common
if: ${{ success() || failure() }}

build_nginx_81:
name: PHP 8.1 - Nginx - MariaDB 10.5 (Note skipping lots of tests in PHP 8.1 for now)
runs-on: ubuntu-20.04
env:
DOCKER_DIR: nginx_81
OPENEMR_DIR: /usr/share/nginx/html/openemr
CHROMIUM_INSTALL: "apt-get update; apt-get install -y chromium-driver; export PANTHER_CHROME_DRIVER_BINARY=/usr/bin/chromedriver"
steps:
- uses: actions/checkout@v2

- name: Install PHP
uses: shivammathur/setup-php@v2
with:
php-version: '8.1'

- name: Report PHP Version
run: php -v

- name: Install npm package
uses: actions/setup-node@v1
with:
node-version: '14'

- name: Dockers environment start
run: |
source ci/ciLibrary.source
dockers_env_start
- name: Main build
run: |
source ci/ciLibrary.source
composer_github_auth
main_build
- name: CCDA build
run: |
source ci/ciLibrary.source
ccda_build
- name: Install and configure
run: |
source ci/ciLibrary.source
install_configure
- name: Unit testing (skipping in PHP 8.1 for now)
run: |
source ci/ciLibrary.source
#build_test_unit
if: ${{ success() || failure() }}

- name: E2e testing (skipping in PHP 8.1 for now)
run: |
source ci/ciLibrary.source
#build_test_e2e
if: ${{ success() || failure() }}

- name: Api testing (skipping in PHP 8.1 for now)
run: |
source ci/ciLibrary.source
#build_test_api
if: ${{ success() || failure() }}

- name: Fixtures testing (skipping in PHP 8.1 for now)
run: |
source ci/ciLibrary.source
#build_test_fixtures
if: ${{ success() || failure() }}

- name: Services testing (skipping in PHP 8.1 for now)
run: |
source ci/ciLibrary.source
#build_test_services
if: ${{ success() || failure() }}

- name: Validators testing (skipping in PHP 8.1 for now)
run: |
source ci/ciLibrary.source
#build_test_validators
if: ${{ success() || failure() }}

- name: Controllers testing (skipping in PHP 8.1 for now)
run: |
source ci/ciLibrary.source
#build_test_controllers
if: ${{ success() || failure() }}

- name: Common testing (skipping in PHP 8.1 for now)
run: |
source ci/ciLibrary.source
#build_test_common
if: ${{ success() || failure() }}
22 changes: 0 additions & 22 deletions ci/apache_73_105/docker-compose.yml

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ version: '3.1'
services:
mysql:
restart: always
image: mariadb:10.5
image: mariadb:10.6
command: ['mysqld','--character-set-server=utf8mb4']
environment:
MYSQL_ROOT_PASSWORD: root
Expand Down
33 changes: 0 additions & 33 deletions ci/nginx_73/docker-compose.yml

This file was deleted.

Loading

0 comments on commit 01c42f5

Please sign in to comment.