[devel] Q: ON_QA

[Новиков Сергей]

08.12.2020 20:10, Dmitry V. Levin пишет:
> On Mon, Dec 07, 2020 at 01:55:35PM +0300, Новиков Сергей wrote:
>> 05.12.2020 19:29, Dmitry V. Levin пишет:
> [...]
>>> Если стало легче, то я предлагаю завести новое состояние, например, ON_QA,
>>> и придумать более подходящий workflow, чем тот, который сложился сейчас,
>>> для тех репозиториев, в которых есть внешний QA.
>> Можно добавить следующие статусы:
> Давайте всё-таки отличать атрибуты и состояния:
>
>> 1. ON_MAINT_REVIEW - таск появился в списке --needs-approval=maint
> Мне кажется, что это классический EPERM.
>
>> 2. IN_QA_QUEUE - таск появился в списке --needs-approval=tester
>> 3. ON_QA_REVIEW - QA взяли таск в работу.
> С точки зрения сборочницы эти состояния (ожидание QA и собственно QA)
> неотличимы, поскольку происходят снаружи.  Если бы сборочница обслуживала
> очередь QA, тогда разные состояния были бы естественны, а так непонятно,
> в чём разница, помимо атрибута, не влияющего ни на что, кроме внешнего
> вида задания.
>
> Например, если бы сборочница обслуживала какую-то очередь CI,
> то можно было бы предложить состояния вроде AWAITING_CI и ON_CI.
>
>> 4. READY_TO_COMMIT - получены апрувы от @maint и @tester
> На мой взгляд, это атрибут, а не состояние.  Состояний у задания с этим
> атрибутом может быть несколько: PENDING, COMMITTING, а также AWAITING,
> BUILDING, и т.д.
>
>> 5. REJECTED_BY_MAINT - отклонен группой @maint
> Мне кажется, что это классический EPERM.
> Я пока не вижу, чем ON_MAINT_REVIEW и REJECTED_BY_MAINT отличаются как
> состояния (т.е. чем отличаются графы переходов из этих состояний).
Предложенные мной статусы/атрибуты направлены на то, чтобы
заинтересованным людям легко можно было определить текущий
статус собранного в стабильный бранч задания.

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


-- 
Новиков Сергей
ООО «Базальт СПО»