Hmm..
Instead the 'name' should change from 'refresh' to 'restart'..

While  you are here, can you also fix the second part mentioned in the bug 
to use :kill...

~Kini

-- 
Sun Microsystems - India Engineering Centre
http://blogs.sun.com/jkini
-------------- next part --------------
On Thu, Dec 13, 2007 at 02:47:44PM +0530, ashwin wrote:
> Hello guys.
> 
> Requesting a sponsor for bug # 6365813
> My contributer's agreement is OS0144
> 
> Bug Details:
> ssh corrects confusion with more confusion
> network/ssh's refresh method is "/lib/svc/method/sshd restart", which seems
> uncouth, since refresh methods should not stop the daemon.  But, in
> fact, /lib/svc/method/sshd restart just runs
> "/usr/bin/kill -HUP `/usr/bin/cat $PIDFILE`", which doesn't stop the daemon
> at all.  The argument should be changed to "refresh".
> 
> 
> changes in the file usr/src/cmd/ssh/etc/ssh.xml
> The argument to /lib/svc/method/sshd should be refresh instead of restart
> 
> Patch:
> 
> --- ssh.xml.orig      Thu Dec 13 14:38:32 2007
> +++ ssh.xml   Thu Dec 13 14:38:59 2007
> @@ -117,7 +117,7 @@
>       <exec_method
>               type='method'
>               name='refresh'
> -             exec='/lib/svc/method/sshd restart'
> +             exec='/lib/svc/method/sshd refresh'
>               timeout_seconds='60' />

I'm a little concerned that you have just fixed the synopsis.
/lib/svc/method/sshd doesn't actually have a "refresh" case.

Ceri
-- 
That must be wonderful!  I don't understand it at all.
                                                  -- Moliere
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
URL: 
<http://mail.opensolaris.org/pipermail/request-sponsor/attachments/20071213/6f564b16/attachment.bin>
-------------- next part --------------
_______________________________________________
request-sponsor mailing list
request-sponsor at opensolaris.org

Reply via email to