[devel] Q: ON_QA

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

Добрый день.

05.12.2020 19:29, Dmitry V. Levin пишет:
> On Thu, Dec 03, 2020 at 09:50:30PM +0300, Anton Farygin wrote:
>> On 03.12.2020 21:44, Vitaly Chikunov wrote:
>>> On Thu, Dec 03, 2020 at 09:36:16PM +0300, Anton Farygin wrote:
>>>> On 03.12.2020 21:30, Vitaly Chikunov wrote:
>>>>> On Thu, Dec 03, 2020 at 09:26:34PM +0300, Dmitry V. Levin wrote:
>>>>>> On Thu, Dec 03, 2020 at 09:14:02PM +0300, Vitaly Chikunov wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> Предлагаю test-only таски не переводить в состояние EPERM. Это позволит
>>>>>>> легко отличить в почте результат run --test-only от run --commit!
>>>>>> А в какое состояние тогда их переводить?
>>>>> Варианты: TESTED, TESTEPERM,
>>>> не надо использовать термин TESTED для заданий, которые не TESTED.
>>> Я думаю, что не надо смешивать разные feature requests в один.
>> Да, ок. Давай этого медведя есть маленькими кусочками.
>>
>> не показывать EPERM для заданий, у которых не может быть EPERM потому
>> что они тестовые - эта идея отличная и, наверное, станет легче.
> Если стало легче, то я предлагаю завести новое состояние, например, ON_QA,
> и придумать более подходящий workflow, чем тот, который сложился сейчас,
> для тех репозиториев, в которых есть внешний QA.
Можно добавить следующие статусы:
1. ON_MAINT_REVIEW - таск появился в списке --needs-approval=maint
2. IN_QA_QUEUE - таск появился в списке --needs-approval=tester
3. ON_QA_REVIEW - QA взяли таск в работу.
4. READY_TO_COMMIT - получены апрувы от @maint и @tester
5. REJECTED_BY_MAINT - отклонен группой @maint
6. REJECTED_BY_QA - отклонен QA