ALT Linux Team development discussions
 help / color / mirror / Atom feed
* [devel] acl и ООО
@ 2021-06-14 13:29 Konstantin Lepikhov
  2021-06-14 14:39 ` Andrey Savchenko
  2021-06-18 18:00 ` Vitaly Lipatov
  0 siblings, 2 replies; 27+ messages in thread
From: Konstantin Lepikhov @ 2021-06-14 13:29 UTC (permalink / raw)
  To: ALT Linux Devel Mailing List

Привет!

ООО в subj это не конкретное ооо б а некая сущность, которая лезет везде где
ей нужно не взирая на установленные правила.

$ ssh build.alt acl sisyphus spirv-tools show
spirv-tools     lakostis

$ rpm --lastchange spirv-tools
...
* Wed Apr 28 2021 Arseny Maslennikov <arseny@altlinux.org> 1:2020.4-alt2.1
- NMU: spec: adapted to new cmake macros.

т.е. есть Арсений и есть вся остальная шушера, которая мешается под
ногами и играет в какие-то свои игры. Мне просто интересно, зачем тогда
нужен acl, если у нас есть Арсений? )

-- 
WBR et al.


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] acl и ООО
  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-18 18:00 ` Vitaly Lipatov
  1 sibling, 1 reply; 27+ messages in thread
From: Andrey Savchenko @ 2021-06-14 14:39 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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

On Mon, 14 Jun 2021 15:29:31 +0200 Konstantin Lepikhov wrote:
> Привет!
> 
> ООО в subj это не конкретное ооо б а некая сущность, которая лезет везде где
> ей нужно не взирая на установленные правила.
> 
> $ ssh build.alt acl sisyphus spirv-tools show
> spirv-tools     lakostis
> 
> $ rpm --lastchange spirv-tools
> ...
> * Wed Apr 28 2021 Arseny Maslennikov <arseny@altlinux.org> 1:2020.4-alt2.1
> - NMU: spec: adapted to new cmake macros.
> 
> т.е. есть Арсений и есть вся остальная шушера, которая мешается под
> ногами и играет в какие-то свои игры. Мне просто интересно, зачем тогда
> нужен acl, если у нас есть Арсений? )

Арсений решил исправить cmake, в результате чего существенная часть
его пользователей сломалась. Что хуже всего, не было обсуждения на
devel грядущих изменений, а лишь post-factum тим уведомили
о сделанных изменениях.

Упомянутое изменение, по-видимому, является частью массового пакета
исправлений после улучшения cmake.

Best regards,
Andrew Savchenko

[-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] acl и ООО
  2021-06-14 14:39 ` Andrey Savchenko
@ 2021-06-14 18:01   ` Konstantin Lepikhov
  2021-06-15  8:10     ` Sergey V Turchin
  0 siblings, 1 reply; 27+ messages in thread
From: Konstantin Lepikhov @ 2021-06-14 18:01 UTC (permalink / raw)
  To: devel

Hi Andrey!

On 06/14/2021, at 05:39:24 PM you wrote:

> On Mon, 14 Jun 2021 15:29:31 +0200 Konstantin Lepikhov wrote:
> > Привет!
> > 
> > ООО в subj это не конкретное ооо б а некая сущность, которая лезет везде где
> > ей нужно не взирая на установленные правила.
> > 
> > $ ssh build.alt acl sisyphus spirv-tools show
> > spirv-tools     lakostis
> > 
> > $ rpm --lastchange spirv-tools
> > ...
> > * Wed Apr 28 2021 Arseny Maslennikov <arseny@altlinux.org> 1:2020.4-alt2.1
> > - NMU: spec: adapted to new cmake macros.
> > 
> > т.е. есть Арсений и есть вся остальная шушера, которая мешается под
> > ногами и играет в какие-то свои игры. Мне просто интересно, зачем тогда
> > нужен acl, если у нас есть Арсений? )
> 
> Арсений решил исправить cmake, в результате чего существенная часть
> его пользователей сломалась. Что хуже всего, не было обсуждения на
> devel грядущих изменений, а лишь post-factum тим уведомили
> о сделанных изменениях.
Я не подтверждал этот таск, почему за меня решили "протолкнуть" этот пакет в
сизиф?

> Упомянутое изменение, по-видимому, является частью массового пакета
> исправлений после улучшения cmake.
Нет, не является. Это самоуправство со стороны root_ldv:

