Comments 41
Пайплайн выполняется обычно один раз, когда девелопер пушит код. Т.е. у вас все пайплайны будут «красными» всегда просто by design и будут становиться зелеными только когда наберется достаточное количество аппрувов. И их еще ретриггерить придется каждый раз.
Пайплайн отражает результат лишь на момент сборки. Поставил "+", перезапустил пайплайн, он зелененький, а потом пришли два сеньора и поставили по минусу — а пайплайн все еще зеленый и джун нажал кнопку «Merge».
Как бы подобную логику сделал я:
— Разрешаем мерж только некому служебному faceless аккаунту, ни один разраб лично кнопку «мерж» нажать не может \ не имеет прав.
— Делаем демона, который крутится в фоне и мониторит или все МР в репозитории или только МР, назначенные на этот служебный аккаунт (что несколько более логично, в том числе с т.з. юзабилити — для мержа МР теперь надо не нажимать кнопку «мерж», а назначить МР на некого вашего merge-bot)
— Для каждого МР, попавшего в проверку, проверяем, проходит ли он по количеству голосов. Проходит — бот вливает МР в мастер. Не проходит — возвращает на автора МР
Все. Да, это не защищает полностью от второго кейса (получены плюсы, мерж запущен, а потом получены два минуса), но это можно обработать, проверяя голоса от конкретных людей. Сам сервис очень легковесен, крутится в фоне, не влияет на основной билд-пайплайн. Профит?
P.S. Небольшой дисклеймер — у нас на работе Gitlab Enterprise, но я по точно такой же схеме сделал бота, который бы ребейзил и мержил МР, избавляя разработчиков от постоянного кликанья «ребейз» (у нас мерж только как fast-forward-only). И в схему аппрувов подобная архитектура вписывается идеально.
> Т.е. у вас все пайплайны будут «красными»
Не совсем так. Обычные пайплайны, выполняющиеся по пушу (на событие коммита), будут зеленые. Красными будут только пайплайны, запущенные для проверки MR (а если точнее, то красной будет стадия проверки голосов, если к моменту проверки голосов было недостаточно. Если в стадиях сборки, тестов и т. д. ошибок не было, то остальные стадии будут зеленые)
> Поставил "+", перезапустил пайплайн, он зелененький
У нас стоит требования минимум три лайка (NEED_VOTES: 3), поэтому джун самостоятельно не сможет вмерджить, пока еще двое коллег не лайкнут
> проверяя голоса от конкретных людей
К сожалению API Gitlab не позволяет узнать, чти это голоса, иначе я бы смог сделать логику с функционалом из платной версии
Не совсем так. Обычные пайплайны, выполняющиеся по пушу (на событие коммита), будут зеленые. Красными будут только пайплайны, запущенные для проверки MР
Пайплайн обычного человека: девелопер пушит ветку, создает МР. Пайплайн запускается автоматически. И, конечно же, речь сейчас идет только про эти пайплайны как самые важные (а что там запускается после мержа в мастер — уже не так важно, какая разница, оно уже в мастере). И именно эти пайплайны и будут красными примерно всегда, если они проверяют голоса (потому что сразу после пуша голосов нет и быть не может).
Или у вас пайплайн запускается вручную, когда разработчику захочется? Ну так это не есть хорошо :)
>У нас стоит требования минимум три лайка (NEED_VOTES: 3), поэтому джун самостоятельно не сможет вмерджить, пока еще двое коллег не лайкнут
Ну вы же понимаете, что от числа общая проблема не зависит? Хоть 10 лайков надо набрать, это не отменяет того, что в описанном подходе имеется проблема. Ну условно джун лайкнул себя и позвал двух стажеров, они лайкнули. Запустили пайплайн — он зеленый. И теперь даже если кто-то поставит минус или стажеры втихаря уберут голоса — пайплайн все еще зеленый (или вы его опять руками перезапускать будете? А сколько ваш МР-пайплайн длится? У нас вот примерно полчаса, каждый раз перезапускать (а тем более когда код не менялся) — это что-то с чем-то). И если у всех есть кнопка «мерж», то ее уже могут нажать, на основании «старого» пайплайна.
Конечно, в реальной команде вы это обговорили и так делать никто не будет. Но, спрашивается, если вы доверяете своей команде, зачем вообще блокировать мерж тогда? :) А если не доверяете — то дыра до сих пор есть.
При пуше ветки запускается пайплайн, и он должен быть зеленый (сборка, тесты). Этот пайплайн выполняется при каждом коммите.
При MR запускается пайплайн, в котором добавлен блок, выполняемый только при MR.
rules:
— if: '$CI_MERGE_REQUEST_TARGET_BRANCH_NAME == «master» || $CI_MERGE_REQUEST_TARGET_BRANCH_NAME =~ /^release\/.*$/'
> Ну вы же понимаете, что от числа общая проблема не зависит?
Конечно, вы все верно написали, с командой все заранее обговорили и команде доверяем. Но с этой доработкой намного лучше в том плане, что как бы ни хотелось, нельзя по быстрому что-то вмерджить.
При пуше ветки запускается пайплайн, и он должен быть зеленый (сборка, тесты). Этот пайплайн выполняется при каждом коммите.
При MR запускается пайплайн, в котором добавлен блок, выполняемый только при MR.
Ну в общем-то это не отменяет того, что я сказал — что у вас все пайплайны в новых MR всегда красные by design ;) И это, на мой взгляд, кроме уже упомянутого, убивает всю идею, когда разработчики привыкают видеть красный пайплайн — туши свет, этого не должно быть.
Ваш workflow отличается от нашего. Мы не видим смысла в просто пушах в ветку. Заставлять девелопера смотреть в какие-то дебри гитлаба в поисках его ветки, чтобы увидеть статус пайплайна — мы на это не готовы. Да, конечно триггеры CI на пуш не отключали, но этим никто не пользуется. Типичный наш подход — девелопер сначала работает локально и как только у него есть или что показать (готовое к ревью или готово все целиком), то он пушит в ветку И сразу же создает MR. Все, дальше вся коммуникация происходит в MR. Там CI постит комменты, там другие разработчики постят комменты, там происходит ревью. Если разработчик хочет получить обратную связь (хоть от CI, хоть от коллег) — он создает MR. «висячих» веток у нас нет. У меня даже алиас такой есть:
git psmr=git push -o merge_request.create
. Просто потому что первый пуш *всегда* создает МР :)У вас: разраб-работа-пуш-пуш-пуш-мр-ревью-вливание.
У нас: разраб-работа-пуш=мр-ревью-пуш-пуш-пуш-ревью-вливание
Время сборки в среднем 20-30 минут. В пайплайне проверяю:
— при компиляции: если в Nexus уже есть артефакт, собранный на этот коммит, то сборка пропускается
— при сборке докер-образа: так же в Nexus проверяю наличие контейнера по хэшу коммита.
Поэтому повторные запуски (например, редеплой) укладываются в одну минуту
К сожалению API Gitlab не позволяет узнать, чти это голоса
Вполне позволяет, возвращает список эмоций с автором и типом эмоции (в том числе лайки/дизлайки) в мерж реквесте по запросу к апи (projects/:project_id/merge_requests/:mr_id/award_emoji
)
Сам использую эту возможность для отслеживания ревью.
Спасибо!
curl --silent --request GET --header "PRIVATE-TOKEN: xxxxx" https://gitlab.com/api/v4/projects/16901018/merge_requests/2/award_emoji | jq ".[]"
{
"id": 4872049,
"name": "thumbsdown",
"user": {
"id": 2801465,
"name": "Ivanov Ivan",
"username": "ivanovi",
"state": "active",
"avatar_url": "https://assets.gitlab-static.net/uploads/-/system/user/avatar/2801465/avatar.png",
"web_url": "https://gitlab.com/ivanovi"
},
"created_at": "2020-06-24T14:07:32.298Z",
"updated_at": "2020-06-24T14:07:32.298Z",
"awardable_id": 62518352,
"awardable_type": "MergeRequest"
}
К сожалению API Gitlab не позволяет узнать, чти это голоса, иначе я бы смог сделать логику с функционалом из платной версии
Можно просто перенести функционал апрувов в комменты. Например, ставить плюсики. Я занимался вот таким костылянием, что и вы, но потом все же взяли ЕЕ минимальный, и накрутили бота, который уже занимается распределением апруверов.
На том же github, до появления встроенной возможности использовали ботов. Если честно тоже самое ожидал увидеть тут — веб хуки на все действия есть. Можно сделать только бота мейнтейнером, дергать бота хуками — он будет считать условия и мержить.
Т. к. не каждый MR нужно сразу мерджить, желательно, чтобы бот понимал статус WIP
Так же хорошо бы иметь возможность указывать список reviewers, причем по каждому репозиторию свой.
Если найду бота с хорошим функционалом — так же напишу.
Почитал документацию, судя по описанию все есть, буду проверять
Я за вечер написал своего бота «merge-bot», пингующего гитлаб и проверяющего назначенные MR, после чего жмакающий «ребейз» и «мерж». Работает уже год, есть косячки время от времени, но в общем проблему «бесконечных ребейзов» и «невозможности сделать merge train» с грехом пополам решили. А сейчас компания снова мигрирует, на этот раз на Гитхаб, поэтому больше фич для этого бота не делаю :)
Пример запуска демона, мониторит любые МР в группе, назначенные на него. Жмакает Rebase, если нужно, потом жмакает Merge. Возвращает обратно автору, если что-то пошло не так. 180 строк питонового кода бота и 100 строк питоновой обертки над Gitlab API (универсальной, можно переиспользовать в других проектах) — все :)
docker run -d --restart=always <our-registry>/merge-bot:latest --key KEY --group GROUP --only-assigned
Кстати, а почему вы юзали хуки, а не поллинг? Ну т.е. для хуков же надо бота вывешивать на открытый порт где-то, немножко больше напряга, чем просто в цикле опрашивать гитлаб постоянно? К тому же, хуки надо добавлять к каждому репозиторию отдельно (актуально, если у вас группа с 10 репозиториями и в каждый добавлять хук — такое себе)
Хуки есть глобальные — на весь инстанс. А там уже бот решает надо ему вмешиваться или нет, например по наличию какого нибудь конфига в репозитории.
Точно! Но это энтерпрайз истории.
А еще в битбакете ущербный CI — т.е. получается придется тащить или Бамбу, или Дженкинс (concourse-ci, spinnaker etc.)
* @allTeam
tests/* @QA
jenkins/* @DevOps
core/* @Architects
Где «группы» заданы в Merge Request Approvals (если используем code owners, то эти группы могут быть, например, с нулем требуемых аппрувов, чисто для удобного отображения), они же видны красивенько в каждом MR (можете лайкнуть мой пропозал gitlab.com/gitlab-org/gitlab/-/issues/222613)
Я не знаю, как такой юзкейс может быть не очевиден. Но текущая версия гитлаба так сделать не позволяет. Будь добр, перечисляй всех QA индивидуально в `CODEOWNERS` файле. Или создавай фейковую группу внутри своей группы, куда пихай QA/DevOps/Архитектов. Что абсолютно неюзабельно.
Такое ощущение (думаю, что это близко к правде, видя, как работает гитлаб), что Merge Request Approvals и Code Owners как фичи писали совершенно разные люди, не задумывавшиеся о том, что обе фичи должна быть достаточно тесно заинтегрированы. Вот и получили что имеем.
Такое ощущение (думаю, что это близко к правде, видя, как работает гитлаб), что Merge Request Approvals и Code Owners как фичи писали совершенно разные люди, не задумывавшиеся о том, что обе фичи должна быть достаточно тесно заинтегрированы. Вот и получили что имеем.
это agile. Задача — вывести фичу на рынок. Охватить юзеров, которым она нужна. А потом (когда-нибудь!!!) можно разные фичи проинтегрировать.
Мы когда думали о том, как настроить апрувы… в общем сразу стало ясно, что хотеть будем всегда больше, чем может предложить гитлаб.
По-этому купив минимальную лицензию, что бы просто апрувы были, накидали всю логику в виде хука, который уже назначает кому апрувить. И в целом довольны. Лишь маленькие проекты юзают встроенный механизм, назначая руками
Откуда у вас будет глобальная группа «фронтендеры»? Т.е. просто рабочая гитлаб-группа то у вас одна «МояКомандаНейм», в ней там десяток репозиториев и тулов. И тут вы создаете фейковую пустую папку\группу «фронтендеры» только для того, чтобы иметь возможность звать этих людей из codeowners другого репозитория? Вам не кажется, что это дичайший костыль? Разрешили бы хоть создавать абстрактные группы просто где-то, а не делать группу синонимом «папки с репозиториями». Плюс, у меня *уже* эти «фронтендеры» описаны в «группе» из Merge Request Approvals (с 0 требуемых аппрувов, из чисто утилитарных соображений, потому что очень удобно видеть в МР, какие люди жмакали на синюю кнопку: QA ли, бекендеры ли, а может, архитект). Зачем плодить сущности?
Вы все правильно говорите. А все почему? Потому что аджайл. Потому 1st class citizens в гитлабе — это репозитории. А все остальное — это… да, лучше не увлекаться, а то обвинят в нарушении прав меньшинств. Вы сами правильно заметили, что единственный способ заиметь группу пользователей — это сделать группу репо, в нее накидать юзеров, а репо не добавлять. Далее группу можно использовать по имени.
Далее. Контейнерный регистр — то же самое. Приходится делать метарепозиторий, который пустой, чтобы в него вгружать докер образы отдельной категории (ну, не знаю — пересобранные под нашу инфру официальные).
Та же история с деплой токенами (с деплой ключами, кстати, получше) — почему до сих пор нет токена на write_repository? Хотя write_registry очень даже есть.
Ну, и многие-многие другие примеры. И они все хорошо иллюстрируют главную мысль — репо — главное, остальное — сбоку-припекв.
— MyTeamName (группа)
— — Tools (группа)
— — SomeSmallIndependentProject1 (репо)
— — SomeSmallIndependentProject2 (репо)
…
— — SomeSmallIndependentProjectN (репо)
— — MainDeliveryProjectRelatedGroup (группа)
— — — MainProject (группа)
— — — TestAutomationFramework (репо)
— — — ScriptsForMainProject (репо)
— — — WebDashboardForMainProject (репо)
Глобальные группы создаются по смыслу, а смысла в глобальной группе «бекендеров» нет в принципе, потому что нет такого репозитория, который бы принадлежал глобальной группе бекендеров. Есть одна топ-левел группа на команду (это низшая часть дерева, выше еще 5 уровней иерархии для более высоких уровней организации, других команд, поскольку у нас компания большая). Внутри уже создаются репозитории для каждой тулы\проекта, по надобности создаются группы (например стало у нас тулов больше пары репозиториев — создали группу Tools), но смысла создавать фейковые группы для ролей индивидов (бекендеры, фронтендеры, QA, архитекторы) не вообще. Потому что в репозиторий контрибьютят все и фейковая группа — костылище
Потому что в репозиторий контрибьютят все и фейковая группа — костылище
ну да ) а кто-то разве говорит иначе?
Я выше объяснил почему это так происходит.
— MyTeamName (группа)
— — Tools (группа)
— — SomeSmallIndependentProject1 (репо)
— — SomeSmallIndependentProject2 (репо)
На самом деле интересный вопрос — нужно бить по командам или по продуктам ) Потому что команды могут меняться (очевидно), а структура продуктов/проектов — более медленно изменяющаяся формация. Ну, и закон Конвея в деле )
Вы что-то странное говорите. Почему группа одна? Не одна, а как-раз много групп, с разными правами на разные репозитории.
Касательно именно аппрува мерж-реквестов сейчас ведется работа над переносом этого функционала в бесплатную версию, по просьбам сообщества, так что вероятно в некотором будущем не будет необходимости в таких костылях с award_emoji. (ссылка на тикет)
Code review в Gitlab CE: если Merge request approvals нет, но очень хочется