Re: [devel] hazardous girar builder

["Dmitry V. Levin" <ldv@altlinux.org>]

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@.


-- 
ldv