Skip to content
This repository was archived by the owner on Aug 16, 2021. It is now read-only.

Conversation

NikolayS
Copy link
Collaborator

@NikolayS NikolayS commented Jun 25, 2018

Большой PR, в котором:

  • используется публичный докер-образ со всеми нашими «плюшками» https://gitlab.com/NikolayS/docker-postgres-sqitch (сам образ тут: https://hub.docker.com/r/postgresmen/postgres-with-stuff/; название будет изменено на nancy-blahblah)
  • опция --run-on — определяет, где будет запускаться, локально (docker run ...) или докер-машина в AWS EC2
  • опции --aws-*** адаптированы к --run-on aws
  • существующие тесты адаптированы к --run-on aws
  • вариант --run-on localhost реализован и является вариантом по умолчанию
  • добавлены два простых теста для --run-on localhost (временно отключены)
  • немного улучшен синтаксис тестов, из запуск и отчёт об ошибках
  • добавлен файл .gitignore
  • для варианта --run-on localhost вместо копирования файлов (особенно касается больших файлов типа дампа БД) используются хардлинки. Может не работать, если на компьютере несколько разделов (см TODO)
  • директория ./docker объявлена DEPRECATED
  • в конце прогона выводится summary отчёта pgBadger (кол-во запросов, кол-во ошибок, время потраченное на запросы), пример:
-------------------------------------------
Summary:
  Queries number:		 1
  Queries duration:		 0.005  ms
  Errors number:		 0
-------------------------------------------

TODO (скорее всего уже не в этом PR):

  • опция для использования cp, а не хардлинков (сейчас не будет работать вообще, если используется локальный вариант и tmp dir находится не там, где machine_home-директория)
  • переименовать докер-образ, чтобы соответствовал проекту
  • заставить работать dind (docker-in-docker) и тесты /tests/nancy_run_localhost***
  • если 1 тест падает, продолжать и в конце сообщать о количестве фейлов
  • если --run-on aws, то ругаться сразу же, если вдруг задана какая-либо опция --aws-***
  • тесты для предыдущего пункта
  • больше разных тестов для --run-on localhost
  • БАГ: nancy_run_localhost_simple_dump.sh даёт 4 запроса, а nancy_run_localhost_simple_dump_with_index.sh — 2 запроса! Хотя используется одинаковый файл запросов.
  • улучшить синтаксис баша ([[ .. ]] или [ .. ] -- выбрать что-то одно, скорее второе; и тд)
  • использовать getops для парсинга параметров? https://linuxconfig.org/how-to-use-getopts-to-parse-a-script-options (совет @myyorick)
  • поправить ошибку:
Can't locate Text/CSV_XS.pm in @INC (you may need to install the Text::CSV_XS module) (@INC contains: /etc/perl /usr/local/lib/x86_64-linux-gnu/perl/5.22.1 /usr/local/share/perl/5.22.1 /usr/lib/x86_64-linux-gnu/perl5/5.22 /usr/share/perl5 /usr/lib/x86_64-linux-gnu/perl/5.22 /usr/share/perl/5.22 /usr/local/lib/site_perl /usr/lib/x86_64-linux-gnu/perl-base .) at /root/pgbadger/pgbadger line 2740.

@NikolayS NikolayS changed the title Dockerfile moved in a sep. repo; --run-on option (unfinished) Option "--run-on ...", use public docker image; full CI tests; more Jun 26, 2018
@NikolayS NikolayS requested review from myyorick and dmius June 26, 2018 06:29
@NikolayS
Copy link
Collaborator Author

UDPATE: Добавил пару пунктов в описание

@dmius dmius merged commit a69c582 into master Jun 26, 2018
NikolayS pushed a commit that referenced this pull request Sep 18, 2018
Option "--run-on ...", use public docker image; full CI tests; more
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants