ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] inheritance check
Date: Wed, 16 Jun 2021 14:09:50 +0300
Message-ID: <20210616110950.GA8728@altlinux.org> (raw)
In-Reply-To: <4309887.R56niFO833@zerg.malta.altlinux.ru>

On Wed, Jun 16, 2021 at 11:33:47AM +0300, Sergey V Turchin wrote:
> On Wednesday, 16 June 2021 11:10:57 MSK Andrey Savchenko wrote:
> > On Wed, 16 Jun 2021 10:40:48 +0300 Sergey V Turchin wrote:
> > > On Tuesday, 15 June 2021 19:28:37 MSK Gleb Fotengauer wrote:
> > > [...]
> > > > Я не вижу никакого race в случае переставления inheritance check на
> > > > более
> > > > раннюю стадию.
> > > 
> > > 2 одновременно собирающихся пакета с одинаковым NAME-EVR, но разными 
> > > изменениями должны оба её пройти успешно. Или позже ещё что-то сработает?
> > 
> > Очевидно, один из них не пройдёт inheritance check. Кто первый
> > встал — того и тапки.
> Мне совсем не очевидно. Ведь, новые коммиты ни одного из них ещё не влиты в 
> gears. Обе проверки должны пройти до обновления gears одним из них, но к концу 
> сборки у обоих проверки уже сделаны. Разве не так?

Задания для одного репозитория могут обрабатываться независимо друг от
друга, но коммитятся они последовательно, причём закоммичено может быть
только такое задание, которое собрано на текущем состоянии репозитория,
соответственно, результат inheritance check у него актуальный.

Таким образом, inheritance check технически допустимо проводить на любой
стадии обработки задания, когда стало известно имя исходного пакета.

Существует альтернативная идея, которую продвигает Игорь, суть которой
заключается в том, чтобы увеличить количество закоммиченных заданий в
единицу времени за счёт отказа от актуализации сборок и проверок заданий,
и в его модели такие race действительно происходят.


-- 
ldv


  parent reply	other threads:[~2021-06-16 11:09 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-14 13:29 [devel] acl и ООО Konstantin Lepikhov
2021-06-14 14:39 ` Andrey Savchenko
2021-06-14 18:01   ` Konstantin Lepikhov
2021-06-15  8:10     ` Sergey V Turchin
2021-06-15  8:32       ` Konstantin Lepikhov
2021-06-15  8:57         ` Sergey V Turchin
2021-06-15  9:12           ` Konstantin Lepikhov
2021-06-15  9:22             ` [devel] inheritance check Sergey V Turchin
2021-06-15 16:28               ` Gleb Fotengauer-Malinovskiy
2021-06-15 16:59                 ` Vladimir D. Seleznev
2021-06-15 17:04                   ` Gleb Fotengauer-Malinovskiy
2021-06-15 17:45                 ` Konstantin Lepikhov
2021-06-16  7:40                 ` Sergey V Turchin
2021-06-16  8:10                   ` Andrey Savchenko
2021-06-16  8:33                     ` Sergey V Turchin
2021-06-16 10:50                       ` Denis G. Samsonenko
2021-06-16 11:40                         ` Sergey V Turchin
2021-06-16 11:09                       ` Dmitry V. Levin [this message]
2021-06-16 11:47                         ` Sergey V Turchin
2021-06-16 11:52                           ` Dmitry V. Levin
2021-06-19 13:44                         ` Anton V. Boyarshinov
2021-06-19 14:07                           ` Michael Shigorin
2021-06-20 23:31                             ` Dmitry V. Levin
2021-06-16 11:37                       ` Andrey Savchenko
2021-06-17 15:50                         ` Sergey V Turchin
2021-06-15 15:20             ` [devel] acl и ООО Michael Shigorin
2021-06-18 18:00 ` Vitaly Lipatov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210616110950.GA8728@altlinux.org \
    --to=ldv@altlinux.org \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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