[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] Is it a problem that "a vm has been shutdown, but its pidfile has not been removed"



On Thu, Aug 22, 2013 at 09:34:55AM +0000, Wangyufei (A) wrote:
> Hello,
> when do steps as follows:
> 
> 1.  start a vm
> 
> 2.  restart libvirtd service
> 
> 3.  shutdown the vm
> I found the pidfile associated with the vm has not been removed.
> 
>     Analyzing related piece of code, I found when libvirtd service 
> restarted, the vm's _virDomainObj structure(named dom here) in
> libvirtd will be re-established according to the vm's state-file 
> or config-file, but its member of privateData.pidfile will not be
> reassigned value. By the way, qemudriver is used in my libvirt.
> 
> So when the vm is shutdown, its pidfile will not be removed,
> because the member of privateData.pidfile is NULL.
> 
> My questions are :
> 
> 1.  Do you think it is a problem when a vm has been shutdown,
>     but its pidfile has not been removed ?

It is fairly harmless, but it would be good practice to fix
this so it is removed.


> 2.  Do you think it is expected that the member of
>     privateData.pidfile's value missed after libvirtd restart?
> 
>     If it's a bug then I'll submit the patch to fix it.

Yes, that's a bug. When restarting libvirtd, we want to maintain
all the state we had before. So loosing the pidfile string is
a mistake.

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]