Docker #88

SimonFrank14 · 2024-07-28T22:46:31Z

Dieser PR macht den IdP lauffähig unter Docker

Dockerfile und Docker Compose hinzugefügt
default nginx config hinzugefügt für nginx in Docker (kann noch ausgebaut werden und per Volume Mount hinzugefügt werden)
startup.sh Skript für erstmalige Konfigurationen beim ersten Start des Containers

Noch ausstehend:

Prüfen welche Dateien tatsächlich kopiert werden müssen und Docker Image verkleinern
Auto Push zur Docker Registry via Github Pipeline

SimonFrank14 · 2024-07-28T23:36:28Z

Dockerfile

+COPY . .
+
+# Install PHP dependencies including symfony/runtime
+RUN composer install --classmap-authoritative --no-scripts


--no-dev Flag hinzufügen, wenn fertig mit testen

SimonFrank14 · 2024-07-28T23:38:32Z

Ich erhalte aktuell auf der Seite Registrierungscodes erstellen noch einen Fehler über ein fehlendes DB Feld und auch beim hinzufügen des Users fehlt wohl ein Standardwert für das Email Verification Feld.

Entweder ist hier was in den Migrationen falsch oder man müsste nochmal in das Anlegen der User und der Codes schauen.

Ansonsten funktioniert auf den ersten Blick alles.

SimonFrank14 · 2024-07-29T11:05:06Z

@frostieDE Kannst du mir hier weiterhelfen?

frostieDE · 2024-07-29T19:12:21Z

Prüfen welche Dateien tatsächlich kopiert werden müssen und Docker Image verkleinern

Im Grunde kann der gesamte node_modules-Ordner weg - nachdem npm run build aufgerufen wurde und Webpack fertig ist, können alle JS-Abhängigkeiten weg, da sie gebundelt unter public/build/ zu finden sind.

frostieDE · 2024-07-29T19:13:56Z

Ich kenne mich zwar nicht 100%ig mit Docker aus, aber was spricht dagegen, dass man nginx und php-fpm als zwei Container getrennt laufen lässt? (außer natürlich Overhead)

Ist das üblich, dass man die Anwendung gemeinsam mit nginx in einen eigenen Container schmeißt?

SimonFrank14 · 2024-07-29T19:21:39Z

Im Grunde kann der gesamte node_modules-Ordner weg - nachdem npm run build aufgerufen wurde und Webpack fertig ist, können alle JS-Abhängigkeiten weg, da sie gebundelt unter public/build/ zu finden sind.

Das passiert schon jetzt. Allerdings habe ich noch den kompletten Quellcode im Container und das ist ein wenig overhead glaube ich. Außer wir brauchen den tatsächlich...

frostieDE · 2024-07-29T19:22:43Z

Ich erhalte aktuell auf der Seite Registrierungscodes erstellen noch einen Fehler über ein fehlendes DB Feld und auch beim hinzufügen des Users fehlt wohl ein Standardwert für das Email Verification Feld.

Entweder ist hier was in den Migrationen falsch oder man müsste nochmal in das Anlegen der User und der Codes schauen.

Ansonsten funktioniert auf den ersten Blick alles.

Ich schaue mir das die Tage mal an - auf den ersten Blick sollte das ja nicht an Docker liegen sondern ein allgemeines Deployment-Problem sein :'(

SimonFrank14 · 2024-07-29T19:30:47Z

Ich kenne mich zwar nicht 100%ig mit Docker aus, aber was spricht dagegen, dass man nginx und php-fpm als zwei Container getrennt laufen lässt? (außer natürlich Overhead)

Ist das üblich, dass man die Anwendung gemeinsam mit nginx in einen eigenen Container schmeißt?

Kann man so oder so machen. Schöner ist es aber wenn die Dienste getrennt sind. In dem Fall müsste man nochmal an die nginx Config ran, damit der die PHP Dateien aus dem anderen Container hostet. Dafür müsste man allerdings nur den fastcgi_pass Parameter auf <container_name>:9000 setzen, wenn ich das gerade richtig sehe.

Dann hätten wir drei Dienste pro Stack / Docker Compose:

nginx
php
db

frostieDE · 2024-07-31T11:00:51Z

Es gab tatsächlich eine problematische Migration - habe ich mit c46516e behoben. Das sollte dann die Fehlermeldungen lösen. Ich empfehle, die bisher entstandene Datenbank zu löschen.

