radio-t / rt-bot

Мета-репо для движения "создай своего бота"
129 stars 46 forks source link

Бот-голосовалка, попытка номер 2 #111

Closed Hixon10 closed 7 years ago

strayge commented 7 years ago

1) в bot-spec кодировка неправильная 2) локально твоего бота docker-compose build также не собирает, т.к. ты указал запуск командой RUN => оно отрабатывает при билде и не завершается => билд отваливается по таймауту

Hixon10 commented 7 years ago

@strayge ох, сколько же ошибок я сделал, решив, что для написания такого бота мне не нужно ставить локально докер композ. Спасибо :)

strayge commented 7 years ago

И оно при запуске ещё кучу всего пытается загрузить => стартует очень долго, не знаю как пройдут тесты

Hixon10 commented 7 years ago

@strayge посмотрим, сейчас поправлю докерфайл

strayge commented 7 years ago

вот про bot-spec: https://github.com/umputun/rt-bot/issues/93

Hixon10 commented 7 years ago

@strayge ага, ясно. Не видел раньше этой страницы (её бы в инструкцию добавить, конечно).

Hixon10 commented 7 years ago

Кажется, билд мой наконец-то прошёл, но какой-то другой бот не прошёл тесты.

All failed bots: excerpt-bot

strayge commented 7 years ago

Да, ждем @umputun

Hixon10 commented 7 years ago

@strayge спасибо за помощь :) А то с докерами на работе не работал, а из жизненного опыта - только создание простого Докефайл. И вчера создал такой файл, оказалось, RUN - не то, что мне надо :)

strayge commented 7 years ago

@Hixon10 тут же куча примеров есть. Берёшь из любого готового бота и правишь под себя. Я так и делал.

Hixon10 commented 7 years ago

@strayge ага, с композом я так и сделал.

umputun commented 7 years ago

насколько я понимаю, тестов оно все еще не проходит

Hixon10 commented 7 years ago

@umputun сейчас погляжу... Утром тесты проходило.

Hixon10 commented 7 years ago

@umputun есть мысль, что тесты падали из-за Входных Данных С одной ковычкой. На самом-то деле в json по стандарту две ковычки, да ладно. Поправил, погляжу тесты.

Hixon10 commented 7 years ago

Кто бы знал, что это вообще значит, все эти авто тесты:

The command "BOT_BASE_URL="https://localhost" python ci/run_bot_tests.py ." exited with 0.

$ if [ "$TRAVIS_BRANCH" = "master" ]; then
  docker login \
    --username "$DOCKER_REGISTRY_LOGIN" \
    --password "$DOCKER_REGISTRY_PASSWORD" \
    "$DOCKER_REGISTRY_HOST
  docker-compose push
fi
strayge commented 7 years ago

Вроде https://github.com/umputun/rt-bot/pull/120 должен был починить тесты.

Hixon10 commented 7 years ago

@umputun All checks have passed - тесты прошли.

@strayge ага, вроде бы тот PR помог.