Bug 10407 - ugly initscript
Summary: ugly initscript
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: unfs3 (show other bugs)
Version: unstable
Hardware: all Linux
: P2 critical
Assignee: Anton Farygin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-12-12 18:18 MSK by Nick S. Grechukh
Modified: 2006-12-19 20:35 MSK (History)
2 users (show)

See Also:


Attachments
pidfile creation patch (2.17 KB, patch)
2006-12-12 18:19 MSK, Nick S. Grechukh
no flags Details | Diff
init.d/nfs (1.84 KB, text/plain)
2006-12-12 18:20 MSK, Nick S. Grechukh
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nick S. Grechukh 2006-12-12 18:18:30 MSK
вложенный в пакет инитскрипт сказочно крив. в аттаче нормальный скрипт и патч
unfsd для создания pidfile
Comment 1 Nick S. Grechukh 2006-12-12 18:19:26 MSK
Created attachment 1702 [details]
pidfile creation patch
Comment 2 Nick S. Grechukh 2006-12-12 18:20:08 MSK
Created attachment 1703 [details]
init.d/nfs
Comment 3 Nick S. Grechukh 2006-12-12 18:21:56 MSK
можно обойтись без патча, запускать как 

start_daemon --make-pidfile --pidfile "$PIDFILE" --lockfile "$LOCKFILE"
--expect-user root -- $NFSD -b $UNFS_OPTS

но лучше зопатчить
Comment 4 Michael Shigorin 2006-12-12 22:25:17 MSK
2 rider: сделай pls

unfs3 rem gvy
unfs3 add mike

а то промазал, и так оно и застряло

2 gns: спасибо, всё руки не доходили до сказочно кривого...
Comment 5 Nick S. Grechukh 2006-12-13 20:13:11 MSK
2 mike: пожалуйста, приходите еще :)
Comment 6 Michael Shigorin 2006-12-17 16:15:50 MSK
fixed in 0.9.16-alt1
Comment 7 Nick S. Grechukh 2006-12-17 16:22:24 MSK
за неделю активного использования обнаружил проблему: через некоторое время 
service nfs status начинает отвечать nfs is dead, but stale pidfile found. хот 
глазами видно, что это не так: содержимое /var/run/nfs.pid соответствует pid 
unfsd. проблема кажется решилась заменой в скрипте $NFSD на unfsd во всех 
вызовах start-stop-daemon
Comment 8 Michael Shigorin 2006-12-18 15:07:43 MSK
Видишь, пришёл ещё :)

Сделал чуть иначе в 0.9.16-alt2:
NFSD=unfsd
BINARY=/usr/sbin/$NFSD
Comment 9 Nick S. Grechukh 2006-12-19 20:20:47 MSK
вроде BINARY там только для [ -f $BINARY. а зачем проверять наличие каждого
бинарника, у нас же вроде как пакетная система ?
Comment 10 Michael Shigorin 2006-12-19 20:31:54 MSK
(смутившись) Ну мало ли. :)  По-ъорошему, надо бы [ -x ...
Comment 11 Nick S. Grechukh 2006-12-19 20:35:21 MSK
ну, имхо -x нужен только для поддержки legacy admins, делающих chmod -x
/usr/bin/service вместо chkconfig service off ;))