{"action":"created","issue":{"url":"https://api.github.com/repos/Tiendil/the-tale/issues/1763","labels_url":"https://api.github.com/repos/Tiendil/the-tale/issues/1763/labels{/name}","comments_url":"https://api.github.com/repos/Tiendil/the-tale/issues/1763/comments","events_url":"https://api.github.com/repos/Tiendil/the-tale/issues/1763/events","html_url":"https://github.com/Tiendil/the-tale/pull/1763","id":110043206,"number":1763,"title":"gh-1750 Настройка CI","user":{"login":"pavetok","id":912757,"avatar_url":"https://avatars.githubusercontent.com/u/912757?v=3","gravatar_id":"","url":"https://api.github.com/users/pavetok","html_url":"https://github.com/pavetok","followers_url":"https://api.github.com/users/pavetok/followers","following_url":"https://api.github.com/users/pavetok/following{/other_user}","gists_url":"https://api.github.com/users/pavetok/gists{/gist_id}","starred_url":"https://api.github.com/users/pavetok/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pavetok/subscriptions","organizations_url":"https://api.github.com/users/pavetok/orgs","repos_url":"https://api.github.com/users/pavetok/repos","events_url":"https://api.github.com/users/pavetok/events{/privacy}","received_events_url":"https://api.github.com/users/pavetok/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-10-06T16:04:54Z","updated_at":"2015-10-11T14:08:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Tiendil/the-tale/pulls/1763","html_url":"https://github.com/Tiendil/the-tale/pull/1763","diff_url":"https://github.com/Tiendil/the-tale/pull/1763.diff","patch_url":"https://github.com/Tiendil/the-tale/pull/1763.patch"},"body":"* Добавил конфигурационный файл для travis ci\r\n* Немного поправил разворачивание стенда\r\n* Поправил пару нестабильных тестов\r\n\r\nПример сборки https://travis-ci.org/pavetok/the-tale/builds/84646169"},"comment":{"url":"https://api.github.com/repos/Tiendil/the-tale/issues/comments/147199438","html_url":"https://github.com/Tiendil/the-tale/pull/1763#issuecomment-147199438","issue_url":"https://api.github.com/repos/Tiendil/the-tale/issues/1763","id":147199438,"user":{"login":"Tiendil","id":871672,"avatar_url":"https://avatars.githubusercontent.com/u/871672?v=3","gravatar_id":"","url":"https://api.github.com/users/Tiendil","html_url":"https://github.com/Tiendil","followers_url":"https://api.github.com/users/Tiendil/followers","following_url":"https://api.github.com/users/Tiendil/following{/other_user}","gists_url":"https://api.github.com/users/Tiendil/gists{/gist_id}","starred_url":"https://api.github.com/users/Tiendil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Tiendil/subscriptions","organizations_url":"https://api.github.com/users/Tiendil/orgs","repos_url":"https://api.github.com/users/Tiendil/repos","events_url":"https://api.github.com/users/Tiendil/events{/privacy}","received_events_url":"https://api.github.com/users/Tiendil/received_events","type":"User","site_admin":false},"created_at":"2015-10-11T14:08:16Z","updated_at":"2015-10-11T14:08:16Z","body":"По поводу изменений в settings_local_example.py\r\n\r\nno-reply@game.com  надо заменить на no-reply@example.com\r\n\r\nБрать часть параметров из окружения я не против, но тогда к их именам надо приписать THE_TALE_* иначе могут быть пересечения и конфузы (POSTGRES_USER выглядит как параметр относящийся к самому сервису posgres). Кроме того, игра намертво не привязана ни к postgres ни к rabbitmq, поэтому правильным мне видится такой набор имён:\r\n\r\n- THE_TALE_DB\r\n- THE_TALE_DB_USER\r\n- THE_TALE_DB_PASSWORD\r\n\r\n- THE_TALE_AMQP_USER\r\n- THE_TALE_AMQP_PASSWORD\r\n- THE_TALE_AMQP_HOST\r\n\r\nЕсли есть желание, то можно реализовать механизм подтягивания любых конфигов из переменных окружения.\r\n\r\nТак же, поскольку settings_local_example.py — это всё-таки образец, предназаначенный скорее для объясения, чем для использования, то к строкам с os.environ.get(…) надо добавить пояснение, что юзеров и базы надо создавать, пароли указывать и вообще, можно всё руками прописать. А то практика показывает, что игру пытаются запускать довольно неопытные пользователи и не стоит их ещё и переменными окружения нагружать.\r\n"}}