ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Gleb Fotengauer-Malinovskiy <glebfm@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] inheritance check
Date: Tue, 15 Jun 2021 19:28:37 +0300
Message-ID: <YMjVNQ1JCxzEN+Ba@glebfm.cloud.tilaa.com> (raw)
In-Reply-To: <8804807.T7Z3S40VBb@zerg.malta.altlinux.ru>

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

On Tue, Jun 15, 2021 at 12:22:34PM +0300, Sergey V Turchin wrote:
> On Tuesday, 15 June 2021 12:12:27 MSK Konstantin Lepikhov wrote:
> 
> [...]
> > > > Более того, т.к. inheritance check у нас делается в конце сборки это
> > > > очень
> > > > ломает процесс, ведь сначала ждешь несколько часов пока все соберется,
> > > > ставишь зависимости, а потом бах, ваш пакет оказывается кто-то уже
> > > > собрал в
> > > > обход, начинайте все с начала.

К сожалению, для того, чтобы проверить inheritance check нужно знать, как
называется исходный пакет, а эта информация у нас сейчас появляется только
после сборки.
Совсем хорошо было бы, если бы проверки, которые можно делать
одновременно, делались бы одновременно, но это может потребовать довольно
большой переработки сборочницы.

> > > А если оно будет делаться в начале, то будет race, когда будут пытаться
> > > одновременно влететь 2-е разных сборки одного пакета.
> > 
> > но это же проблема реализации, правда?
> Скорее, проблема отсутствия реализации.
> 
> > Ничего не мешает сделать package lock
> Думаю, это отдельная фича. В багзилу есть смысл повесить, вдруг сделают.
> 
> > на стадии запуска таска и держать его , пока таск
> > RUNNING/PENDING/что там еще относительно статуса.

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

-- 
glebfm

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

  reply	other threads:[~2021-06-15 16:28 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 [this message]
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
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=YMjVNQ1JCxzEN+Ba@glebfm.cloud.tilaa.com \
    --to=glebfm@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