Re: [devel] Q: Do not EPERM test-only tasks

[Anton Farygin <rider@basealt.ru>]

On 03.12.2020 22:18, Dmitry V. Levin wrote:
> On Thu, Dec 03, 2020 at 10:07:52PM +0300, Anton Farygin wrote:
>> On 03.12.2020 22:00, Dmitry V. Levin wrote:
>>> On Thu, Dec 03, 2020 at 09:56:48PM +0300, Anton Farygin wrote:
>>>> On 03.12.2020 21:51, Dmitry V. Levin wrote:
>>>>> On Thu, Dec 03, 2020 at 09:30:00PM +0300, 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,
>>>>> Я не понял, что предлагается, убрать acl check из test-only tasks?
>>>>> Или игнорировать результат acl check для test-only tasks?
>>>>>
>>>>> Я пока не понял, что именно не устраивает в нынешней ситуации.
>>>>> Задания в состояние EPERM, которые test-only, явно помечены как test-only.
>>>> Если я правильно понял идею автора - предлагается ввести новое состояние
>>>> сборочного задания, означающее что сборка прошла и задание готово к
>>>> COMMIT в репозиторий.
>>>>
>>>> При этом если в этом задании наступил EPERM, то явно это не показывать в
>>>> состоянии сборочного задания.
>>> Но если задание не прошло acl check, то оно явно не готово быть
>>> закоммиченным в репозиторий.  Если предлагается эту информацию скрыть,
>>> то как этим тогда пользоваться?
>> COMMIT можно сделать и для EPERM задания (с approve и run со стороны).
>> Т.е. - состояние EPERM не означает что заданию нельзя сделать commit.
> В моём понимании вещей EPERM означает, что заданию нельзя сделать commit.
но промежуточного состояния между EPERM и COMMIT же не возникает ?