Bug 93522

On Sat, Feb 18, 2006 at 10:19:34AM +0000, Daniel Gerzo wrote:
> 
> >Number:         93522
> >Category:       docs
> >Synopsis:       [patch] use rc.d script in example describing how to restart mountd in nework-servers chapter
> >Confidential:   no
> >Severity:       non-critical
> >Priority:       low
> >Responsible:    freebsd-doc
> >State:          open
> >Quarter:        
> >Keywords:       
> >Date-Required:
> >Class:          doc-bug
> >Submitter-Id:   current-users
> >Arrival-Date:   Sat Feb 18 10:20:08 GMT 2006
> >Closed-Date:
> >Last-Modified:
> >Originator:     Daniel Gerzo
> >Release:        FreeBSD 6.1-PRERELEASE i386
> >Organization:
> rulez.sk
> >Environment:
> System: 6.1-PRERELEASE #0: Wed Feb 15 02:22:30 CET 2006 i386
> >Description:
> Teach users to get used to use the rc.d scripts since they are nice and
> easy-to-use, so reflect them in the handbok.
> >How-To-Repeat:
> >Fix:
> --- chapter.sgml.diff begins here ---
> --- /usr/home/danger/doc/en_US.ISO8859-1/books/handbook/network-servers/chapter.sgml.orig	Thu Feb 16 09:44:00 2006
> +++ /usr/home/danger/doc/en_US.ISO8859-1/books/handbook/network-servers/chapter.sgml	Sat Feb 18 10:15:15 2006
> @@ -776,10 +776,10 @@
>        <para>You must restart
>          <application>mountd</application> whenever you modify
>          <filename>/etc/exports</filename> so the changes can take effect.
> -        This can be accomplished by sending the HUP signal
> -        to the <command>mountd</command> process:</para>
> +        This can be accomplished by invoking of the
> +	<filename>/etc/rc.d/mountd</filename> script:</para>
>  
> -      <screen>&prompt.root; <userinput>kill -HUP `cat /var/run/mountd.pid`</userinput></screen>
> +      <screen>&prompt.root; <userinput>/etc/rc.d/mountd restart</userinput></screen>
>  


Sending HUP to a process is not the same as restarting is.  In my book,
this specific section is fine as it is.

- Christian

-- 
Christian Brueffer	chris@unixpages.org	brueffer@FreeBSD.org
GPG Key:	 http://people.freebsd.org/~brueffer/brueffer.key.asc
GPG Fingerprint: A5C8 2099 19FF AACA F41B  B29B 6C76 178C A0ED 982D

Hello Christian,

Saturday, February 18, 2006, 12:18:50 PM, you wrote:

> On Sat, Feb 18, 2006 at 10:19:34AM +0000, Daniel Gerzo wrote:

> Sending HUP to a process is not the same as restarting is.  In my book,
> this specific section is fine as it is.

then reload might be sufficient

> - Christian

-- 
Sincerely,
   Daniel Gerzo

Comment 3 Simon L. B. Nielsen 2006-02-18 12:40:03 UTC

On 2006.02.18 14:52:31 +0300, Dmitry Morozovsky wrote:
> On Sat, 18 Feb 2006, Christian Brueffer wrote:
> 
> CB>  > -      <screen>&prompt.root; <userinput>kill -HUP `cat /var/run/mountd.pi=
> CB>  d`</userinput></screen>
> CB>  > +      <screen>&prompt.root; <userinput>/etc/rc.d/mountd restart</userinp=
> CB>  ut></screen>
> CB>  > =20
> CB>  
> CB>  Sending HUP to a process is not the same as restarting is.  In my book,
> CB>  this specific section is fine as it is.
> 
> Yes. I think `/etc/rc.d/mountd reload' whould be used; however, this is 


Sending a HUP signal manually seems a bit silly to me now that we have
the nice way of doing it with the rc.d script.  Being able to simply
write /etc/rc.d/foo restart/reload is one of the things I really miss
when using 4.X systems.

Also since, for not quite as experienced people, remembering the
difference in how to reload/restart different daemons can be hard.  At
least I remember in my early UNIX days that I had to check the head of
/etc/inetd.conf about how to reload it each time.

Just my 0.02c.

> depending on mountd_enable rc.conf setting which should be done explicitly.


How about saying something like:

	See the rc.d chapter for more information on controling services.

Or something like that?  Then we don't have to duplicate the
explanation about _enable="YES".

-- 
Simon L. Nielsen

On 18/2/06 10:19, "Daniel Gerzo" <danger@rulez.sk> wrote:

> +        This can be accomplished by invoking of the
> + <filename>/etc/rc.d/mountd</filename> script:</para>

That should be either "invocation of the", or "invoking the", not what
appears above.

Ceri
-- 
That must be wonderful!  I don't understand it at all.
                                      -- Moliere

On Sat, 18 Feb 2006, Simon L. Nielsen wrote:

SLN> > CB>  > -      <screen>&prompt.root; <userinput>kill -HUP `cat /var/run/mountd.pi=
SLN> > CB>  d`</userinput></screen>
SLN> > CB>  > +      <screen>&prompt.root; <userinput>/etc/rc.d/mountd restart</userinp=
SLN> > CB>  ut></screen>
SLN> > CB>  > =20
SLN> > CB>  
SLN> > CB>  Sending HUP to a process is not the same as restarting is.  In my book,
SLN> > CB>  this specific section is fine as it is.
SLN> > 
SLN> > Yes. I think `/etc/rc.d/mountd reload' whould be used; however, this is 
SLN> 
SLN> Sending a HUP signal manually seems a bit silly to me now that we have
SLN> the nice way of doing it with the rc.d script.  Being able to simply
SLN> write /etc/rc.d/foo restart/reload is one of the things I really miss
SLN> when using 4.X systems.
SLN> 
SLN> Also since, for not quite as experienced people, remembering the
SLN> difference in how to reload/restart different daemons can be hard.  At
SLN> least I remember in my early UNIX days that I had to check the head of
SLN> /etc/inetd.conf about how to reload it each time.
SLN> 
SLN> Just my 0.02c.
SLN> 
SLN> > depending on mountd_enable rc.conf setting which should be done explicitly.
SLN> 
SLN> How about saying something like:
SLN> 
SLN> 	See the rc.d chapter for more information on controling services.
SLN> 
SLN> Or something like that?  Then we don't have to duplicate the
SLN> explanation about _enable="YES".

Agreed.

Something like attached patch? Please fell free to reword and commit.

Sincerely,
D.Marck                                     [DM5020, MCK-RIPE, DM3-RIPN]
------------------------------------------------------------------------
*** Dmitry Morozovsky --- D.Marck --- Wild Woozle --- marck@rinet.ru ***
------------------------------------------------------------------------

Comment 6 Vanilla I. Shu 2006-02-21 15:11:46 UTC

State Changed
From-To: open->closed

Committed, thanks.

Comment 7 Vanilla I. Shu 2006-02-21 15:13:32 UTC

State Changed
From-To: closed->open

I closed the wrong pr, now open it again.

Hello,

   I hope that this is fine now ;-)

-- 
Sincerely,
   Daniel Gerzo

Comment 9 Giorgos Keramidas 2006-05-10 20:23:44 UTC

State Changed
From-To: open->closed

Committed a slightly modified version of the final patch, 
which mentions both HUP signal, rc.d 'reload' option of mountd 
rc.d script and includes a pointer to linkend="configtuning-rcd". 

Thanks for the patches :) 

Comment 10 Giorgos Keramidas 2006-05-10 20:23:44 UTC

Responsible Changed
From-To: freebsd-doc->keramida