Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft: Docker deploy #749

Closed
wants to merge 37 commits into from
Closed

Draft: Docker deploy #749

wants to merge 37 commits into from

Conversation

glader
Copy link
Contributor

@glader glader commented Oct 22, 2021

Готов представить на ваш суд выкатку движка через образ докера. Сборка проверена, можно посмотреть здесь.

Что надо сделать для выкатки:

  • прописать в секреты CI переменные, которые упомянуты в deploy команде. Не обязательно все, но нужные.
  • получить токен с правами на registry и прописать в секреты под именем TOKEN.

Может возникнуть проблема с базой, лучше сделать бекап перед первым деплоем.

Не все решения мне здесь нравятся, но лучше я не придумал. Готов обсудить. Буду благодарен, если кто-то еще проверит выкатку на пустой сервер.

@glader glader requested a review from vas3k as a code owner October 22, 2021 18:09
@dmitvitalii
Copy link
Collaborator

dmitvitalii commented Oct 23, 2021

Локальный
docker-compose up

Creating 4affclub_webpack_1 ...
Creating 4affclub_redis_1   ...
Creating club_postgres      ... error

ERROR: for club_postgres  Cannot create container for service postgres: ConflictCreating 4affclub_webpack_1 ... done
Creating 4affclub_redis_1   ... done
rename) that container to be able to reuse that name.

ERROR: for postgres  Cannot create container for service postgres: Conflict. The container name "/club_postgres" is already in use by container "2108566b739a2e28defaaa3672aaae8d404d006b4655d35e369206f49a096d90". You have to remove (or rename) that container to be able to reuse that name.
ERROR: Encountered errors while bringing up the project.

Не будет такого при выкладки в прод? Или надо железно на пустом сервере тестить?

UPD: снёс все образы локально, запускается и логинится.

@glader
Copy link
Contributor Author

glader commented Oct 23, 2021

@dmitvitalii странно, в docker-compose нет ни слова про 4aff. Откуда у тебя это?

@glader glader changed the title Docker deploy Draft: Docker deploy Oct 24, 2021
vas3k
vas3k previously approved these changes Oct 29, 2021
Copy link
Owner

@vas3k vas3k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Наконец-то дошли руки посмотреть. Выглядит отлично, пойду прописвать секреты, а в какой-нибудь из выходных засяду следить за деплоем. Спасибо тебе!

.github/workflows/deploy.yml Outdated Show resolved Hide resolved
@glader
Copy link
Contributor Author

glader commented Oct 30, 2021

Только не мерджи плз, пока я крон не добавлю. А то нехорошо получится.

@vas3k
Copy link
Owner

vas3k commented Oct 30, 2021

А, ты прям сюда хотел крон дописывать? Давай тогда, я как раз о нём думал. Еще вопрос: как ты хостишь статику? В старой версии она собиралась в папочку frontend/dist и там снаружи стоял nginx.conf https://github.com/vas3k/vas3k.club/blob/master/etc/nginx/vas3k.club.conf#L38-L43 который как раз её раздавал на правильно URL. В новой же фронтенд билдится внутри докерфайла, а как он сервится наружу тогда? Не джангой же

@glader
Copy link
Contributor Author

glader commented Oct 30, 2021

Крон обязательно надо сюда, потому что при такой выкатке на сервере не будет кода приложения, и нечего будет запускать из внешнего крона.

@glader
Copy link
Contributor Author

glader commented Oct 30, 2021

Со статикой пришлось немного поизвращаться, не уверен, что правильно, но лучше не нашел пока варианта. Сначала маунт, потом копирование. Маунт указывает на папку на хосте, в которой раньше лежала статика, поэтому конфиг внешнего nginx править не надо.

glader and others added 15 commits November 1, 2021 21:16
* Убран тип поста Боль

* Фикс миграции

Co-authored-by: Mikhail Polykovskij <mikhail.polykovskiy@x5.ru>
* Badges draft

* Badges CSS for comments, posts, profiles

* Almost done, needs data and icons

* CSS fixes + frontend bugs

* More logical fixes

* Fixing bugs, rename things

* Add icons and initial data
* Add friends page vas3k#723

* Adds subscription status to profile

* send 403 if user tries to look someones friends vas3k#723

* bring back old name for add to friends button vas3k#723

* Fixes typo, covers PR suggestions

* Visual improvements

Co-authored-by: ollkostin <ollkostin@gmail.com>
Co-authored-by: kedMertens <kedmail@pm.me>
Co-authored-by: vas3k <me@vas3k.ru>
Signed-off-by: Vitalii Dmitriev <dmitvitalii@gmail.com>
* Draft for cooking achievements

* Fixing bg color and using proper URLs

* Fixing pictures order

* Fixing pastry_cook_2
@glader
Copy link
Contributor Author

glader commented Nov 1, 2021

Гребаный гитхаб. Я обновил ветку из мастера, и он теперь показывает все эти коммиты, которые уже есть в мастере. Я хз, заново PR делать? Вастрик, давай в гитлаб съедем, там такой херни нету.

@glader
Copy link
Contributor Author

glader commented Nov 3, 2021

@vas3k давай я на днях заново PR сделаю. Этот уже не спасти :)

@vas3k
Copy link
Owner

vas3k commented Nov 3, 2021

Ты просто ребейзнулся (или вмержил) старую ветку, скорее всего свой старый мастер, а в этом репозитории origin/master уже ускакал вперед, так что гитхаб и показывает теперь все изменения между этим и твоим репо. Я тоже хз как чинить это если честно, попробовал ребейзнуть твою ветку, но там нет моей :(

@vas3k
Copy link
Owner

vas3k commented Nov 3, 2021

@glader давай, спасибо 🙏

@glader glader mentioned this pull request Nov 4, 2021
@vas3k vas3k closed this Nov 4, 2021
@vas3k vas3k removed the do not merge label Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants