[devel] hazardous girar builder

Alexey V. Vissarionov gremlin на altlinux.org
Вт Янв 12 14:10:03 MSK 2021


On 2021-01-12 10:55:17 +0300, Anton V. Boyarshinov wrote:

 >> Поддерживаю предложение rider@ насчёт взвода test-only
 >> для следующей сборки при достижении FAILED/EPERM.
 > Только сейчас тестер, администратор репозитория или
 > владелец acl может выдать approve на задание и отправить
 > его в репозиторий, а в таком случае не сможет, так как
 > переключать статус может только владелец задания (и это
 > логично).

И это неправильно. Параметр "--commit" должен обозначать
"отправить в репу" и ничего больше, а test-only должно
быть свойством итерации, а не задания.

То есть, когда мейнтейнер создает сборочное задание, оно
может либо просто не собраться (в этом случае все совсем
просто и понятно), либо собраться до состояния "готово к
отправке в репу". Все, сборка на этом завершена - у нас
есть готовые пакеты. Но выполнение задания продолжается.

Задание с собранными пакетами, в свою очередь, может
1. либо упереться в ACL и остановиться на EPERM,
2. либо упереться в отсутствие "--commit",
3. либо обработать параметр "--commit" и уехать в репу.

Да, очередность проверок важна!

В первом случае его может пропихнуть кто-то, указанный в
ACL. Во втором - создатель (и, пожалуй, кто угодно из ACL).
А в третьем, очевидно, и пихать ничего не надо.


-- 
Alexey V. Vissarionov
gremlin ПРИ altlinux ТЧК org; +vii-cmiii-ccxxix-lxxix-xlii
GPG: 0D92F19E1C0DC36E27F61A29CD17E2B43D879005 @ hkp://keys.gnupg.net


Подробная информация о списке рассылки Devel