* Re: [devel] hazardous girar builder
@ 2021-01-11 12:30 11% ` Arseny Maslennikov
0 siblings, 0 replies; 2+ results
From: Arseny Maslennikov @ 2021-01-11 12:30 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- Attachment #1: Type: text/plain, Size: 1104 bytes --]
On Mon, Jan 11, 2021 at 12:32:57PM +0100, Alexey Gladkov wrote:
> On Mon, Jan 11, 2021 at 02:25:43PM +0300, Anton Farygin wrote:
> > > [...]
> > > > изменение
> > > > постоянно-запоминаемых параметров задания опциями --test-only/--commit к
> > > > task run не отражено.
> > > А может сделать всегда "test-only", если "--commit" явно не указан?
> > > Или для "run" и "--commit" могут потребоваться разные права?
> > >
> > да, мне тоже кажется что статус --commit должен сбрасываться на test-only
> > сразу после FAIL/EPERM сборки.
>
> +1
>
> Вот это точно!
>
+1.
Таким образом, test_only фактически прекратит быть независимым свойством
задания и станет свойством итерации, что со стороны пользователя
сборочницы вполне естественно.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [relevance 11%]
* Re: [devel] hazardous girar builder
@ 2021-01-12 11:10 6% ` Alexey V. Vissarionov
0 siblings, 0 replies; 2+ results
From: Alexey V. Vissarionov @ 2021-01-12 11:10 UTC (permalink / raw)
To: ALT Linux Team development discussions
On 2021-01-12 10:55:17 +0300, Anton V. Boyarshinov wrote:
>> Поддерживаю предложение rider@ насчёт взвода test-only
>> для следующей сборки при достижении FAILED/EPERM.
> Только сейчас тестер, администратор репозитория или
> владелец acl может выдать approve на задание и отправить
> его в репозиторий, а в таком случае не сможет, так как
> переключать статус может только владелец задания (и это
> логично).
И это неправильно. Параметр "--commit" должен обозначать
"отправить в репу" и ничего больше, а test-only должно
быть свойством итерации, а не задания.
То есть, когда мейнтейнер создает сборочное задание, оно
может либо просто не собраться (в этом случае все совсем
просто и понятно), либо собраться до состояния "готово к
отправке в репу". Все, сборка на этом завершена - у нас
есть готовые пакеты. Но выполнение задания продолжается.
Задание с собранными пакетами, в свою очередь, может
1. либо упереться в ACL и остановиться на EPERM,
2. либо упереться в отсутствие "--commit",
3. либо обработать параметр "--commit" и уехать в репу.
Да, очередность проверок важна!
В первом случае его может пропихнуть кто-то, указанный в
ACL. Во втором - создатель (и, пожалуй, кто угодно из ACL).
А в третьем, очевидно, и пихать ничего не надо.
--
Alexey V. Vissarionov
gremlin ПРИ altlinux ТЧК org; +vii-cmiii-ccxxix-lxxix-xlii
GPG: 0D92F19E1C0DC36E27F61A29CD17E2B43D879005 @ hkp://keys.gnupg.net
^ permalink raw reply [relevance 6%]
Results 1-2 of 2 | reverse
2021-01-07 14:08 [devel] [#263468] EPERM (try 14) llvm-common.git=11.0.0-alt2 srpm=llvm11.0-11.0.0-alt2.src.rpm Arseny Maslennikov
2021-01-07 20:52 ` Konstantin Lepikhov
2021-01-07 21:18 ` Arseny Maslennikov
2021-01-11 9:13 ` Konstantin Lepikhov
2021-01-11 10:25 ` [devel] hazardous girar builder (was: Re: [#263468] EPERM (try 14) llvm-common.git=11.0.0-alt2 srpm=llvm11.0-11.0.0-alt2.src.rpm) Arseny Maslennikov
2021-01-11 10:44 ` [devel] hazardous girar builder Dmitry V. Levin
2021-01-11 10:49 ` Arseny Maslennikov
2021-01-11 10:56 ` Dmitry V. Levin
2021-01-11 11:08 ` Arseny Maslennikov
2021-01-11 11:22 ` Sergey V Turchin
2021-01-11 11:25 ` Anton Farygin
2021-01-11 11:32 ` Alexey Gladkov
2021-01-11 12:30 11% ` Arseny Maslennikov
2021-01-11 12:30 ` Michael Shigorin
2021-01-12 7:55 ` Anton V. Boyarshinov
2021-01-12 11:10 6% ` Alexey V. Vissarionov
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git