SimonFrank14 · 2024-07-31T11:10:15Z

Es gab tatsächlich eine problematische Migration - habe ich mit c46516e behoben. Das sollte dann die Fehlermeldungen lösen. Ich empfehle, die bisher entstandene Datenbank zu löschen.

Ok. Dann teste ich das später mal. Ich würde vorschlagen, dass wir das Image dann per GitHub Action automatisch bauen und in die Docker Registry laden. Dafür braucht man aber einen Account. Ich teste das erstmal auf meinem Account, aber für's mergen sollten wir dann am Besten einen Account für die ganze SchulIT Umgebung nutzen.

SimonFrank14 · 2024-07-31T11:12:33Z

Im Grunde kann der gesamte node_modules-Ordner weg - nachdem npm run build aufgerufen wurde und Webpack fertig ist, können alle JS-Abhängigkeiten weg, da sie gebundelt unter public/build/ zu finden sind.

Das passiert schon jetzt. Allerdings habe ich noch den kompletten Quellcode im Container und das ist ein wenig overhead glaube ich. Außer wir brauchen den tatsächlich...

Brauchen wir abgesehen von dem Node Modules Ordner alle Dateien aus dem Repo?

Und welche Volumes sollten noch persistent gespeichert werden abgesehen von der DB?

frostieDE · 2024-07-31T14:27:29Z

Brauchen wir abgesehen von dem Node Modules Ordner alle Dateien aus dem Repo?

docs/ kann gelöscht werden, da der Ordner die Online-Dokumentation enthält. Die wird lokal im Projekt nicht eingebunden und muss somit nicht ins Image
.github wird für GitHub Actions benötigt und muss auch nicht ins Image
public/.htaccess kann theoretisch auch gelöscht werden, da wir im Image Apache nicht nutzen

Beim Ordner tests/ müsste mal prüfen, ob man den gefahrlos löschen kann. Der dort enthaltende Quelltext wird zwar nicht zur Laufzeit benötigt, aber ich weiß nicht, inwiefern der Namespace irgendwo eingebunden ist.

Alle anderen Dateien müssen jedoch im Image landen.

frostieDE · 2024-07-31T14:30:14Z

Und welche Volumes sollten noch persistent gespeichert werden abgesehen von der DB?

Der Ordner certs/ enthält die Zertifikate für den Identity Provider, die natürlich pro Instanz anders sind. Müsste also in ein Volume?! (kenne mich zu wenig mit Docker aus...)

Ansonsten gibt es halt noch die Datei .env.local, welche die Konfigurationsdatei darstellt.

Alles andere ist (statischer) Quelltext und kommt ins Image.

SimonFrank14 · 2024-08-01T21:25:02Z

Ich kenne mich zwar nicht 100%ig mit Docker aus, aber was spricht dagegen, dass man nginx und php-fpm als zwei Container getrennt laufen lässt? (außer natürlich Overhead)
Ist das üblich, dass man die Anwendung gemeinsam mit nginx in einen eigenen Container schmeißt?

Kann man so oder so machen. Schöner ist es aber wenn die Dienste getrennt sind. In dem Fall müsste man nochmal an die nginx Config ran, damit der die PHP Dateien aus dem anderen Container hostet. Dafür müsste man allerdings nur den fastcgi_pass Parameter auf <container_name>:9000 setzen, wenn ich das gerade richtig sehe.

Dann hätten wir drei Dienste pro Stack / Docker Compose:

nginx

php

db

Ich habe das hier gerade einmal ausprobiert. Das macht für uns einfach keinen Sinn das zu trennen. Wir müssten den Code sowohl in den nginx Container als auch in den fpm Container rein reichen. Dafür müssten wir ein Volume haben und dort den Code vom Docker Host aus reinladen. Das macht einfach keinen Sinn und das Setup nur deutlich komplizierter. Ich halte es für einfacher, wenn es einen Web Container gibt, der den kompletten PHP und nginx Service beinhaltet und einen separaten DB Service. Das Verlinken von verschiedenen Diensten in einem nginx würde eh wegen der Docker Netzwerk Isolation nicht den Standards entsprechen

- Removed unnecessary files - Added db ready check in docker compose --> WEB Container only starts after db with migrations etc. - Set Timezone for Docker Container - Integrated DB Env Vars in .env.local and .env - persist cert folder of web service

SimonFrank14 · 2024-08-01T22:03:09Z