http://git.altlinux.org/tasks/archive/done/_263/269879/logs/events.56.2.log
...
check-subtask-perms: #21700: spirv-tools: approved by superuser root_ldv

г-н ldv, прокомментируйте пожалуйста.

-- 
WBR et al.


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] acl и ООО
  2021-06-14 18:01   ` Konstantin Lepikhov
@ 2021-06-15  8:10     ` Sergey V Turchin
  2021-06-15  8:32       ` Konstantin Lepikhov
  0 siblings, 1 reply; 27+ messages in thread
From: Sergey V Turchin @ 2021-06-15  8:10 UTC (permalink / raw)
  To: devel

On Monday, 14 June 2021 21:01:53 MSK Konstantin Lepikhov wrote:

[...]
> http://git.altlinux.org/tasks/archive/done/_263/269879/logs/events.56.2.log
> ...
> check-subtask-perms: #21700: spirv-tools: approved by superuser root_ldv
Мне пришлось потом исправлять это так
http://git.altlinux.org/people/zerg/packages/?
p=ilmbase.git;a=commitdiff;h=e5d5e9492a100103b9ee9ad2de866b05fa2e7bdd
, чтобы можно было собирать в p9 сейчас, а не когда-нибудь потом.

> г-н ldv, прокомментируйте пожалуйста.


-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] acl и ООО
  2021-06-15  8:10     ` Sergey V Turchin
@ 2021-06-15  8:32       ` Konstantin Lepikhov
  2021-06-15  8:57         ` Sergey V Turchin
  0 siblings, 1 reply; 27+ messages in thread
From: Konstantin Lepikhov @ 2021-06-15  8:32 UTC (permalink / raw)
  To: devel

Hi Sergey!

On 06/15/2021, at 11:10:19 AM you wrote:

> On Monday, 14 June 2021 21:01:53 MSK Konstantin Lepikhov wrote:
> 
> [...]
> > http://git.altlinux.org/tasks/archive/done/_263/269879/logs/events.56.2.log
> > ...
> > check-subtask-perms: #21700: spirv-tools: approved by superuser root_ldv
> Мне пришлось потом исправлять это так
> http://git.altlinux.org/people/zerg/packages/?
> p=ilmbase.git;a=commitdiff;h=e5d5e9492a100103b9ee9ad2de866b05fa2e7bdd
> , чтобы можно было собирать в p9 сейчас, а не когда-нибудь потом.
> 
> > г-н ldv, прокомментируйте пожалуйста.
Да я уже все себе исправил. Проблема в том, что вот эти изменения прошли
абслютно незамеченными, т.к. я был в полной уверенности, что раз в acl
указан _один_ человек, никто в обход без уведомления никаких изменений
делать не будет.

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

-- 
WBR et al.


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] acl и ООО
  2021-06-15  8:32       ` Konstantin Lepikhov
@ 2021-06-15  8:57         ` Sergey V Turchin
  2021-06-15  9:12           ` Konstantin Lepikhov
  0 siblings, 1 reply; 27+ messages in thread
From: Sergey V Turchin @ 2021-06-15  8:57 UTC (permalink / raw)
  To: devel

On Tuesday, 15 June 2021 11:32:36 MSK Konstantin Lepikhov wrote:

[...]
> Проблема в том, что вот эти изменения прошли
> абслютно незамеченными, т.к. я был в полной уверенности, что раз в acl
> указан _один_ человек, никто в обход без уведомления никаких изменений
> делать не будет.
В той местячковой рассылке были все, чьи пакеты затронули, но я не предполагал 
что, это даёт возможность игнорировать ACL.

> Более того, т.к. inheritance check у нас делается в конце сборки это очень
> ломает процесс, ведь сначала ждешь несколько часов пока все соберется,
> ставишь зависимости, а потом бах, ваш пакет оказывается кто-то уже собрал в
> обход, начинайте все с начала.
А если оно будет делаться в начале, то будет race, когда будут пытаться 
одновременно влететь 2-е разных сборки одного пакета.

-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] acl и ООО
  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 15:20             ` [devel] acl и ООО Michael Shigorin
  0 siblings, 2 replies; 27+ messages in thread
From: Konstantin Lepikhov @ 2021-06-15  9:12 UTC (permalink / raw)
  To: devel

Hi Sergey!

On 06/15/2021, at 11:57:20 AM you wrote:

