From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ldv@altlinux.org>
Date: Thu, 3 Dec 2020 22:27:59 +0300
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Message-ID: <20201203192759.GD16739@altlinux.org>
References: <20201203181402.vvsliz3vxrshyahr@altlinux.org>
 <20201203182634.GC16060@altlinux.org>
 <20201203183000.tlu4x6kxoxfd6vyy@altlinux.org>
 <20201203185109.GD16129@altlinux.org>
 <20201203190402.7vr6tz27gffu76uf@altlinux.org>
 <20201203191652.GA16739@altlinux.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20201203191652.GA16739@altlinux.org>
Subject: Re: [devel] Q: Do not EPERM test-only tasks
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: Thu, 03 Dec 2020 19:27:59 -0000
Archived-At: <http://lore.altlinux.org/devel/20201203192759.GD16739@altlinux.org/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

On Thu, Dec 03, 2020 at 10:16:53PM +0300, Dmitry V. Levin wrote:
> On Thu, Dec 03, 2020 at 10:04:02PM +0300, Vitaly Chikunov wrote:
> > On Thu, Dec 03, 2020 at 09:51:09PM +0300, Dmitry V. Levin wrote:
> > > On Thu, Dec 03, 2020 at 09:30:00PM +0300, Vitaly Chikunov wrote:
> > > > On Thu, Dec 03, 2020 at 09:26:34PM +0300, Dmitry V. Levin wrote:
> > > > > On Thu, Dec 03, 2020 at 09:14:02PM +0300, Vitaly Chikunov wrote:
> > > > > > Hi,
> > > > > > 
> > > > > > Предлагаю test-only таски не переводить в состояние EPERM. Это позволит
> > > > > > легко отличить в почте результат run --test-only от run --commit!
> > > > > 
> > > > > А в какое состояние тогда их переводить?
> > > > 
> > > > Варианты: TESTED, TESTEPERM,
> > > 
> > > Я не понял, что предлагается, убрать acl check из test-only tasks?
> > 
> > Я не предлагал вариантов как это реализовать.
> > 
> > > Или игнорировать результат acl check для test-only tasks?
> > > 
> > > Я пока не понял, что именно не устраивает в нынешней ситуации.
> > > Задания в состояние EPERM, которые test-only, явно помечены как test-only.
> > 
> > Да, но все равно это путает. Почему-то у меня EPERM оверрайдит test-only
> > в голове. Впрочем, если считаешь, что пользы от этого нет то ОК, для
> > этого и обсуждение.
> 
> Допустим, это путает, а какой вариант будет путать меньше?
>  
> Если acl check будет происходить после того, как задание (не важно,
> test-only или нет) оказалось в состоянии TESTED, это будет удобнее?
>   
> Условно говоря, BUILDING -> TESTED -> CHECKPERM -> PENDING|EPERM
> (где CHECKPERM - это некое новое промежуточное, не обязательно видимое
> состояние)?

Допустим, test-only задание находится в состоянии TESTED, и автор этого
задания делает ему task run --commit, в результате чего задание переходит
в состояние EPERM.  Вопрос, какие уведомления по каким адресам и с каким
содержанием рассылать в этой ситуации?


-- 
ldv