2396-novy-docker #2409
2396-novy-docker #2409
Conversation
|
muzes tohle pak rozdelit pouze na ten docker container a na ostatni zmeny? zatim to chapu jako draft, ale je toho hrozně moc. Principialne bych drzel radeji vice mensich docker containerů nez jeden co obsahuje vše včetně adminera |
|
No problém je, že změny které dělám zasahují do funkčnosti testů a nasazování. Ten původní docker-compose se používal všude. Jediný kontejner, který obsahuje více věcí je ten PHP, kde je navíc node. To by šlo odebrat, ale nepřijde mi to až tak důležité, když se kontejner nepoužívá v produkci. Ostatní jsou striktně oddělené. Pouštět jdou i po jednom. |
ok |
juradee
force-pushed
the
2396-novy-docker
branch
5 times, most recently
from
e02e261 to
22f0e36
Compare
|
Tyjo, z rychlýho pohledu někoho, kdo úplně běžně nedělá s Dockerem (teda jako jo, ale jinak :D) je to pro mě ještě větší černá magie než předtím... A jseš si jistej tou migrací na Make? Přijde mi, že pro PHP je Phing takový de-facto standard a moc nevidím, proč ho měnit... (ale obecně je mi to dost jedno, dokud to pomocí dokumentace dokážu spustit...) |
|
Prozatím jsem tam nechal obojí. Mě se na tom make líbí to, že nemusíš mít vendor, vše může být na jednom místa (docker, yarn, composer). |
juradee
force-pushed
the
2396-novy-docker
branch
from
22f0e36 to
2bd1cf5
Compare
|
@sinacek mrkni prosím na PR. Abych mohl odebrat draft, je potřeba ještě minimálně vyzkoušet nasazování. To úplně nechci dělat naslepo a sám. Rád ti všechny změny vysvětlím na nějakém meetu, nebo klidně i zde. |
.docs/instalace.md
Outdated
| @@ -1,41 +0,0 @@ | |||
| # Instalace pro lokální vývoj | |||
There was a problem hiding this comment.
návod jak to rozchodit by tu zůstal měl i když to bude uplně jinak
README.md
Outdated
| @@ -3,12 +3,12 @@ | |||
| [](https://codecov.io/gh/skaut/Skautske-hospodareni) | |||
| [](https://app.fossa.com/projects/git%2Bgithub.com%2Fskaut%2FSkautske-hospodareni?ref=badge_shield) | |||
|
|
|||
| **Doporučujeme přečíst si, [jak funguje vývoj a plánování práce](.docs/planovani-prace.md).** | |||
| **Doporučujeme přečíst si, [jak funguje vývoj a plánování práce](docs/readme/planovani-prace.md).** | |||
| Předejdeme tím případným nedorozuměním. | |||
|
|
|||
| Pro zapojení do vývoje doporučujeme přečíst: | |||
There was a problem hiding this comment.
Asi půjde odstranit. Mrknu na to.
There was a problem hiding this comment.
Pokud tam nebude vitepress, je to irelevantní
There was a problem hiding this comment.
stejne to bude v nejake slozce ne?
There was a problem hiding this comment.
Ano, udělám revert a nechám to tak, jak to bylo
| MYSQL_CHARACTER_SET: utf8 | ||
|
|
||
| services: | ||
| traefik: |
There was a problem hiding this comment.
traefik abychom nemuseli menit etc/hosts?
There was a problem hiding this comment.
Je to router pro ty kontejnery. Dělá managment vnitřní sítě, zajišťuje vzájemnou viditelnost. Nastavuje *.localhost domény. Hosts změna je vyžádána redirectem zpět ze skautisu a alespoň na WSL nejde tento záznam obejít. Operační systém pak neví, kam požadavek směřovat. Vevnitř wsl už to funguje. Nevím jak by se to chovalo na čistém linuxu, ale zkusím to zjistit
There was a problem hiding this comment.
Traefik znám, ale není mi jasné kdo na koho potřebuje vidět na doméně. Protože main container se připojí k db pres název kontejneru a zvenku na main container se pripojí pres /etc/hosts.
Adminer bych hodil jen na jiny port, at nemame kvuli tomu dalsi container co žere pamět
There was a problem hiding this comment.
Adminer je jediný kontejner, co tam ve výsledku být nemusí. Port ven nemá vůbec. Sestava má pouze dva porty a to 80 pro aplikaci a 3306 pro případné připojení mysql serveru například k IDE
There was a problem hiding this comment.
Kdyz vystavit adminer na localhost:8080 tak neni potreba pro něj traefik
There was a problem hiding this comment.
to pak nebudu mít přístupné pod doménou adminer.localhost
Takhle port neblokuju vůbec
There was a problem hiding this comment.
abys to mel na domene, tak potrebujes traefik a to bych to radeji nechal jen na vlastnim portu, to nic neblokuje a nebere to prostredky navíc
There was a problem hiding this comment.
Bez traefiku ta sestava nebude fungovat
There was a problem hiding this comment.
Je reverse proxy, schová pod sebou všechny služby, zajistí pěkná url, zamezí kolizi portů.
| - containers/php/ini/xdebug.ini | ||
| - ./../.env.test | ||
|
|
||
| nginx: |
There was a problem hiding this comment.
proc nginx kdyz tu mas i traefik? na lebede bezi apache
There was a problem hiding this comment.
Nejsem si úplně jistý, jestli traefik podporuje FastCGI protocol. V minulosti to bez nginx nešlo
|
Prošel jsem to a okomentoval co mi bylo nejasné. Co se týká nasazování, tak bych to asi upravil, ze uz tahle vetev se nasadi na test-h.skauting.cz pri kazdém svém běhu. Az to odladime, tak to zmenime na nasazovani na obe prostredi. |
juradee
force-pushed
the
2396-novy-docker
branch
2 times, most recently
from
f5accfd to
2ca5c0a
Compare
|
Přišlo mi upozornění na mention, ale nic nevidím, tak jestli je tu něco na mě, tak jsem zmaten :D |
juradee
force-pushed
the
2396-novy-docker
branch
from
425e219 to
d35b656
Compare
juradee
force-pushed
the
2396-novy-docker
branch
from
ee1c142 to
f9b0acc
Compare
No description provided.