Re: [devel] hazardous girar builder

[Arseny Maslennikov <arseny@altlinux.org>]

[-- Attachment #1: Type: text/plain, Size: 4494 bytes --]

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)

Я ж говорю — недоразумение.

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]