Ich habe noch ein Problem mit dem Docker Image:

Seit dem Merge erhalte ich den folgenden Fehler:

5.711 > build
5.711 > encore production
5.711
5.786 Browserslist: caniuse-lite is outdated. Please run:
5.786   npx update-browserslist-db@latest
5.786   Why you should do it regularly: https://github.com/browserslist/update-db#readme
5.793 Running webpack ...
5.793
11.86 Browserslist: caniuse-lite is outdated. Please run:
11.86   npx update-browserslist-db@latest
11.86   Why you should do it regularly: https://github.com/browserslist/update-db#readme
11.86 Browserslist: caniuse-lite is outdated. Please run:
11.86   npx update-browserslist-db@latest
11.86   Why you should do it regularly: https://github.com/browserslist/update-db#readme
11.86 Browserslist: caniuse-lite is outdated. Please run:
11.86   npx update-browserslist-db@latest
11.86   Why you should do it regularly: https://github.com/browserslist/update-db#readme
12.73  DONE  Compiled successfully in 6569ms9:58:16 PM
12.73
12.73 16 files written to public/build
12.73 webpack compiled successfully
17.58 21:58:16 CRITICAL  [console] Error thrown while running command "assets:install". Message: "Too few arguments to function SchulIT\CommonBundle\EventSubscriber\ConsoleSubscriber::__construct(), 0 passed in /var/www/html/var/cache/prod/ContainerCJnuec6/getConsoleSubscriberService.php on line 21 and exactly 2 expected" ["exception" => ArgumentCountError { …},"command" => "assets:install","message" => "Too few arguments to function SchulIT\CommonBundle\EventSubscriber\ConsoleSubscriber::__construct(), 0 passed in /var/www/html/var/cache/prod/ContainerCJnuec6/getConsoleSubscriberService.php on line 21 and exactly 2 expected"]
21.32
21.32 In ConsoleSubscriber.php line 13:
21.32
21.32   Too few arguments to function SchulIT\CommonBundle\EventSubscriber\ConsoleS
21.32   ubscriber::__construct(), 0 passed in /var/www/html/var/cache/prod/Containe
21.32   rCJnuec6/getConsoleSubscriberService.php on line 21 and exactly 2 expected
21.32
21.32
21.32 assets:install [--symlink] [--relative] [--no-cleanup] [--] [<target>]
21.32
------
failed to solve: process "/bin/sh -c npm install     && npm run build     && php bin/console assets:install" did not complete successfully: exit code: 255

Hast du eine Idee, woran das liegen könnte?

Mit Commit 42edb49 funktioniert noch alles.

SimonFrank14 · 2024-08-01T22:07:12Z

Es gab tatsächlich eine problematische Migration - habe ich mit c46516e behoben. Das sollte dann die Fehlermeldungen lösen. Ich empfehle, die bisher entstandene Datenbank zu löschen.

Ok. Dann teste ich das später mal. Ich würde vorschlagen, dass wir das Image dann per GitHub Action automatisch bauen und in die Docker Registry laden. Dafür braucht man aber einen Account. Ich teste das erstmal auf meinem Account, aber für's mergen sollten wir dann am Besten einen Account für die ganze SchulIT Umgebung nutzen.

Ich konnte es leider noch nicht testen, weil ich den Branch noch nicht bauen kann (s.o.)

Create Pipeline for auto publish on release

Docker image

Create Pipeline for auto publish on release

SimonFrank14 · 2024-08-01T22:50:01Z

Ich habe noch ein Problem mit dem Docker Image:

Seit dem Merge erhalte ich den folgenden Fehler:

