[devel] hazardous girar builder

[Leonid Krivoshein]

12.01.2021 16:18, Alexey V. Vissarionov пишет:
> On 2021-01-12 15:53:16 +0300, Arseny Maslennikov wrote:
>
>   >>> Поддерживаю предложение rider@ насчёт взвода test-only
>   >>> для следующей сборки при достижении FAILED/EPERM.
>   >> Только сейчас тестер, администратор репозитория или владелец
>   >> acl может выдать approve на задание и отправить его в
>   >> репозиторий, а в таком случае не сможет, так как переключать
>   >> статус может только владелец задания (и это логично).
>   > Прямо-таки отправить, _вопреки_ воле владельца/автора
>   > задания? Это что-то новенькое.
>
> Не вопреки воле, а по результатам тестирования.
>
>   > Разве мнение тестера, администратора сборочницы или другого
>   > одобрителя не является лишь рядовой галкой в списке этих
>   > мнений, и для совершения коммита в репозиторий не необходимо,
>   > чтобы все эти галки были булево истинными?
>
> Я в соседнем сообщении расписал чуть более логичный вариант
> workflow, нежели то, что сейчас реализовано у нас. Напомню,
> там параметр "--commit" определяет свойство итерации, а не
> задания, а для успешно собранного задания (с готовыми пакетами)
> предусмотрены три варианта: (1) уперлись в ACL, нужен аппрув,
> а для этого нужно тестирование; (2) уперлись в отсутствие
> параметра "--commit", нужно тестирование; (3) все в порядке,
> выполняем commit, пакеты уезжают в репу.
>
> 1 и 2 варианты объединяет то, что пакеты на _этой_ итерации
> не отправляются в репу автоматически. Но как только их кто-то
> зааппрувил - не вижу причин, почему бы им не попасть в репу.

Если бы этот "атрибут" сборки можно было бы изменить в процессе сборки, 
пока итерация не закончилась, это было бы и весьма полезно, и дало бы 
ожидаемый эффект (конечно после прочтения мануала). Мне кажется, сейчас 
данный "атрибут" воспринимается многими как свойство итерации, а не 
задания, именно из-за отсутствия такой возможности.


-- 
Best regards,
Leonid Krivoshein.