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

Автоматический деплоймент #130

Closed
umputun opened this Issue Nov 20, 2016 · 20 comments

Comments

Projects
None yet
2 participants
@umputun
Copy link
Owner

umputun commented Nov 20, 2016

  • Добавить на bot.radio-t.com API для этого дела - GET /deploy. С http auth.
  • Поменять загрузку всех внешнких ботов с одноразовой (во времq старта), на периодическую (дергать /list каждую минуту, и догружать/перегружать если появился/исчез бот)
  • Добавить вызов API для deplpy со стороны CI

@umputun umputun self-assigned this Nov 20, 2016

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

GET /deploy

Наверно все-таки POST кошернее.

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

Также, вместе с этим изменением желательно сделать ветку develop и деплоить ее в тестовый чат.
Или сейчас деплой идет сразу в основной чат?

@umputun

This comment has been minimized.

Copy link
Owner Author

umputun commented Nov 20, 2016

для этого тебе надо строить image с тэгами. :latest для мастера, :develop для девелоп

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

Это не проблема. Только тебе нужно два сервера :)

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

Я как-то об этом не подумал сначала. Наверно пока лишнее.

umputun added a commit that referenced this issue Nov 20, 2016

@umputun

This comment has been minimized.

Copy link
Owner Author

umputun commented Nov 20, 2016

добавил сервис на https://bot.radio-t.com:1443/deploy

его пришлось делать отдельным и независимым от того, что он деплоит и он запускается через docker-compose-deploy.yml

можешь попробовать добавить это дело в CI. Только имей ввиду - он не даст делать несколько деплоев одновременно. Пока бежит один запрос, друой будет падать на timeout

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

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

@umputun

This comment has been minimized.

Copy link
Owner Author

umputun commented Nov 20, 2016

да. если это долго делать то трэвис упадет на timeout

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

Пока можно ограничить лимит одновременных сборок в мастере (на уровне travis).

@umputun

This comment has been minimized.

Copy link
Owner Author

umputun commented Nov 20, 2016

я добавил DEPLOY_USER и DEPLOY_PASSWD в travis

@selevit selevit referenced this issue Nov 20, 2016

Merged

Ci auto deploy #133

@umputun

This comment has been minimized.

Copy link
Owner Author

umputun commented Nov 20, 2016

хмм, wtf?

Cannot connect to the Docker daemon. Is the docker daemon running on this host?

это оно так на /usr/bin/docker images --no-trunc | grep none | awk "{print \$3}" | xargs /usr/bin/docker rmi При этом там есть DOCKER_HOST и compose его видит.

Если есть идеи - дайте знать.

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

Так ты ж его в докере запускаешь?
Надо пробросить /var/run/docker.sock как volume.

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

Хм, оно ж вроде и есть у тебя.

@umputun

This comment has been minimized.

Copy link
Owner Author

umputun commented Nov 20, 2016

конечно есть, compose без этого бы ругался тоже.

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

unix://var/run/docker.sock

Может быть unix:///var/run/docker.sock? (три слеша, два для протокола, другой для пути).

@umputun

This comment has been minimized.

Copy link
Owner Author

umputun commented Nov 20, 2016

хмм, а зачем? пусть ждет своего часа. или вызывай с timeout своим курлом или чем оно там дергается

@umputun

This comment has been minimized.

Copy link
Owner Author

umputun commented Nov 20, 2016

и таки, да unix:///var/run/docker.sock, thx

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

Да, про 503 я уже передумал.
Ну и да, когда курлу говоришь перенаправлять поток, то он в stderr выводит статистику по выполнению запроса (изменяется время, количество переданных байт).
Поэтому билд будет выполняться и не убьется трэвисом.

@selevit

This comment has been minimized.

Copy link
Contributor

selevit commented Nov 20, 2016

Значит композу пофиг на ENV DOCKER_HOST, если он не ругался.

@umputun

This comment has been minimized.

Copy link
Owner Author

umputun commented Nov 21, 2016

маловероятно. Скорее всего у него хватает ума на default в такой ситуавии

@umputun umputun closed this Nov 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.