[JDEV] Parent PID changed, exiting...

[David Waite]

You will need to:
1) patch the jabber source with Keith's patch
2) verify that your jabber.xml file(s) only have <stdout/> tags if they are
being launched by another jabberd process. Your main jabber.xml should not
contain any <stdout/> tags.

-David Waite

Ayite Gaba wrote:

> Let's me see if i get this right.
> To fix the "Parent PID changed" problem i have to modify the body of this
> function:  result base_stdout_heartbeat(void *arg)
>
> Is that it? I do not need to modify any other config files? if i still do
> have to do more modifications, please let me known which ones. Thanks a lot
> for your help, -- Ayite
>
> >From: Keith Minkler <kminkler at jabber.com>
> >Reply-To: jdev at jabber.org
> >To: jdev at jabber.org
> >Subject: Re: [JDEV] Parent PID changed, exiting...
> >Date: Mon, 12 Feb 2001 15:51:58 -0500
> >
> >On Mon, Feb 12, 2001 at 01:13:21PM -0600, Thomas Muldowney wrote:
> > > It's possible that it's legacy code from 1.2, but I'm not sure.  It was
> >missed
> > > by all of us as jabber.org runs in scripts to capture all the debug
> >output, we
> > > apologize.
> > >
> > > --temas
> >
> >this one has to do with linking two jabberd's together.. it was put in as a
> >way
> >for the child process to know when it is supposed to exit (when the parent
> >process
> >exits).
> >
> >you should ONLY be using a <stdout/> flag in a config file that is to be
> ><exec/>'d
> >inside of another PARENT jabberd.
> >
> >There is another section where it checks it's PID, and that is in dnsrv.. a
> >similar
> >instance is when dnsrv is used, the process forks itself, to run a child
> >dnsrv
> >module.  This child process also checks it's parent PID to see if it
> >changes to 1,
> >then exits if it does.
> >
> >This has worked for some time, and is only meant to happen in those two
> >very distinct
> >instances.. if you are running a single jabberd, and have a <stdout/> flag
> >inside
> >of it, then that is WRONG, and you need to take that <stdout/> flag out..
> >that is
> >only meant to be used while linking jabberd's together with a config
> >section such as:
> ><exec>./jabberd/jabberd -c ./jabber2.xml</exec>
> >
> >Please check your config for errors, thank you!
> >
> >also, if there is an easier/better way for a child process to check if its
> >parent
> >has died, I'd love to change this code... I've looked in a lot of places
> >for code
> >that accomplishes this, with no luck...  this was the best I could do in
> >short notice,
> >and it fixed all of our "zombie" jabberd problems.
> >
> >Keith Minkler
> >
> > >
> > > On Mon, Feb 12, 2001 at 09:34:04AM -0800, james rogers wrote:
> > > > In the jabber-1.4RC/jabberd/base direcrtory in a file called
> >base_stdout.c on line 51 is the function that was added for jabberd version
> >1.4 (i.e. not found in jabberd 1.2).  This is the function that is needless
> >and causing the server to exit when it's parent process is changed.o
> > > >
> > > > Here it is:
> > > >
> > > > result base_stdout_heartbeat(void *arg)
> > > > {
> > > >     static int parent = 0;
> > > >
> > > >     if(parent == 0) parent = getppid();
> > > >
> > > >     if(parent != getppid())
> > > >     {
> > > >         /* parent pid has changed, bail */
> > > >         log_alert("stdout","Parent PID has changed, Server Exiting");
> > > >         exit(1);
> > > >     }
> > > >
> > > >     return r_DONE;
> > > > }
> > > >
> > > > I am guessing that if we change it to the following that we will keep
> >running when we logout of the shell...
> > > >
> > > > result base_stdout_heartbeat(void *arg)
> > > > {
> > > >     return r_DONE;
> > > > }
> > > >
> > > > But before I do this in my own installation...  Why was this check
> >added?  What are we actually checking for here and is there another way to
> >do the same check?
> > > >
> > > >
> > > > ___________________________________________________
> > > > GO.com Mail
> > > > Get Your Free, Private E-mail at http://mail.go.com
> > > >
> > > >
> > > >
> > > > _______________________________________________
> > > > jdev mailing list
> > > > jdev at jabber.org
> > > > http://mailman.jabber.org/listinfo/jdev
> >
> >
> ><< attach3 >>
>
> _________________________________________________________________
> Get your FREE download of MSN Explorer at http://explorer.msn.com
>
> _______________________________________________
> jdev mailing list
> jdev at jabber.org
> http://mailman.jabber.org/listinfo/jdev