> On Tuesday, 15 June 2021 11:32:36 MSK Konstantin Lepikhov wrote:
> 
> [...]
> > Проблема в том, что вот эти изменения прошли
> > абслютно незамеченными, т.к. я был в полной уверенности, что раз в acl
> > указан _один_ человек, никто в обход без уведомления никаких изменений
> > делать не будет.
> В той местячковой рассылке были все, чьи пакеты затронули, но я не предполагал 
> что, это даёт возможность игнорировать ACL.
возможности дает root_ldv, в независимости от рассылки.

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

-- 
WBR et al.


^ permalink raw reply	[flat|nested] 27+ messages in thread

* [devel] inheritance check
  2021-06-15  9:12           ` Konstantin Lepikhov
@ 2021-06-15  9:22             ` Sergey V Turchin
  2021-06-15 16:28               ` Gleb Fotengauer-Malinovskiy
  2021-06-15 15:20             ` [devel] acl и ООО Michael Shigorin
  1 sibling, 1 reply; 27+ messages in thread
From: Sergey V Turchin @ 2021-06-15  9:22 UTC (permalink / raw)
  To: devel

On Tuesday, 15 June 2021 12:12:27 MSK Konstantin Lepikhov wrote:

[...]
> > > Более того, т.к. inheritance check у нас делается в конце сборки это
> > > очень
> > > ломает процесс, ведь сначала ждешь несколько часов пока все соберется,
> > > ставишь зависимости, а потом бах, ваш пакет оказывается кто-то уже
> > > собрал в
> > > обход, начинайте все с начала.
> > 
> > А если оно будет делаться в начале, то будет race, когда будут пытаться
> > одновременно влететь 2-е разных сборки одного пакета.
> 
> но это же проблема реализации, правда?
Скорее, проблема отсутствия реализации.

> Ничего не мешает сделать package lock
Думаю, это отдельная фича. В багзилу есть смысл повесить, вдруг сделают.

> на стадии запуска таска и держать его , пока таск
> RUNNING/PENDING/что там еще относительно статуса.


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] acl и ООО
  2021-06-15  9:12           ` Konstantin Lepikhov
  2021-06-15  9:22             ` [devel] inheritance check Sergey V Turchin
@ 2021-06-15 15:20             ` Michael Shigorin
  1 sibling, 0 replies; 27+ messages in thread
From: Michael Shigorin @ 2021-06-15 15:20 UTC (permalink / raw)
  To: devel

On Tue, Jun 15, 2021 at 11:12:27AM +0200, Konstantin Lepikhov wrote:
> но это же проблема реализации, правда? Ничего не мешает сделать
> package lock на стадии запуска таска и держать его, пока таск
> RUNNING/PENDING/что там еще относительно статуса.

...кроме здравого смысла: у такого подхода будут свои косяки,
причём ты же бы совершенно справедливо их и раскритиковал.
Начиная с возможности дружеского DoS или даже DDoS.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  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
                                   ` (2 more replies)
  0 siblings, 3 replies; 27+ messages in thread
From: Gleb Fotengauer-Malinovskiy @ 2021-06-15 16:28 UTC (permalink / raw)
  To: ALT Linux Team development discussions

[-- 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 --]

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  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
  2 siblings, 1 reply; 27+ messages in thread
From: Vladimir D. Seleznev @ 2021-06-15 16:59 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Tue, Jun 15, 2021 at 07:28:37PM +0300, Gleb Fotengauer-Malinovskiy wrote:
> 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 нужно знать, как
> называется исходный пакет, а эта информация у нас сейчас появляется только
> после сборки.

Проверить имя исходного пакета можно собрав только исходный пакет,
обычно они быстро собираются. Возможно, достаточно просто модифицировать
сборочницу, сначала собирая только source-пакет, делая все проверки на
inheritance-check и sourceid, а потом уже собирать actual пакет.

-- 
   WBR,
   Vladimir D. Seleznev


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-15 16:59                 ` Vladimir D. Seleznev
@ 2021-06-15 17:04                   ` Gleb Fotengauer-Malinovskiy
  0 siblings, 0 replies; 27+ messages in thread
From: Gleb Fotengauer-Malinovskiy @ 2021-06-15 17:04 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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

