Re: [devel] Q: ON_QA

["Alexey V. Vissarionov" <gremlin@altlinux.org>]

On 2020-12-09 11:50:37 +0300, Новиков Сергей wrote:

 >>> Можно добавить следующие статусы:
 >> Давайте всё-таки отличать атрибуты и состояния:
 >>> 5. REJECTED_BY_MAINT - отклонен группой @maint
 >> Мне кажется, что это классический EPERM.
 >> Я пока не вижу, чем ON_MAINT_REVIEW и REJECTED_BY_MAINT
 >> отличаются как состояния (т.е. чем отличаются графы
 >> переходов из этих состояний).
 > Предложенные мной статусы/атрибуты направлены на то, чтобы
 > заинтересованным людям легко можно было определить текущий
 > статус собранного в стабильный бранч задания.

А велика ли разница? Ну, разве что EPERM можно разделить на
"awaiting approval" и "rejected" - в первом случае действия
от мейнтейнера еще не было, а во втором мейнтейнер уже сказал
свое веское.

 > Так как в данный момент мейнтейнер, собирающий в стабильный
 > бранч, часто не понимает какой статус у задания и на каком
 > этапе проверки оно находится, он видит только статус EPERM,
 > хотя задание может быть уже отклонено группой @maint или
 > группой @tester.

Привязка к текущей бизнес-логике конторы => неуниверсально =>
не нужно.

 >>> 6. REJECTED_BY_QA - отклонен QA
 >> Это состояние задания я тоже пока не понимаю.
 >> В какие другие состояния возможен переход из этого состояния?
 > Получается это не состояние, а атрибут. С данным атрибутом,
 > возможен переход в любые другие состояния.

Ну и на зачем он тогда такой нужен? Если по уму, то rejected -
это частный случай failed.


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