search

Pepita73 / webproghu_dev

Webprog.hu apache-php7.2, Drupal 8.5.5
1 stars 1 forks source link

Adatbázis IP/host #2

Closed Endyl closed 6 years ago

Endyl commented 6 years ago

Ahogy az a README-ben is látható, többé-kevésbé random IP-t kap az adatbázis container, amit aztán a vonatkozó settings.php módosítgatásával kell korrigálni (ami macerás mind automatizálás, mind git-béli kerülgetés terén).

Mivel sem docker, sem hálózat "mágus" nem vagyok, bocsánat ha nem jó a megközelítés, de így hirtelen két ötletem támadt, amire találtam valamilyen megoldást:

Az elsőre alapozva csináltam egy működő felállást, a másodikat még nem volt időm kipróbálni.

Van jobb megoldás? Ha nincs, a fentiek közül melyik a szerencsésebb?

hunsly commented 6 years ago

Igen, van: 4a76637 settings.php és a docker-compose.yaml Nekem így működött.

Pepita73 commented 6 years ago

Gondoltam, hogy a links-el van bibi, ezért commenteltem ki. :) Ki kéne próbálni, hogy:

    links:
      - db:localhost

vagy megfordítva, és akkor a configban localhost-ra lehet hivatkozni. Az lenne a tuti szerintem.

Endyl commented 6 years ago

Fejlesztő környezetben jó ez így szerintem, ahogy van. Esetleg az erre vonatkozó beállításokat ki lehetne mozgatni egy settings.local.php-be (ami nyilvános és production-re vonatkozik az maradjon a settings.php-ben, a fejlesztő környezetre vonatkozók meg mehetnek a local-ba), de ez már egy külön issue lenne.

Endyl commented 6 years ago

Közben olvasgattam még a témában, és a links elvileg már deprecated/legacy, és helyette a custom bridge hálózatot ajánlják, ahol alapból látják egymást a containerek service név alapján (compose infó). Itt is lehet alias-t adni a service-nek hálózatonként, de szerintem nem lenne túl szerencsés így a localhost alias.

Ha lesz alkalmam, kipróbálom.

hunsly commented 6 years ago

A megoldásom miért is nem jó?

Endyl commented 6 years ago

Senki nem mondta, hogy nem jó; működik rendesen :) Csak miközben más témában olvasgattam a compose doksiját, megakadt a szemem a nagy piros legacy figyelmeztetésen, így gondoltam megosztom itt az infót, hogy idővel kivehetik a dockerből, viszont van alternatíva.

Szóval felőlem nyugodtan csinálj egy PR-t, amiben csak az IP probléma megoldása van (akár linksszel, akár custom networkkel) aztán akkor zárható az issue. Bár lehet, hogy jobb lenne előbb merge-ölni #8-at, hogy minél előbb rendben legyenek az EOL-ok.

Pepita73 commented 6 years ago

@hunsly , nyitott branch-en lévő megoldás még nem megoldás. Lehet bekerül, lehet nem. Teljesen egyetértek Endyllel (kiemelés tőlem):

csinálj egy PR-t, amiben csak az IP probléma megoldása van

Ha lehet, jó lenne inkább custom networkkel.

A config megoldása már másik issue-branch-PR szerintem, ott is érdemes lenne követni az eddig felmerült ötleteket (local vs all). Ezt meg kéne előzze a composer-es telepítés, hogy lássuk a könyvtárstruktúrát, jók legyenek a volume-ok.

Pepita73 commented 6 years ago

Nem szép, de egyelőre maradt a LINKS + settings.php + bekerült egy .env.dev

Pepita73 commented 6 years ago

Találtam egy egyszerű és jó példát network-re:

https://developer.okta.com/blog/2017/10/11/developers-guide-to-docker-part-3

Endyl commented 6 years ago

Jah, ilyenre gondoltam én is.

Pepita73 commented 6 years ago

Fontos, hogy mindenkinek uptodate Docker legyen, mert én hamar beszoptam a network option-öket. Most nálam Version 17.09.0-ce-win33 (13620) van, szeretne update-elni 18.x-re, de nem engedem, mert 2/2 meghalás volt update során. Jelenleg viszont ezzel jól műxik a "custom default network". (Menet közben találtam, hogy ez kb ugyanaz, mint a régi network_mode: bridge + links:, csak most már lehet többet is megadni. Ha default a neve, akkor nem kell megadni a service-eknél, alapból azt fogják használni.

Custom driver-be én nem szeretnék belemenni soha, az már nagyon rendszergizda feladat. :)