5.711 > build
5.711 > encore production
5.711
5.786 Browserslist: caniuse-lite is outdated. Please run:
5.786   npx update-browserslist-db@latest
5.786   Why you should do it regularly: https://github.com/browserslist/update-db#readme
5.793 Running webpack ...
5.793
11.86 Browserslist: caniuse-lite is outdated. Please run:
11.86   npx update-browserslist-db@latest
11.86   Why you should do it regularly: https://github.com/browserslist/update-db#readme
11.86 Browserslist: caniuse-lite is outdated. Please run:
11.86   npx update-browserslist-db@latest
11.86   Why you should do it regularly: https://github.com/browserslist/update-db#readme
11.86 Browserslist: caniuse-lite is outdated. Please run:
11.86   npx update-browserslist-db@latest
11.86   Why you should do it regularly: https://github.com/browserslist/update-db#readme
12.73  DONE  Compiled successfully in 6569ms9:58:16 PM
12.73
12.73 16 files written to public/build
12.73 webpack compiled successfully
17.58 21:58:16 CRITICAL  [console] Error thrown while running command "assets:install". Message: "Too few arguments to function SchulIT\CommonBundle\EventSubscriber\ConsoleSubscriber::__construct(), 0 passed in /var/www/html/var/cache/prod/ContainerCJnuec6/getConsoleSubscriberService.php on line 21 and exactly 2 expected" ["exception" => ArgumentCountError { …},"command" => "assets:install","message" => "Too few arguments to function SchulIT\CommonBundle\EventSubscriber\ConsoleSubscriber::__construct(), 0 passed in /var/www/html/var/cache/prod/ContainerCJnuec6/getConsoleSubscriberService.php on line 21 and exactly 2 expected"]
21.32
21.32 In ConsoleSubscriber.php line 13:
21.32
21.32   Too few arguments to function SchulIT\CommonBundle\EventSubscriber\ConsoleS
21.32   ubscriber::__construct(), 0 passed in /var/www/html/var/cache/prod/Containe
21.32   rCJnuec6/getConsoleSubscriberService.php on line 21 and exactly 2 expected
21.32
21.32
21.32 assets:install [--symlink] [--relative] [--no-cleanup] [--] [<target>]
21.32
------
failed to solve: process "/bin/sh -c npm install     && npm run build     && php bin/console assets:install" did not complete successfully: exit code: 255

Hast du eine Idee, woran das liegen könnte?

Mit Commit 42edb49 funktioniert noch alles.

Scheinbar läuft der Build auf anderen Maschinen (wie z.B. der Github Action) durch.

Zum Testen enfach docker build ./ im Repo machen, wenn Docker läuft. Ich tippe mal auf irgendeinen Cache bei mir, der nicht aufgeräumt wurde, muss ich nochmal nach suchen

…to docker-image

SimonFrank14 · 2024-08-01T23:48:28Z

Ich habe noch ein Problem mit dem Docker Image:
Seit dem Merge erhalte ich den folgenden Fehler:

5.711 > build
5.711 > encore production
5.711
5.786 Browserslist: caniuse-lite is outdated. Please run:
5.786   npx update-browserslist-db@latest
5.786   Why you should do it regularly: https://github.com/browserslist/update-db#readme
5.793 Running webpack ...
5.793
11.86 Browserslist: caniuse-lite is outdated. Please run:
11.86   npx update-browserslist-db@latest
11.86   Why you should do it regularly: https://github.com/browserslist/update-db#readme
11.86 Browserslist: caniuse-lite is outdated. Please run:
11.86   npx update-browserslist-db@latest
11.86   Why you should do it regularly: https://github.com/browserslist/update-db#readme
11.86 Browserslist: caniuse-lite is outdated. Please run:
11.86   npx update-browserslist-db@latest
11.86   Why you should do it regularly: https://github.com/browserslist/update-db#readme
12.73  DONE  Compiled successfully in 6569ms9:58:16 PM
12.73
12.73 16 files written to public/build
12.73 webpack compiled successfully
17.58 21:58:16 CRITICAL  [console] Error thrown while running command "assets:install". Message: "Too few arguments to function SchulIT\CommonBundle\EventSubscriber\ConsoleSubscriber::__construct(), 0 passed in /var/www/html/var/cache/prod/ContainerCJnuec6/getConsoleSubscriberService.php on line 21 and exactly 2 expected" ["exception" => ArgumentCountError { …},"command" => "assets:install","message" => "Too few arguments to function SchulIT\CommonBundle\EventSubscriber\ConsoleSubscriber::__construct(), 0 passed in /var/www/html/var/cache/prod/ContainerCJnuec6/getConsoleSubscriberService.php on line 21 and exactly 2 expected"]
21.32
21.32 In ConsoleSubscriber.php line 13:
21.32
21.32   Too few arguments to function SchulIT\CommonBundle\EventSubscriber\ConsoleS
21.32   ubscriber::__construct(), 0 passed in /var/www/html/var/cache/prod/Containe
21.32   rCJnuec6/getConsoleSubscriberService.php on line 21 and exactly 2 expected
21.32
21.32
21.32 assets:install [--symlink] [--relative] [--no-cleanup] [--] [<target>]
21.32
------
failed to solve: process "/bin/sh -c npm install     && npm run build     && php bin/console assets:install" did not complete successfully: exit code: 255

