MaMaKow / dienstplan-apotheke

Ein Programm zur umfassenden Führung eines Dienstplanes für Apotheken.
GNU Affero General Public License v3.0
12 stars 2 forks source link

[IMP] Set up the system with docker #113

Open fkantelberg opened 3 years ago

fkantelberg commented 3 years ago

Hallo @MaMaKow,

für einen Kunden sollte dieses Projekt als Schichtplan aufgesetzt werden. Damit dies reibungsloser in unsere Infrastruktur passt, haben wir uns entschieden dies mit Hilfe von docker/docker-compose umzusetzen. Die Änderungen würden wir gern zurückspielen, da man damit auch schneller mal PHP und Datenbankversionen testen kann.

Dabei sind ebenso ein paar Probleme im PHP aufgetaucht, die wir vorerst mittels eines Patches (fixes.patch) gelöst haben. Den Patch würde ich vor dem Merge wieder entfernen, wenn die Änderungen soweit von dir abgenommen sind.

Leider war es nicht möglich die config.php durch docker zu erstellen, da ich keine schnelle Möglichkeit gefunden habe, das Datenbankschema und den Admin-Nutzer einfach über ein php-Skript zu erstellen. Dies wäre gut, damit man noch einfacher Parametrisieren und Aufsetzen kann, ohne die Datenbank-Verbindung etc. eintragen zu müssen.

OSevangelist commented 3 years ago

Hallo @MaMaKow leider war das ganze im Test für unseren Anwendungssfall dann doch nicht geeignet, da wir es für eine NICHT-Apotheken Domäne verwenden wollten. Außerdem hat sich gezeigt das das o.g. Role Out meines Kollegen @fkantelberg auch noch das Problem hatte dass zusätzlich noch

SET GLOBAL sql_mode=(SELECT REPLACE(@@sql_mode,'ONLY_FULL_GROUP_BY',''));

Im Datenbankcontainer auszuführen war um Probleme mit der verwendeten GROUP BY Funktion zu eleminieren. Dennoch wäre auch aus meiner Sicht ein Docker-Compose basiertes Setup für das Projekt wünschenswert.

MaMaKow commented 3 years ago

Vielen Dank für die Kommentare. Ich werde heute Abend mal drauf schauen, wenn die Kinder im Bett sind.

Mit docker habe ich mich bisher nicht beschäftigt, weil ich primär zusätzliche Komplexität sehe, ohne großen Benefit für den Anwender.