From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gremlin@basealt.ru>
X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on
 sa.local.altlinux.org
X-Spam-Level: 
X-Spam-Status: No, score=-4.3 required=5.0 tests=ALL_TRUSTED,BAYES_00,
 HEADER_FROM_DIFFERENT_DOMAINS,RP_MATCHES_RCVD autolearn=unavailable
 autolearn_force=no version=3.4.1
Date: Tue, 12 Jan 2021 14:10:03 +0300
From: "Alexey V. Vissarionov" <gremlin@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Message-ID: <20210112111002.GB20621@altlinux.org>
References: <X/cVwC8AFP3JtwfF@cello> <20210107205220.GA1094824@lks.home>
 <X/d6sTGUOaQQ1UHx@cello> <20210111091327.GA1694278@lks.home>
 <X/wnjOlGi/hLRaEi@cello> <20210111104419.GA5476@altlinux.org>
 <X/wtJBOw7F96+2Uc@cello> <20210111105610.GB5476@altlinux.org>
 <20210111123020.GI5178@imap.altlinux.org>
 <20210112075517.0f87e328@tower>
MIME-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20210112075517.0f87e328@tower>
Subject: Re: [devel] hazardous girar builder
X-BeenThere: devel@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: ALT Linux Team development discussions <devel@lists.altlinux.org>
List-Id: ALT Linux Team development discussions <devel.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/devel>
List-Post: <mailto:devel@lists.altlinux.org>
List-Help: <mailto:devel-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Tue, 12 Jan 2021 11:10:08 -0000
Archived-At: <http://lore.altlinux.org/devel/20210112111002.GB20621@altlinux.org/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

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