Hast du eine Idee, woran das liegen könnte?
Mit Commit 42edb49 funktioniert noch alles.

Scheinbar läuft der Build auf anderen Maschinen (wie z.B. der Github Action) durch.

Zum Testen enfach docker build ./ im Repo machen, wenn Docker läuft. Ich tippe mal auf irgendeinen Cache bei mir, der nicht aufgeräumt wurde, muss ich nochmal nach suchen

Ok, bei mir klappt es jetzt auch auf dem Rechner, nachdem ich das Repo nochmal neu gepullt habe. Dann könnte man das von mir aus mergen. Ich habe auch schon ein Test Deploy auf unserem Docker Schul-Server hinbekommen.

SimonFrank14 · 2024-08-01T23:48:36Z

Man könnte jetzt noch das Repo für den Upload im Docker Hub ändern.

SimonFrank14 · 2024-08-01T23:49:02Z

Die Doku habe ich auch schon angepasst. Ich stelle den Branch mal auf Ready to merge. Du kannst ja dann mal mit Ruhe über alles drüber schauen.

frostieDE · 2024-08-02T11:32:11Z

Eine Sache, die im Image noch fehlt (sorry, die Dokumentation dazu ist versteckt), sind Cronjobs... https://docs.schulit.de/idp/guides/cronjobs#cronjobs-auf-betriebssystemebene-ausf%C3%BChren

Kann man das mittels Crontab in Docker Images lösen?

SimonFrank14 · 2024-08-02T12:38:21Z

Eine Sache, die im Image noch fehlt (sorry, die Dokumentation dazu ist versteckt), sind Cronjobs... https://docs.schulit.de/idp/guides/cronjobs#cronjobs-auf-betriebssystemebene-ausf%C3%BChren

Kann man das mittels Crontab in Docker Images lösen?

Ich hätte zwei Ideen, müsste ich aber noch im Detail prüfen:

Man greift über Docker auf die Konsole von PHP zu. Habe ich in der neuen Doku genutzt, um den Benutzer anzulegen:

docker exec it <container_name> /usr/bin/php /var/www/html/bin/console shapecode:cron:run

Oder man könnte auch schauen, ob man crontab noch mit ins Image einbinden kann.

