Bug 10407

Summary: ugly initscript
Product: Sisyphus Reporter: Nick S. Grechukh <gns>
Component: unfs3Assignee: Anton Farygin <rider>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: critical    
Priority: P2 CC: mike, vvk
Version: unstable   
Hardware: all   
OS: Linux   
Attachments:
Description Flags
pidfile creation patch
none
init.d/nfs none

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 ;))