On Tue, Jun 15, 2021 at 04:59:26PM +0000, Vladimir D. Seleznev wrote:
> On Tue, Jun 15, 2021 at 07:28:37PM +0300, Gleb Fotengauer-Malinovskiy wrote:
> > 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, нужно знать, как
> > называется исходный пакет, а эта информация у нас сейчас появляется только
> > после сборки.
> 
> Проверить имя исходного пакета можно собрав только исходный пакет,
> обычно они быстро собираются. Возможно, достаточно просто модифицировать
> сборочницу, сначала собирая только source-пакет, делая все проверки на
> inheritance-check и sourceid, а потом уже собирать actual пакет.

Идея собирать исходные пакеты отдельно от бинарных уже возникала раньше.

Например, без этого очень сложно решать проблему, которая возникла в связи
с внедрением disttag и проявляется в noarch-подпакетах в случае, когда на
некоторых архитектурах меняется сборочная среда (а на некоторых
архитектурах не меняется).

-- 
glebfm

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

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-15 16:28               ` Gleb Fotengauer-Malinovskiy
  2021-06-15 16:59                 ` Vladimir D. Seleznev
@ 2021-06-15 17:45                 ` Konstantin Lepikhov
  2021-06-16  7:40                 ` Sergey V Turchin
  2 siblings, 0 replies; 27+ messages in thread
From: Konstantin Lepikhov @ 2021-06-15 17:45 UTC (permalink / raw)
  To: devel

Hi Gleb!

On 06/15/2021, at 07:28:37 PM you wrote:

<skip>
> Я так и не понял, о каких гонках и о каких летающих релизах идёт речь.
> Я не вижу никакого race в случае переставления inheritance check на более
> раннюю стадию.
> С другой стороны, мне кажется логичным, что проверка, для которой есть
> штатный способ отключения, делается последней.
"логичное" и "штатное" это для тебя, мне эти вещи абсолютно неочевидны по
причинам, озвученным выше. Проблема нашей сборочницы - она устарела в
плане иноформативности, т.к. расчитана только на email как единственное
средство общения и аудита (ну и общение голосом). А т.к. рассылкой и
bugzilla теперь уже никто не пользуется (я сужу по трафику в рассылке и
комментариям на ошибки), то и понять что там происходит людям со стороны
уже затруднительно.

Даже мне после стольких лет работы, собрать пакет это "спуститься в забой"
после всяких gitlab'ов и github и визуальных MR'ов.

-- 
WBR et al.


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-15 16:28               ` Gleb Fotengauer-Malinovskiy
  2021-06-15 16:59                 ` Vladimir D. Seleznev
  2021-06-15 17:45                 ` Konstantin Lepikhov
@ 2021-06-16  7:40                 ` Sergey V Turchin
  2021-06-16  8:10                   ` Andrey Savchenko
  2 siblings, 1 reply; 27+ messages in thread
From: Sergey V Turchin @ 2021-06-16  7:40 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Tuesday, 15 June 2021 19:28:37 MSK Gleb Fotengauer wrote:

[...]
> Я не вижу никакого race в случае переставления inheritance check на более
> раннюю стадию.
2 одновременно собирающихся пакета с одинаковым NAME-EVR, но разными 
изменениями должны оба её пройти успешно. Или позже ещё что-то сработает?

[...]

-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-16  7:40                 ` Sergey V Turchin
@ 2021-06-16  8:10                   ` Andrey Savchenko
  2021-06-16  8:33                     ` Sergey V Turchin
  0 siblings, 1 reply; 27+ messages in thread
From: Andrey Savchenko @ 2021-06-16  8:10 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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

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. Кто первый
встал — того и тапки.

Best regards,
Andrew Savchenko