* Added Github Action to automatically build image (#2) Create Pipeline for auto publish on release * Added restart prop to web container * boost startup of web container after db ready * added docker installation docu

SimonFrank14 · 2024-08-03T23:12:58Z

Noch ToDo:

Pipeline anpassen
Externen Cron Job testen
Lokaler Docker Build schlägt aktuell plötzlich wieder fehl --> Da muss ich mal nach dem Fehler suchen

SimonFrank14 · 2024-08-03T23:46:59Z

Der Fehler hängt mit dem --no-dev Flag im Composer zusammen. Hier fehlt scheinbar ein Modul (zumindest, wenn ich den dev Mode in den env wieder anschalte). Ich weiß jetzt allerdings nicht, ob das fehlende Modul jetzt nur im dev Mode als Fehlermeldung geworfen wird oder nicht, weil der Build ja nach wie vor ein Prod Build ist.

Ohne no-dev Flag läuft es aber sauber durch und lässt sich auch starten

* Added Github Action to automatically build image (#2) Create Pipeline for auto publish on release * Added restart prop to web container * boost startup of web container after db ready * added docker installation docu * nginx fix

frostieDE · 2024-08-04T09:40:37Z

Der Fehler hängt mit dem --no-dev Flag im Composer zusammen. Hier fehlt scheinbar ein Modul (zumindest, wenn ich den dev Mode in den env wieder anschalte). Ich weiß jetzt allerdings nicht, ob das fehlende Modul jetzt nur im dev Mode als Fehlermeldung geworfen wird oder nicht, weil der Build ja nach wie vor ein Prod Build ist.

Ohne no-dev Flag läuft es aber sauber durch und lässt sich auch starten

Ursache: Die Umgebungsvariable ENV ist noch auf dev anstatt auf prod gestellt. Allerdings liegt das nicht an einer fehlenden .env.local-Datei, da die Variable in .env definiert wird und standardmäßig auf prod gestellt ist.

frostieDE · 2024-08-04T09:42:36Z

Nachtrag (mal wieder nicht richtig gelesen...): Schau mal ins Log (var/log/prod.log), ob dort eine Fehlermeldung auftaucht. Für den Fall, dass du in die Dev-Umgebung wechseln willst, musst du den Composer-Befehl ohne --no-dev ausführen. (Hintergrund ist der, dass Composer im Falle der prod-Umgebung den ganzen Dev-Krams nicht herunterladen und einbinden soll)

frostieDE · 2024-08-04T09:44:18Z

Eine Sache, die im Image noch fehlt (sorry, die Dokumentation dazu ist versteckt), sind Cronjobs... https://docs.schulit.de/idp/guides/cronjobs#cronjobs-auf-betriebssystemebene-ausf%C3%BChren
Kann man das mittels Crontab in Docker Images lösen?

Ich hätte zwei Ideen, müsste ich aber noch im Detail prüfen:
1. Man greift über Docker auf die Konsole von PHP zu. Habe ich in der neuen Doku genutzt, um den Benutzer anzulegen:
docker exec it <container_name> /usr/bin/php /var/www/html/bin/console shapecode:cron:run
2. Oder man könnte auch schauen, ob man crontab noch mit ins Image einbinden kann.

Lösung 1 erscheint mir natürlich flexibler (weil man kann dann selbst entscheiden, ob man Crontab oder bspw. systemd nutzt - ich nutze letzteres), aber Lösung 2 hat den Charme, dass es "out-of-the-box" funktioniert. Ich kenne mich nicht so wahnsinnig gut in der Dockerwelt aus, aber ich hätte vermutlich erwartet, dass das Docker-Image "out-of-the-box" funktioniert (und würde daher Lösung 2 vorschlagen). Aber ich nehme sehr gerne Gegenargumente an :)

SimonFrank14 · 2024-08-04T09:51:16Z

Eine Sache, die im Image noch fehlt (sorry, die Dokumentation dazu ist versteckt), sind Cronjobs... https://docs.schulit.de/idp/guides/cronjobs#cronjobs-auf-betriebssystemebene-ausf%C3%BChren
Kann man das mittels Crontab in Docker Images lösen?

Ich hätte zwei Ideen, müsste ich aber noch im Detail prüfen:
1. Man greift über Docker auf die Konsole von PHP zu. Habe ich in der neuen Doku genutzt, um den Benutzer anzulegen:
docker exec it <container_name> /usr/bin/php /var/www/html/bin/console shapecode:cron:run
2. Oder man könnte auch schauen, ob man crontab noch mit ins Image einbinden kann.
Lösung 1 erscheint mir natürlich flexibler (weil man kann dann selbst entscheiden, ob man Crontab oder bspw. systemd nutzt - ich nutze letzteres), aber Lösung 2 hat den Charme, dass es "out-of-the-box" funktioniert. Ich kenne mich nicht so wahnsinnig gut in der Dockerwelt aus, aber ich hätte vermutlich erwartet, dass das Docker-Image "out-of-the-box" funktioniert (und würde daher Lösung 2 vorschlagen). Aber ich nehme sehr gerne Gegenargumente an :)

Ich habe beides noch nicht genutzt und müsste mir Mal anschauen, wie man die nutzt.

Per Kommandozeile auf den Container zugreifen finde ich jetzt auch nicht so schön, zumal sich der Name oder die ID ja auch schnell ändern können und es dann zu Fehlern kommt.

Ich würde mal schauen, ob man den Zugriff nach außen schön gestaltet bekommt, sodass man crontab oder systemd als zusätzlichen Service in der Docker Compose definieren kann.

ci: added docker image for idp

99a2226

SimonFrank14 commented Jul 28, 2024

View reviewed changes

SimonFrank14 added 2 commits August 1, 2024 23:53

Improvments for Docker Image

42edb49

- Removed unnecessary files - Added db ready check in docker compose --> WEB Container only starts after db with migrations etc. - Set Timezone for Docker Container - Integrated DB Env Vars in .env.local and .env - persist cert folder of web service

Merge branch 'SchulIT:master' into docker-image

e321da7

Set no-dev flag for composer install

e925105

SimonFrank14 added 6 commits August 2, 2024 00:18

Create docker-image.yml

3e4d08d

Create Pipeline for auto publish on release

Merge pull request #1 from SimonFrank14/docker-image

3aa7f55

Docker image

