incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chip Childers <chip.child...@sungard.com>
Subject Re: [ACS41][Patch Request]
Date Wed, 13 Mar 2013 00:42:34 GMT
On Tue, Mar 12, 2013 at 06:39:09PM -0600, Marcus Sorensen wrote:
> This addresses CLOUDSTACK-600.  Please get buyoff from Wido if
> possible, since it's his patch. I've reviewed and tested it, but I
> want to make sure I'm not wrong in assuming that this should be in 4.1
> 
> commit 5dfcd309f10e5bd6a918f7fdff3f44a3dff2374a
> Author: Wido den Hollander <wido@widodh.nl>
> Date:   Thu Feb 7 22:58:20 2013 +0100
> 
>     agent: Do not define domains persistent in libvirt
> 
>     We used to define domains persistent in libvirt, which caused XML
> definitions
>     to stay there after a reboot of the hypervisor.
> 
>     We however don't do anything with those already defined domains,
> actually, we wipe
>     all defined domains when starting the agent.
> 
>     Some users however reported that libvirt started these domains
> after a reboot
>     before the CloudStack agent was started.
> 
>     By starting domains from the XML description and not defining them
> we prevent
>     them from ever being stored in libvirt.
>

Wido - I'll let you comment before we apply this.

Mime
View raw message