[libvirt] [PATCH 5/5] Update logging documentation

Amy Griffis amy.griffis at hp.com
Thu Aug 6 15:32:34 UTC 2009


Daniel P. Berrange wrote:  [Thu Aug 06 2009, 10:07:26AM EDT]
> > > I guess that should be "service libvirtd restart" ?
> > > 
> > > (i.e. all reload does is issue a SIGHUP)
> > 
> > Oh, I forgot I was looking at debian. :-)  Their start script for
> > libvirtd actually does a restart under the reload. Generalizing
> > on "restart" for the docs would certainly cover all cases...
> 
> Ewww! They are separate actions for a reason ! Reload (via SIGHUP) is a
> much less dangerous operation, than a full stop/start based restart. At
> this point in time, a full restart should only be required for changes
> to the daemon/driver config files libvirtd.conf, or qemu.conf. Everything
> else (guest configs, etc) should just work with a mere SIGHUP. Someone
> might like to fix 'reload' in Debian's initscripts...

Scratch that, I must have made a typo when I wrote down what
I had tried. And then looked at the script this morning when
I was not quite awake. I just looked again, and it doesn't do
anything funky.

Amy




More information about the libvir-list mailing list