Update docker-image.yml

d00535b

Update docker-image.yml

51cef3d

Update docker-image.yml

6ccc5f9

Added Github Action to automatically build image (#2)

46d0614

Create Pipeline for auto publish on release

SimonFrank14 added 3 commits August 2, 2024 01:34

Added restart prop to web container

752f3cb

Merge branch 'docker-image' of https://github.com/SimonFrank14/idp in…

0920017

…to docker-image

boost startup of web container after db ready

d3a0be9

SimonFrank14 marked this pull request as ready for review August 1, 2024 23:49

Docker image (#3)

07d49cc

* Added Github Action to automatically build image (#2) Create Pipeline for auto publish on release * Added restart prop to web container * boost startup of web container after db ready * added docker installation docu

SimonFrank14 marked this pull request as draft August 3, 2024 23:11

SimonFrank14 added 2 commits August 4, 2024 01:41

nginx fix

282c253

remove no dev flag temp

4c4476f

SimonFrank14 added 2 commits August 4, 2024 09:21

Merge branch 'master' into docker-image

65338de

Docker image (#4)

e6930f1

* Added Github Action to automatically build image (#2) Create Pipeline for auto publish on release * Added restart prop to web container * boost startup of web container after db ready * added docker installation docu * nginx fix

SimonFrank14 added 12 commits August 8, 2024 17:43

Update docker-image.yml

1ef6d47

Update docker-image.yml

a24d4c1

Merge branch 'SchulIT:master' into master

58e3c4e

Update docker-image.yml

2b0ff79

Update docker-image.yml

580d854

Update docker-image.yml

20e68f0

Update docker-image.yml

4139d1c

some optimizations for the docker build image

d8c5208

Merge branch 'master' into docker-image

1956310

Update docker-image.yml (#5)

336aab5

fixed filesystem permission issue

adc6b0a

Cache was not included in chown

05f1ead

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Docker #88

Docker #88

SimonFrank14 commented Jul 28, 2024

SimonFrank14 Jul 28, 2024

SimonFrank14 commented Jul 28, 2024

SimonFrank14 commented Jul 29, 2024

frostieDE commented Jul 29, 2024

frostieDE commented Jul 29, 2024

SimonFrank14 commented Jul 29, 2024

frostieDE commented Jul 29, 2024

SimonFrank14 commented Jul 29, 2024

frostieDE commented Jul 31, 2024

SimonFrank14 commented Jul 31, 2024

SimonFrank14 commented Jul 31, 2024

frostieDE commented Jul 31, 2024

frostieDE commented Jul 31, 2024

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024 •

edited

Loading

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024

frostieDE commented Aug 2, 2024

SimonFrank14 commented Aug 2, 2024

SimonFrank14 commented Aug 3, 2024

SimonFrank14 commented Aug 3, 2024

frostieDE commented Aug 4, 2024

frostieDE commented Aug 4, 2024

frostieDE commented Aug 4, 2024

SimonFrank14 commented Aug 4, 2024 •

edited

Loading

Docker #88

Are you sure you want to change the base?

Docker #88

Conversation

SimonFrank14 commented Jul 28, 2024

SimonFrank14 Jul 28, 2024

Choose a reason for hiding this comment

SimonFrank14 commented Jul 28, 2024

SimonFrank14 commented Jul 29, 2024

frostieDE commented Jul 29, 2024

frostieDE commented Jul 29, 2024

SimonFrank14 commented Jul 29, 2024

frostieDE commented Jul 29, 2024

SimonFrank14 commented Jul 29, 2024

frostieDE commented Jul 31, 2024

SimonFrank14 commented Jul 31, 2024

SimonFrank14 commented Jul 31, 2024

frostieDE commented Jul 31, 2024

frostieDE commented Jul 31, 2024

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024 • edited Loading

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024

SimonFrank14 commented Aug 1, 2024

frostieDE commented Aug 2, 2024

SimonFrank14 commented Aug 2, 2024

SimonFrank14 commented Aug 3, 2024

SimonFrank14 commented Aug 3, 2024

frostieDE commented Aug 4, 2024

frostieDE commented Aug 4, 2024

frostieDE commented Aug 4, 2024

SimonFrank14 commented Aug 4, 2024 • edited Loading

SimonFrank14 commented Aug 1, 2024 •

edited

Loading

SimonFrank14 commented Aug 4, 2024 •

edited

Loading