[-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-16  8:10                   ` Andrey Savchenko
@ 2021-06-16  8:33                     ` Sergey V Turchin
  2021-06-16 10:50                       ` Denis G. Samsonenko
                                         ` (2 more replies)
  0 siblings, 3 replies; 27+ messages in thread
From: Sergey V Turchin @ 2021-06-16  8:33 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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 одним из них, но к концу 
сборки у обоих проверки уже сделаны. Разве не так?

-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  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:37                       ` Andrey Savchenko
  2 siblings, 1 reply; 27+ messages in thread
From: Denis G. Samsonenko @ 2021-06-16 10:50 UTC (permalink / raw)
  To: ALT Linux Team development discussions

Hi!

> > Очевидно, один из них не пройдёт inheritance check. Кто первый
> > встал — того и тапки.
> Мне совсем не очевидно. Ведь, новые коммиты ни одного из них ещё не влиты в
> gears. Обе проверки должны пройти до обновления gears одним из них, но к концу
> сборки у обоих проверки уже сделаны. Разве не так?

Можно наверно два раза проверять, до сборки и перед коммитом.

-- 
Всего доброго,

Денис.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-16  8:33                     ` Sergey V Turchin
  2021-06-16 10:50                       ` Denis G. Samsonenko
@ 2021-06-16 11:09                       ` Dmitry V. Levin
  2021-06-16 11:47                         ` Sergey V Turchin
  2021-06-19 13:44                         ` Anton V. Boyarshinov
  2021-06-16 11:37                       ` Andrey Savchenko
  2 siblings, 2 replies; 27+ messages in thread
From: Dmitry V. Levin @ 2021-06-16 11:09 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-16  8:33                     ` Sergey V Turchin
  2021-06-16 10:50                       ` Denis G. Samsonenko
  2021-06-16 11:09                       ` Dmitry V. Levin
@ 2021-06-16 11:37                       ` Andrey Savchenko
  2021-06-17 15:50                         ` Sergey V Turchin
  2 siblings, 1 reply; 27+ messages in thread
From: Andrey Savchenko @ 2021-06-16 11:37 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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

On Wed, 16 Jun 2021 11:33:47 +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 одним из них, но к концу 
> сборки у обоих проверки уже сделаны. Разве не так?

Ну один соберётся чуть вперёд другого, попадёт в репозиторий
и сломает проверку наследования второму пакету.

Best regards,
Andrew Savchenko

[-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-16 10:50                       ` Denis G. Samsonenko
@ 2021-06-16 11:40                         ` Sergey V Turchin
  0 siblings, 0 replies; 27+ messages in thread
From: Sergey V Turchin @ 2021-06-16 11:40 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Wednesday, 16 June 2021 13:50:29 MSK Denis G wrote:
> Hi!
> 
> 
> > > Очевидно, один из них не пройдёт inheritance check. Кто первый
> > > встал — того и тапки.
> > 
> > Мне совсем не очевидно. Ведь, новые коммиты ни одного из них ещё не влиты
> > в
 gears. Обе проверки должны пройти до обновления gears одним из них, но
> > к концу сборки у обоих проверки уже сделаны. Разве не так?
> 
> 
> Можно наверно два раза проверять, до сборки и перед коммитом.
Да. По крайней мере, это очевидно.

> 
> -- 
> Всего доброго,
> 
> Денис.
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel


-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  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
  1 sibling, 1 reply; 27+ messages in thread
From: Sergey V Turchin @ 2021-06-16 11:47 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Wednesday, 16 June 2021 14:09:50 MSK Dmitry V wrote:
> 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 у него актуальный.
Получается, в общем случае на 2-й итерации сборки этих заданий наконец-то 
кому-то из них повезёт, а 2-й отправится на 3-ю пересборку и уже обломается?

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

[...]

-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-16 11:47                         ` Sergey V Turchin
@ 2021-06-16 11:52                           ` Dmitry V. Levin
  0 siblings, 0 replies; 27+ messages in thread
From: Dmitry V. Levin @ 2021-06-16 11:52 UTC (permalink / raw)
  To: ALT Devel discussion list

On Wed, Jun 16, 2021 at 02:47:33PM +0300, Sergey V Turchin wrote:
> On Wednesday, 16 June 2021 14:09:50 MSK Dmitry V wrote:
> > 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 у него актуальный.
> Получается, в общем случае на 2-й итерации сборки этих заданий наконец-то 
> кому-то из них повезёт, а 2-й отправится на 3-ю пересборку и уже обломается?

Номера итераций и пересборок может отличаться от этого примера, но в целом
картина именно такая.

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

Видимо, с момента появления файла /tasks/$N/gears/$I/package .


-- 
ldv


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-16 11:37                       ` Andrey Savchenko
@ 2021-06-17 15:50                         ` Sergey V Turchin
  0 siblings, 0 replies; 27+ messages in thread
From: Sergey V Turchin @ 2021-06-17 15:50 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Wednesday, 16 June 2021 14:37:41 MSK Andrey Savchenko wrote:
> On Wed, 16 Jun 2021 11:33:47 +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 одним из них,
> > но к концу сборки у обоих проверки уже сделаны. Разве не так?
> 
> 
> Ну один соберётся
> чуть вперёд другого
Слишком громко сказано для заданий больше чем из одн одного пакета.
Учитывайте это всегда, пожалуйста, при обсуждении проблем сборочницы, т.к. у 
меня эта ситуация может занять более одних суток запросто.

> , попадёт в репозиторий
> и сломает проверку наследования второму пакету.
> 
> Best regards,
> Andrew Savchenko


-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] acl и ООО
  2021-06-14 13:29 [devel] acl и ООО Konstantin Lepikhov
  2021-06-14 14:39 ` Andrey Savchenko
@ 2021-06-18 18:00 ` Vitaly Lipatov
  1 sibling, 0 replies; 27+ messages in thread
From: Vitaly Lipatov @ 2021-06-18 18:00 UTC (permalink / raw)
  To: ALT Linux Devel Mailing List

Упомянутые изменения стали возможны благодаря слишком широкому ACL на 
cmake, за что приношу свои извинения.

$ ssh gear.alt acl sisyphus cmake show
gear.alt:sisyphus ACL for cmake	lav zerg @qa @everybody

Konstantin Lepikhov писал 14.6.21 16:29:
> Привет!
> 
> ООО в subj это не конкретное ооо б а некая сущность, которая лезет 
> везде где
> ей нужно не взирая на установленные правила.
> 
> $ ssh build.alt acl sisyphus spirv-tools show
> spirv-tools     lakostis
> 
> $ rpm --lastchange spirv-tools
> ...
> * Wed Apr 28 2021 Arseny Maslennikov <arseny@altlinux.org> 
> 1:2020.4-alt2.1
> - NMU: spec: adapted to new cmake macros.
> 
> т.е. есть Арсений и есть вся остальная шушера, которая мешается под
> ногами и играет в какие-то свои игры. Мне просто интересно, зачем тогда
> нужен acl, если у нас есть Арсений? )

