[devel] init scripts policy clarification

[devel] init scripts policy clarification

[Igor Vlasenko]

Господа,
хотелось бы прояснить наше полиси по InitScripts.
на
http://freesource.info/wiki/AltLinux/Sisyphus/devel/InitScripts
сказано:
> скрипты, которые входят в пакеты, которые используют 
> макросы %post_service и %preun_service, обязаны реализовывать 
> обработку первого аргумента condrestart и condstop соответственно;
Эти проверки (на condrestart и condstop) делаются сейчас в
repocop-unittest-init-condrestart.
При наличии %{post/preun}_service и отсутствии соответствующего
cond* выдается fail, и это правильно, так как %{post/preun}_service
не отработает.

Но что делать, если в скрипте нет вызова %{post/preun}_service
и одновременно нет целей condrestart и condstop?

Предупреждать ли (через warn) о желательности condrestart и condstop?

Или у нас этого требования нет, оставлять как есть, 
и считать, что майнтайнерам виднее?


-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine

Re: [devel] init scripts policy clarification

[Igor Vlasenko]

On Sat, Mar 08, 2008 at 07:57:17PM +0200, Igor Vlasenko wrote:
> Господа,
> хотелось бы прояснить наше полиси по InitScripts.
> на
> http://freesource.info/wiki/AltLinux/Sisyphus/devel/InitScripts
> сказано:
> > скрипты, которые входят в пакеты, которые используют 
> > макросы %post_service и %preun_service, обязаны реализовывать 
> > обработку первого аргумента condrestart и condstop соответственно;
> Эти проверки (на condrestart и condstop) делаются сейчас в
> repocop-unittest-init-condrestart.
> При наличии %{post/preun}_service и отсутствии соответствующего
> cond* выдается fail, и это правильно, так как %{post/preun}_service
> не отработает.
> 
> Но что делать, если в скрипте нет вызова %{post/preun}_service
> и одновременно нет целей condrestart и condstop?
> 
> Предупреждать ли (через warn) о желательности condrestart и condstop?
> 
> Или у нас этого требования нет, оставлять как есть, 
> и считать, что майнтайнерам виднее?
Наверно последнее. Никакие стандарты condrestart и condstop не
требуют, да и не следует умножать сущности без необходимости.

Так что уберу warn, если нет возражений.

-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine

Re: [devel] init scripts policy clarification

[Dmitry V. Levin]

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

On Sat, Mar 08, 2008 at 07:57:17PM +0200, Igor Vlasenko wrote:
> хотелось бы прояснить наше полиси по InitScripts.
> на
> http://freesource.info/wiki/AltLinux/Sisyphus/devel/InitScripts
> сказано:
> > скрипты, которые входят в пакеты, которые используют 
> > макросы %post_service и %preun_service, обязаны реализовывать 
> > обработку первого аргумента condrestart и condstop соответственно;
> Эти проверки (на condrestart и condstop) делаются сейчас в
> repocop-unittest-init-condrestart.
> При наличии %{post/preun}_service и отсутствии соответствующего
> cond* выдается fail, и это правильно, так как %{post/preun}_service
> не отработает.

Логично.

> Но что делать, если в скрипте нет вызова %{post/preun}_service
> и одновременно нет целей condrestart и condstop?
> 
> Предупреждать ли (через warn) о желательности condrestart и condstop?

Предупреждать.  Это не must, но should.


-- 
ldv

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

Re: [devel] init scripts policy clarification

[Igor Vlasenko]

On Sat, Mar 08, 2008 at 11:09:53PM +0300, Dmitry V. Levin wrote:
> > Но что делать, если в скрипте нет вызова %{post/preun}_service
> > и одновременно нет целей condrestart и condstop?
> > Предупреждать ли (через warn) о желательности condrestart и condstop?
> Предупреждать.  Это не must, но should.

Ok.

Желательно тогда на 
http://freesource.info/wiki/AltLinux/Sisyphus/devel/InitScripts
отразить, и перенести эту страничку в иерархию Policy.

И еще вопрос:
Что у нас с LSB?
http://refspecs.freestandards.org/LSB_3.1.0/LSB-Core-generic/LSB-Core-generic/iniscrptact.html

у нас нет  try-restart, force-reload.
try-restart это в точности condrestart, конечно.

-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine

Re: [devel] init scripts policy clarification

[Mikhail Gusarov]

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

Twas brillig at 22:35:34 08.03.2008 UTC+02 when Igor Vlasenko did gyre and gimble:

 IV> Желательно тогда на
 IV> http://freesource.info/wiki/AltLinux/Sisyphus/devel/InitScripts
 IV> отразить,

В чём состоит утверждение, которые стоит отразить? "Инитскриптам
рекомендуется иметь condrestart и condstop"?

 IV> и перенести эту страничку в иерархию Policy.

Ну, сейчас страничка слабо похожа на "политику": смесь "must", примеров,
пожеланий и рекомендаций на такую не тянет.

Вторая проблема: кто будет прочёсывать все существующие пакеты на
соответствие политике и исправлять? Невыполняемые policy - это даже
хуже, чем отстуствие таковых.

Если у вас есть желание привести страничку и инитскрипты в репозитории в
порядок - было бы замечательно. Я могу помочь с приведением черновика
политики в порядок (подозреваю, что та страница весьма и весьма
неполна).

-- 

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

Re: [devel] init scripts policy clarification

[Igor Vlasenko]

