[devel] LSB initscript -- грабли

[devel] LSB initscript -- грабли

[Денис Смирнов]

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

В инитскрипте asterisk написано:

# Required-Start: $local_fs $network $remote_fs
# Should-Start: zaptel

Поясняю что это значит с моей точки зрения -- asterisk может работать без
zaptel, поэтому zaptel не является обязательной компонентой для установки,
и астериск не зависит от zaptel. Однако если zaptel используется, то
астериск должен быть запущен после zaptel.

Так вот. Эта конструкция -- не работает. Я сейчас пользуясь инсталлером от
legion@ добавляю в профиль zaptel и asterisk, и вижу симлинки в rc.3:

S50asterisk
S50zaptel

таким образом zaptel стартует после астериска.

Если я переношу зависимость на zaptel в required-start а не should-start,
то все становится замечательно.

Кто виноват и что делать?

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

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

Re: [devel] LSB initscript -- грабли

[Alexey Gladkov]

Денис Смирнов wrote:
> Кто виноват и что делать?

Виноват разумеется chkconfig. Он просто не знает про Should-Start.

$ fgrep -ri 'required-start' chkconfig-1.3.29/* |wc -l
3

$ fgrep -ri 'should-start' chkconfig-1.3.29/* |wc -l
0

P.S. В chkconfig поддержка LSB несколько странная (с моей точки зрения).

-- 
Rgrds, legion

Re: [devel] LSB initscript -- грабли

[Денис Смирнов]

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

On Tue, Jan 13, 2009 at 02:11:59AM +0300, Alexey Gladkov wrote:

AG> Виноват разумеется chkconfig. Он просто не знает про Should-Start.
AG> $ fgrep -ri 'required-start' chkconfig-1.3.29/* |wc -l
AG> 3
AG> $ fgrep -ri 'should-start' chkconfig-1.3.29/* |wc -l
AG> 0

no comments :(

AG> P.S. В chkconfig поддержка LSB несколько странная (с моей точки зрения).

Насколько я понимаю, он просто required-start трактует также как
should-start (по крайней мере после rpm -e zaptel и zaptel в
required-start у астериска -- chkconfig'у не поплохело).

Исходя из этого предлагаю запатчить chkconfig чтобы он should-start
трактовал также как required-start.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

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

Re: [devel] LSB initscript -- грабли

[Alexey Gladkov]

Денис Смирнов wrote:
> Насколько я понимаю, он просто required-start трактует также как
> should-start (по крайней мере после rpm -e zaptel и zaptel в
> required-start у астериска -- chkconfig'у не поплохело).

Вы знаете как и куда слать патчи :)

> Исходя из этого предлагаю запатчить chkconfig чтобы он should-start
> трактовал также как required-start.

По хорошему, chkconfig нужно переписать нафиг. Потому что в
большинстве своей работы это shell-скрипт на Си.

Если оставить за скобками реализацию LSB в этой программе и оставить
SysV функционал, то chkconfig вырождается в shell-скрипт на 12K ... я
это точно знаю. :)

-- 
Rgrds, legion

Re: [devel] LSB initscript -- грабли

[Денис Смирнов]

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

On Tue, Jan 13, 2009 at 12:43:50PM +0300, Alexey Gladkov wrote:

>> Насколько я понимаю, он просто required-start трактует также как
>> should-start (по крайней мере после rpm -e zaptel и zaptel в
>> required-start у астериска -- chkconfig'у не поплохело).
AG> Вы знаете как и куда слать патчи :)

(заглянув в chkconfig) я столько не выпью!

>> Исходя из этого предлагаю запатчить chkconfig чтобы он should-start
>> трактовал также как required-start.
AG> По хорошему, chkconfig нужно переписать нафиг. Потому что в
AG> большинстве своей работы это shell-скрипт на Си.
AG> Если оставить за скобками реализацию LSB в этой программе и оставить
AG> SysV функционал, то chkconfig вырождается в shell-скрипт на 12K ... я
AG> это точно знаю. :)

Грустно это. Сейчас уже о параллельном запуске думать по-хорошему надо...

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------


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

Re: [devel] LSB initscript -- грабли

[Alexey Gladkov]

Денис Смирнов wrote:
> (заглянув в chkconfig) я столько не выпью!

:)))

А я выпил, но LSB реализовать было пока лень. Тем более chkconfig
работает и никто не жалуется на него.

-- 
Rgrds, legion

Re: [devel] LSB initscript -- грабли

[Денис Смирнов]

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

On Sat, Jan 31, 2009 at 07:29:27PM +0300, Alexey Gladkov wrote:
>> (заглянув в chkconfig) я столько не выпью!
AG> :)))
AG> А я выпил, но LSB реализовать было пока лень. Тем более chkconfig
AG> работает и никто не жалуется на него.

Жалуются разве что на отсутствии поддержки LSB :)

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

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

Re: [devel] LSB initscript -- грабли

[Ivan Fedorov]

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

Alexey Gladkov <legion-u2l5PoMzF/Uox3rIn2DAYQ@public.gmane.org> writes:

> Денис Смирнов wrote:
>> (заглянув в chkconfig) я столько не выпью!
>
> :)))
>
> А я выпил, но LSB реализовать было пока лень. Тем более chkconfig
> работает и никто не жалуется на него.
Нуу... вообще-то в новоустановленных дистрах теперь не работает сервис
clock... тут правда надо вешать багу на startup, а не на chkconfig, но
обратная совместимость там точно хромает...

PS: кстати надо бы и репокоп наверное обучить, если ещё не...

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

Re: [devel] LSB initscript -- грабли

[Alexey Gladkov]

Денис Смирнов wrote:
> Жалуются разве что на отсутствии поддержки LSB :)

Как ты видел, поддержка там есть ... может не полная, но это другой
вопрос :)

В рамках моего for-fun, могу реализовать поддержку LSB:

http://git.altlinux.org/people/legion/public/chkconfig.sh.git

не думаю, что это будет вай-вай как сложно.

-- 
Rgrds, legion

Re: [devel] LSB initscript -- грабли

[Alexey Gladkov]

Ivan Fedorov wrote:
> Нуу... вообще-то в новоустановленных дистрах теперь не работает сервис
> clock... тут правда надо вешать багу на startup, а не на chkconfig, но
> обратная совместимость там точно хромает...

Если вы нашли багу, то её нужно исправлять. Повесьте багу с описанием
симптомов, чтобы не забылось.

-- 
Rgrds, legion

Re: [devel] LSB initscript -- грабли

[Dmitry V. Levin]

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

On Sun, Feb 01, 2009 at 04:12:55PM +0300, Alexey Gladkov wrote:
> Ivan Fedorov wrote:
> > Нуу... вообще-то в новоустановленных дистрах теперь не работает сервис
> > clock... тут правда надо вешать багу на startup, а не на chkconfig, но
> > обратная совместимость там точно хромает...
> 
> Если вы нашли багу, то её нужно исправлять. Повесьте багу с описанием
> симптомов, чтобы не забылось.

Вообще-то clock не является сервисом с точки зрения chkconfig (в
частности, его нельзя включить или выключить с помощью chkconfig), но
если вы действительно нашли багу, не держите её при себе,
поделитесь ей с багзиллой.


-- 
ldv

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

Re: [devel] LSB initscript -- грабли

[Alexey Gladkov]

Dmitry V. Levin wrote:
> если вы действительно нашли багу, не держите её при себе,
> поделитесь ей с багзиллой.

Угу ... именно поэтому и попросил повесить багу :)

-- 
Rgrds, legion

Re: [devel] LSB initscript -- грабли

[Igor Vlasenko]

On Sun, Feb 01, 2009 at 01:45:47AM +0300, Ivan Fedorov wrote:
> Нуу... вообще-то в новоустановленных дистрах теперь не работает сервис
> clock... тут правда надо вешать багу на startup, а не на chkconfig, но
> обратная совместимость там точно хромает...

> PS: кстати надо бы и репокоп наверное обучить, если ещё не...

С удовольствием, только хочу понять проблему.
по контексту я не понял :(
чтобы начать, надо иметь описание проблемы
и идею алгоритма проверки.



-- 

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

Re: [devel] LSB initscript -- грабли

[Ivan Fedorov]

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

Igor Vlasenko <vlasenko-mwWcnE6MXDzNsqkvetdkXg@public.gmane.org> writes:

> On Sun, Feb 01, 2009 at 01:45:47AM +0300, Ivan Fedorov wrote:
>> Нуу... вообще-то в новоустановленных дистрах теперь не работает сервис
>> clock... тут правда надо вешать багу на startup, а не на chkconfig, но
>> обратная совместимость там точно хромает...
>
>> PS: кстати надо бы и репокоп наверное обучить, если ещё не...
>
> С удовольствием, только хочу понять проблему.
> по контексту я не понял :(
> чтобы начать, надо иметь описание проблемы
> и идею алгоритма проверки.
Ну в инит-скрипте clock из пакета не прописаны какие-то поля, которые
нынче необходимы чтобы выполнить:

chkconfig --add clock

Чего именно там не хватает я уже не помню, а времени съездить и
посмотреть нету, потому и баги нет...

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

Re: [devel] LSB initscript -- грабли

[Igor Vlasenko]

On Sun, Feb 01, 2009 at 10:36:39PM +0300, Ivan Fedorov wrote:
> в инит-скрипте clock из пакета не прописаны какие-то поля, которые
> нынче необходимы чтобы выполнить:
> 
> chkconfig --add clock
> 
> Чего именно там не хватает я уже не помню, а времени съездить и
> посмотреть нету, потому и баги нет...

Да, действительно нет.
clock выглядит (и расположен) как инит-скрипт,
но в действительности вызывается из rc.sysinit

$ grep -r clock /etc/rc.d/rc.sysinit 
# Set the system clock (when /etc/adjtime is missing)
[ -s /etc/adjtime ] || /etc/init.d/clock start
# Set the system clock (when /etc/adjtime is present)
[ ! -s /etc/adjtime ] || /etc/init.d/clock start

IMHO, либо уж сделать его честным init script,
либо переложить в
/etc/rc.d/scripts.

К сожалению, не так легко отличить 
инит-скрипт с ошибкой от просто скрипта.

Если это тема важна, можно предложить полиси,
что в /etc/rc.d/init.d не должно быть ничего 
кроме init scriptов, 

библиотек функций
/etc/init.d/functions
/etc/init.d/functions-compat
/etc/init.d/outformat

и традиционных исключений.
/etc/init.d/halt
/etc/init.d/killall
/etc/init.d/local
/etc/init.d/single

Относительно такого полиси проверку написать можно.

-- 

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

Re: [devel] LSB initscript -- грабли

[Mikhail Gusarov]

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


Twas brillig at 22:11:43 01.02.2009 UTC+02 when vlasenko@imath.kiev.ua did gyre and gimble:

 IV> /etc/init.d/functions
 IV> /etc/init.d/functions-compat
 IV> /etc/init.d/outformat

А почему бы эти тоже не переложить в $PATH (оставив симлинки на какое-то
время в целях обратной совместимости)? Совместимость с редхатоидами в
этом месте важна? Диковато выглядит использование того же outformat.


-- 

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

Re: [devel] LSB initscript -- грабли

[Денис Смирнов]

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

On Sun, Feb 01, 2009 at 04:09:18PM +0300, Alexey Gladkov wrote:
>> Жалуются разве что на отсутствии поддержки LSB :)
AG> Как ты видел, поддержка там есть ... может не полная, но это другой
AG> вопрос :)
AG> В рамках моего for-fun, могу реализовать поддержку LSB:
AG> http://git.altlinux.org/people/legion/public/chkconfig.sh.git
AG> не думаю, что это будет вай-вай как сложно.

Если после этого твой chkconfig заменит нынешний -- то это будет просто
чудесно :)

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

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