zuzusik Jul 18 2013 at 20:18

Review Board + Mercurial — опыт внедрения и автоматизации процесса code review

7 min

13K

Programming*Version control systems*Mercurial*

From sandbox

Comments 24

danilchenko Jul 18 2013 at 20:25

при этом ее главный минус — она не имеет GUI

Ой, правда что ли?

zuzusik Jul 18 2013 at 20:38

Да, правда
www.reviewboard.org/docs/manual/dev/users/tools/post-review/

post-review is a command line tool for Windows, Linux and MacOS X that simplifies both creating and updating review requests.

Если Вы о том, что это не минус — то смотря для кого как, естественно. Для нас — минус, т.к. у нас большинство пользователей сидит на TortoiseHg

Paul Jul 19 2013 at 00:42

Официальная тулза имеет явный плюс — она поддерживается разработчиками Review Board, при этом ее главный плюс — она не имеет GUI.

Fixed.

ostapbender Jul 18 2013 at 20:41

Очень все странно. Прежде всего — зачем ревьюить каждый коммит? Не проще ли было ревьюить все изменения из feature branch перед мерджем в mainline?

zuzusik Jul 18 2013 at 20:47

Процесс разработки построен таким образом, что фичи разбиваются на минимально возможные части, каждая из которых является оконченным функционалом, готовым для использования. Для каждой части создается тикет в JIRA. Каждый коммит содержит код такой части, поэтому есть смысл ревьювить каждый коммит.

ostapbender Jul 18 2013 at 20:55

То есть все коммитят в mainline?

zuzusik Jul 18 2013 at 20:59

В подавляющем большинстве случаев — да. Иногда для фич открываются свои ветки, но опять же — каждый коммит в такой ветке соответствует определенному оконченному функционалу, поэтому есть смысл его ревьювить.

ostapbender Jul 18 2013 at 21:08

Понял.

Вы с GitHub/BitBucket и их Pull/Merge Request'ами работали? Если да, то было бы интересно услышать сравнение оных с работой в RB. Я к тому спрашиваю, что ревью отдельных коммитов все же кажется мне несколько неоптимальным.

zuzusik Jul 18 2013 at 22:03

С Pull Request пока работал всего один раз (на GitHub) — сделал и получил ответ, что он принят. Так что по этому поводу ничего сказать не могу.

RB, кстати, позволяет постить ревью сразу для диапазона коммитов. В нашем случае в этом нет особой потребности. Хотя, если фичи действительно большие и на подфичи не разбиваются, то я согласен — ревьювить каждый коммит было бы неоптимально, а для каждой фичилучше было бы создавать отдельную ветку.

NeverFoldAQ Jul 18 2013 at 21:18

0,5,10,15,20,25,30,35,40,45,50,55 * * * * /home/viktor/autoreview/pull_a5.sh

Я чего то не знаю?

zuzusik Jul 18 2013 at 21:28

Извините, Вы о чем?

script88 Jul 18 2013 at 21:29

5 * * * * /home/viktor/autoreview/pull_a5.sh

AVI Jul 18 2013 at 21:37

*/5 * * * *

script88 Jul 18 2013 at 21:39

я тупанул)

zuzusik Jul 18 2013 at 21:39

Ага, понял.
Невнимательно прочитал man. Перечитал еще раз, каждые 5 минут, это будет вот так:
*/5 * * * *
То, что Вы написали — это каждую 5-ю минуту часа.
Но вариант
0,5,10,15,20,25,30,35,40,45,50,55 * * * *
тоже вполне рабочий, хотя и записан неоптимально.

NeverFoldAQ Jul 18 2013 at 21:52

Мне прям захотелось попросить Вас привести пример добавления скрипта на выполнение каждую минуту :)

kingu Jul 23 2013 at 00:26

echo "`seq -s, 1 60` * * * * /../script" >> /../crontab
Но можно и просто * :)

AlexPublic Jul 18 2013 at 22:36

Неслабые такие требования у системки… Т.е. я понимаю когда требуется много памяти под большую нагрузку. Но что бы даже не запускалось при двух гигах…

Интересно на чём оно такое написано.

zuzusik Jul 18 2013 at 22:41

Прочитайте внимательней, под 2 GB запускалась, но

при просмотре diff-ов то и дело вылетала ошибка «Can not allocate memory».

А написана RB на Python.

AlexPublic Jul 18 2013 at 22:58

Ну так это равносильно просто запуску — не было же орды запросов на сервер.

Вообще они там явно что-то накосячили. Достаточно взглянуть сколько занимает ресурсов тот же самый diff в самом mercurial.

А написана RB на Python.

Хм, я сам люблю серверные вещи на Питоне по быстрому делать. Он конечно не самый шустрый, но таких диких результатов что-то не припомню. Или взять тот же Django — весьма жирный фреймворк, а на 2 гигах вполне себе летать будет, если не орды пользователей.

Paul Jul 19 2013 at 00:48

У нас работает на виртуалочке с 2GB и ни разу не жаловалась на «Can not allocate memory». Ревью буквально вчера постили на 276 файлов, скушала и не подавилась. Правда у нас версия 1.6.3, может они в 1.7.* чего-то накрутили требовательного к памяти, не знаю.

Paul Jul 19 2013 at 00:51

Специально сейчас пошел полистал ревью это огромное и параллельно смотрел хтопом сколько памяти откушано. Выше 420 MB не поднимается. Что само по себе странно, ибо у нас под memcached отдан 1GB.

Paul Jul 19 2013 at 00:57

Если я правильно понял у вас ревью постится с сервера автоматизированно, люди в этом участия не принимают. Зачем тогда вы мучались с плагином под thg, на сервере то можно без проблем и post-review использовать в пост-коммит хуке.

zuzusik Jul 20 2013 at 13:17

Тут сыграло роль то, что мы до этого уже пользовали плагин и с ним разобрались. А так, да, можно было и post-review использовать.