[devel] hazardous girar builder

[Dmitry V. Levin]

On Mon, Jan 11, 2021 at 01:49:08PM +0300, Arseny Maslennikov wrote:
> On Mon, Jan 11, 2021 at 01:44:21PM +0300, Dmitry V. Levin wrote:
> > On Mon, Jan 11, 2021 at 01:25:16PM +0300, Arseny Maslennikov wrote:
> > [...]
> > > Я вообще планировал _завтра_ отправить это задание.
> > > Извините, произошло недоразумение; ниже о нём.
> > > 
> > > Когда я создавал задание, я выставил ему test-only=yes.
> > > 14 попытку я собрал с --commit (мне в своё время объяснили, что задание
> > > в состоянии TESTED никогда не получит одобрения, а, чтобы ему иметь шанс
> > > быть рассмотренным, его нужно с --commit собрать до EPERM).
> > > У нас в инфраструктуре есть рассылка писем людям об изменениях ACL в
> > > отношении этих людей, слава Богу, но об одобрении подзаданий её нет.
> > > Ещё чуть позже я отправил try 15, которая соответствует диффу из
> > > предыдущего моего письма. Для этого я выполнил `ssh girar task run
> > > 263468', полагая, что test_only=yes. Где-то между этим моментом и try 14
> > > test_only _приняло_ значение no (ещё раз простите, я до той поры не
> > > видел нужды проверять все параметры после каждого действия с заданием; я
> > > полагал, что такую подставу сборочница себе не позволяет). Задание
> > > улетело в сизиф; хорошо, что это затронуло лишь 2 пакета — их мы с
> > > мейтейнерами починили.
> > > 
> > > Если робот сам по желанию откуда-то взявшейся у него левой пятки
> > > позволяет себе менять нарочно выставленные параметры, этот робот —
> > > опасный, если не сказать "плохой". Сейчас все читатели devel@ могут
> > > наблюдать этому подтверждение: человека ошибочно записали в идиоты — или
> > > в мудаки — не по его поступкам, а из-за происшествий вне его
> > > зоны ответственности.
> > 
> > Jan  6 12:03:54 gyle girar-sh: arseny: -c task run 263468
> > Jan  6 15:19:40 gyle girar-builder: user=arseny task=263468 repo=sisyphus try=13 iter=1 status=TESTED
> > Jan  7 11:03:16 gyle girar-sh: arseny: -c task run 263468 --commit
> > Jan  7 11:22:01 gyle girar-builder: user=arseny task=263468 repo=sisyphus try=14 iter=1 status=EPERM
> > Jan  7 15:37:30 gyle girar-sh: shrek: -c acl sisyphus llvm11.0 add arseny
> > Jan  8 10:52:19 gyle girar-sh: arseny: -c task run 263468
> > Jan  8 14:28:08 gyle girar-builder: user=arseny task=263468 repo=sisyphus try=15 iter=2 status=DONE
> > (время указано в UTC)
> > 
> > На мой взгляд, тут не видно никаких аномалий, система сделала
> > в точности то, о чём её проинструктировали arseny@ и shrek на .
> 
> А на мой взгляд, система сделала то, о чём её следовало
> проинструктировать так:
> 
> > Jan  6 12:03:54 gyle girar-sh: arseny: -c task run 263468
> > Jan  6 15:19:40 gyle girar-builder: user=arseny task=263468 repo=sisyphus try=13 iter=1 status=TESTED
> > Jan  7 11:03:16 gyle girar-sh: arseny: -c task run 263468 --commit
> > Jan  7 11:22:01 gyle girar-builder: user=arseny task=263468 repo=sisyphus try=14 iter=1 status=EPERM
> > Jan  7 15:37:30 gyle girar-sh: shrek: -c acl sisyphus llvm11.0 add arseny
> > Jan  8 10:52:19 gyle girar-sh: arseny: -c task run 263468 --commit
> > Jan  8 14:28:08 gyle girar-builder: user=arseny task=263468 repo=sisyphus try=15 iter=2 status=DONE
> > (время указано в UTC)
> 
> Я ж говорю — недоразумение.

Второй --commit избыточен, см.
https://lists.altlinux.org/pipermail/devel/2019-March/207169.html

> Наверное, один из вариантов решения проблемы — привести документацию в
> соответствие с текущим поведением.

В том тексте, который считается документацией, параметр --commit вообще
описан?


-- 
ldv