-- 
С уважением,
Виталий Липатов,
ALT Linux Team


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-16 11:09                       ` Dmitry V. Levin
  2021-06-16 11:47                         ` Sergey V Turchin
@ 2021-06-19 13:44                         ` Anton V. Boyarshinov
  2021-06-19 14:07                           ` Michael Shigorin
  1 sibling, 1 reply; 27+ messages in thread
From: Anton V. Boyarshinov @ 2021-06-19 13:44 UTC (permalink / raw)
  To: Dmitry V. Levin; +Cc: ALT Linux Team development discussions

В Wed, 16 Jun 2021 14:09:50 +0300
"Dmitry V. Levin" <ldv@altlinux.org> пишет:

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

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


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-19 13:44                         ` Anton V. Boyarshinov
@ 2021-06-19 14:07                           ` Michael Shigorin
  2021-06-20 23:31                             ` Dmitry V. Levin
  0 siblings, 1 reply; 27+ messages in thread
From: Michael Shigorin @ 2021-06-19 14:07 UTC (permalink / raw)
  To: devel

On Sat, Jun 19, 2021 at 04:44:24PM +0300, Anton V. Boyarshinov wrote:
> > Таким образом, inheritance check технически допустимо проводить на любой
> > стадии обработки задания, когда стало известно имя исходного пакета.
> Лично мне не очень важно, на какой стадии проверяется
> inheritance check, но было бы удобно, если бы его можно было бы
> отключить до того, как проверка показала эту необходимость.

+1

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info


^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [devel] inheritance check
  2021-06-19 14:07                           ` Michael Shigorin
@ 2021-06-20 23:31                             ` Dmitry V. Levin
  0 siblings, 0 replies; 27+ messages in thread
From: Dmitry V. Levin @ 2021-06-20 23:31 UTC (permalink / raw)
  To: devel

On Sat, Jun 19, 2021 at 05:07:00PM +0300, Michael Shigorin wrote:
> On Sat, Jun 19, 2021 at 04:44:24PM +0300, Anton V. Boyarshinov wrote:
> > > Таким образом, inheritance check технически допустимо проводить на любой
> > > стадии обработки задания, когда стало известно имя исходного пакета.
> > Лично мне не очень важно, на какой стадии проверяется
> > inheritance check, но было бы удобно, если бы его можно было бы
> > отключить до того, как проверка показала эту необходимость.
> 
> +1

Filed a bug: https://bugzilla.altlinux.org/40252


-- 
ldv


^ permalink raw reply	[flat|nested] 27+ messages in thread

end of thread, other threads:[~2021-06-20 23:31 UTC | newest]

Thread overview: 27+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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
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

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