Fix CI #1237

jdecool · 2022-02-03T19:29:38Z

The CI is red and this PR fix it.

There's 3 problems:

Thruway docker image (docker/thruway/Dockerfile) use voryx/thruway which also uses thruway/client which is not currently compatible with PHP 8.0. So before the library is compatible, I've fixed the PHP version to 7.4 for this container.
docker/Dockerfile image is executed with PHP 7.4 & 8.0 and use PEAR to install some extensions. As PR is not fully compatible with PHP 8.0, required dependencies are compiled.
enqueue libraries depends on thruway/pawl-transport which depend on ratchet/pawl (^0.3.1). The 0.3.x version doesn't support guzzlehttp/psr7 2.x branch. So we needs to restrict guzzlehttp/psr7 to 1.x branch

jdecool · 2022-02-04T10:01:43Z

jdecool · 2022-02-04T10:09:31Z

Could you retry the failed job ?

docker/Dockerfile

makasim · 2022-02-08T10:48:41Z

Thanks!

jdecool added 3 commits February 3, 2022 20:35

Force makasim/nginx-php-fpm for thruway Docker service

1930c72

Fix PEAR/pecl usage in Docker

d23df86

Fix PHP version usage in Docker container

c3901cd

jdecool changed the title ~~Force makasim/nginx-php-fpm for thruway Docker service~~ Fix CI Feb 3, 2022

jdecool marked this pull request as ready for review February 3, 2022 21:27

jdecool mentioned this pull request Feb 3, 2022

Allow ext-rdkafka 6 usage #1233

Merged

Fix guzzlehttp/psr7 version

a0cc3ea

Steveb-p reviewed Feb 6, 2022

View reviewed changes

docker/Dockerfile Show resolved Hide resolved

Steveb-p approved these changes Feb 6, 2022

View reviewed changes

makasim merged commit fa2a406 into php-enqueue:master Feb 8, 2022

jdecool deleted the fix-ci branch February 8, 2022 11:09

Provide feedback