Refactor docker support

[amd-9]

Вынес в issue список того, чего не хватает в данный момент на мой взгляд

• неработают тесты
• в READMe.md нет описания того как работать с проектам в контейнерах
• рефакторинг Dockerfile

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

1. Dockerfile должен быть самодостаточным, чтоб собрать hexlet-cv и стратуть без необходимости прописывать CMD или ENTRYPOINT в явном виде
2. При использовании контейнеризации в качетсве легковесного варианта для разработки в компоуз подставляется volume с исходным кодом на локальном диске
3. Должна быть возможность разрабатывать не имея установленного Ruby на локальной машине. Правишь код в текстовом редакторе, стартанул компоуз, получил результат

У нас как то разъехался процесс разработки надо данной фичей на два ПРа :) - #437 и #436

[amd-9]

@usernaimandrey @fey @grozwalker предлагаю коллективно обсудить

[usernaimandrey]

@amd-9 про проблему с тестами описал в коментариях к ПР, я не понял за чем его приняли, он был не доработан, если знаете как пофиксить пуллреквесты велком)
По пунктам что вы описали вроде все есть, ну кроме тестов и описания, опять же если у вас есть свой взгляд на ситуацию то пулл реквксты велком это же опенсорс:) ни кто не обидится, возможно ваш следующий пр будет лучше чем мой.

[fey]

тесты не работают, потому что в тестах подставляется request.host = 'www.example.com', который по идее должен подтягиваться из енв файла .env.test, но нифига. Все подтягивается из .env. @grozwalker наверное твоя помощь нужна

Dockerfile должен быть самодостаточным, чтоб собрать hexlet-cv и стратуть без необходимости прописывать CMD или ENTRYPOINT в явном виде

Сейчас все это есть, на самом деле. Только порядок слоев по-хорошему стоит поменять. Щас нода ставится ДО установки пееременных, типа директории проекта. Вероятность того, тчо версия ноды будет меняться выше, чем директория приложения и другие переменные.

По сути вся работа щас сводится к make compose-setup, make compose

Кстати мне кажется, что нужно через configus указать нужный хост, а не обращаться к нему напрямую из контроллера. Тогда для прода можно будет указать хост, а для дева или теста будет просто значение в конфиге @grozwalker так же норм будет?

Можно даже попробовать сделать как в КБ щас https://github.com/hexlet-basics/hexlet-basics/blob/b96870042e39530e37b65ec3c9a746073aff99e4/app/lib/configus.rb#L59

[fey]

Плюс докер композ конфиг стоит доработать, убрать env_file секцию имхо. Стоит гем dotenv, который подтягивает переменные из файла.
Что еще. Щас не работает в докере разработка в контейнере фронтенда, потому что ничего не будет пересобираться. Для этого надо или сделать отдельный сервис или закинуть хероку в образ, чтобы просто запускать heroku local
Щас в композе большая страшная команда

[fey]

Ну и щас можно кодить спокойно, запуская все в контейнере. Вскоде выбрать remote containerк примеру

[usernaimandrey]

тесты не работают, потому что в тестах подставляется request.host = 'www.example.com'

точно спасибо

[usernaimandrey]

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

[usernaimandrey]

Плюс докер композ конфиг стоит доработать, убрать env_file секцию имхо.

да это решает проблемму с тестами

[usernaimandrey]

@amd-9 доделаешь? единственное я пофикшу запуск тестов) - это моя не доработка)

[amd-9]

@amd-9 доделаешь? единственное я пофикшу запуск тестов) - это моя не доработка)

Да. Добавлю по пунктам

[grozwalker]

Сори, не увидел этот ишьюс. Проблема еще актуальна? вроде фиксили докер

[usernaimandrey]

Здесь вроде @amd-9 сказал до шлифует)

[grozwalker]

Но я все равно не понимаю что именно надо дошлифовать? вроде же все работает

[usernaimandrey]

Но я все равно не понимаю что именно надо дошлифовать? вроде же все работает

по рекомендации @fey можно порядок слоев поменять и закинуть хероку в образ, что бы фронтенд пересобирался