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

Anton Farygin rider на basealt.ru
Чт Дек 3 22:07:52 MSK 2020


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.

как вариант можно сделать одновременное отображения сразу двух 
состояний, например: [BUILD DONE][EPERM]




Подробная информация о списке рассылки Devel