temporalio / samples-php

Temporal PHP SDK samples
Other
93 stars 42 forks source link

Smaller container #54

Open trin4ik opened 5 months ago

trin4ik commented 5 months ago

What was changed

More order in Dockerfile for php instance.

Why?

Прогнал все примеры на работоспособность с новым контейнером, вроде пашет, но я недавно общаюсь с темпуралом и не уверен, что все результаты были верными. По сути это просто оптимизированный контейнер, который и весит меньше и его проще переносить в продакшен. Если подобное тут не нужно, в примерах, то никто и не расстроится ) Если же подобное актуально -- я бы ещё потестировал и подправил бы compose файл.

CLAassistant commented 5 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

trin4ik commented 5 months ago

По сути это просто alpine сборка. Убрал docker-php-ext-install/docker-php-ext-enable в пользу install-php-extensions. Перенёс все внешние бинарники максимально высоко, чтобы при изменении чего либо не делать лишних действий, так же разбил докерфайл на 3 слоя.

  1. билд php со всеми зависимостями + скачивание бинарников
  2. сборка vendor директории
  3. сборка автолоада.

2 и 3 шаг нужен, чтобы опять же не пересобирать composer целиком, если у вас просто поменялся код. экономит время в ci.