On Sun, Mar 09, 2008 at 02:48:12AM +0600, Mikhail Gusarov wrote:
>  IV> Желательно тогда на
>  IV> http://freesource.info/wiki/AltLinux/Sisyphus/devel/InitScripts
>  IV> отразить,
> В чём состоит утверждение, которые стоит отразить? "Инитскриптам
> рекомендуется иметь condrestart и condstop"?
да.
>  IV> и перенести эту страничку в иерархию Policy.
> Ну, сейчас страничка слабо похожа на "политику": смесь "must", примеров,
> пожеланий и рекомендаций на такую не тянет.

Почему?
Документ не в стиле решений ХVIII партсъезда?
Наоборот, такой стиль делает страничку читабельной.

Откуда же еще можно узнать о "must" -
Например:
>скрипты, которые входят в пакеты, которые используют макросы %post_service и %preun_service, обязаны реализовывать обработку первого аргумента condrestart и condstop соответственно;

Это либо знать, либо долго и мучительно ловить баг.
Такие вещи просто обязаны быть в Policy.

> Вторая проблема: кто будет прочёсывать все существующие пакеты на
> соответствие политике и исправлять? Невыполняемые policy - это даже
> хуже, чем отстуствие таковых.
Де-факто репокоп уже прочёсывает.
 
> Если у вас есть желание привести страничку и инитскрипты в репозитории в
> порядок - было бы замечательно. Я могу помочь с приведением черновика
> политики в порядок (подозреваю, что та страница весьма и весьма
> неполна).

Пока вообще пытаюсь понять, что в ней должно быть.
Например, уже третий раз интересуюсь,
поддерживает ли наш chkconfig X-Start-Before или нет?

Вообще, самые страшные полиси - это тайные полиси.

Тайные полиси - это такие полиси, которые
известны только майнтайнеру некоторой подсистемы,
но их несоблюдение (по неведению) другими майнтайнерами
порождает реально существующие неочевидные баги.

Полиси - это не мнение и не навязывание, 
это в первую очередь карта минного поля.

-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine

Re: [devel] init scripts policy clarification

[Mikhail Gusarov]

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

Twas brillig at 23:26:21 08.03.2008 UTC+02 when Igor Vlasenko did gyre and gimble:

 >> Ну, сейчас страничка слабо похожа на "политику": смесь "must",
 >> примеров, пожеланий и рекомендаций на такую не тянет.

 IV> Почему?  Документ не в стиле решений ХVIII партсъезда?

Потому что им невозможно пользоваться.

-- 

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

Re: [devel] init scripts policy clarification

[Dmitry V. Levin]

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

On Sat, Mar 08, 2008 at 11:26:21PM +0200, Igor Vlasenko wrote:
[...]
> Например, уже третий раз интересуюсь,
> поддерживает ли наш chkconfig X-Start-Before или нет?

Нет, не поддерживает.  И вообще, тот код в chkconfig, который
поддерживает LSB'шные секции, ещё кривее чем остальной.


-- 
ldv

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

Re: [devel] init scripts policy clarification

[Slava Semushin]

08.03.08, Igor Vlasenko<vlasenko / imath.kiev.ua> написал(а):
[...]
>  на http://freesource.info/wiki/AltLinux/Sisyphus/devel/InitScripts
>  сказано:
>  > скрипты, которые входят в пакеты, которые используют
>  > макросы %post_service и %preun_service, обязаны реализовывать
>  > обработку первого аргумента condrestart и condstop соответственно;

Я несколько раз прочитал это предложение и не могу его понять.
Объясните, пожалуйста, о каком первом аргументе идёт речь? И как его
обрабатывать? Когда я смотрел в последний раз template для
init-скрипта, то не видел там в обработчике condrestart/condstop
никаких упоминаний о первом аргументе..

[...]

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] init scripts policy clarification

[Dmitry V. Levin]

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

On Mon, Mar 10, 2008 at 01:29:38AM +0600, Slava Semushin wrote:
> 08.03.08, Igor Vlasenko<vlasenko / imath.kiev.ua> написал(а):
> [...]
> >  на http://freesource.info/wiki/AltLinux/Sisyphus/devel/InitScripts
> >  сказано:
> >  > скрипты, которые входят в пакеты, которые используют
> >  > макросы %post_service и %preun_service, обязаны реализовывать
> >  > обработку первого аргумента condrestart и condstop соответственно;
> 
> Я несколько раз прочитал это предложение и не могу его понять.
> Объясните, пожалуйста, о каком первом аргументе идёт речь? И как его
> обрабатывать? Когда я смотрел в последний раз template для
> init-скрипта, то не видел там в обработчике condrestart/condstop
> никаких упоминаний о первом аргументе..

Речь идёт о том, в каких случаях значение первого аргумента
startup-скрипта, равное condrestart или condstop, _должно_ обрабатываться.


-- 
ldv

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

Re: [devel] init scripts policy clarification

[Mikhail Gusarov]

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


Twas brillig at 23:19:58 09.03.2008 UTC+03 when Dmitry V. Levin did gyre and gimble:

[-- Attachment #2.1: Type: text/plain, Size: 351 bytes --]


 DVL> Речь идёт о том, в каких случаях значение первого аргумента
 DVL> startup-скрипта, равное condrestart или condstop, _должно_
 DVL> обрабатываться.

Переформулировал на страничке.

2php-coder: так понятнее?

-- 

[-- Attachment #2.2: Type: application/pgp-signature, Size: 188 bytes --]

Re: [devel] init scripts policy clarification

[Slava Semushin]

10.03.08, Mikhail Gusarov<dottedmag / dottedmag.net> написал(а):
>   DVL> Речь идёт о том, в каких случаях значение первого аргумента
>   DVL> startup-скрипта, равное condrestart или condstop, _должно_
>   DVL> обрабатываться.
>
>  Переформулировал на страничке.

Спасибо!

>  2php-coder: так понятнее?

Да, теперь полный clarification :)


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru