[libvirt] [PATCH] Add actions to virDomainLifecycle enum

Jim Fehlig jfehlig at novell.com
Fri Aug 20 20:39:42 UTC 2010


Jim Fehlig wrote:
> Eric Blake wrote:
>   
>> Hmm.  These two new values are only valid for on_crash, but I don't see
>> any code that rejects them for on_reboot or on_poweroff.  Do we need a
>> separate enum here, or do we just need to add better checking to the
>> remaining clients to detect enum values they can't support?
>>   
>>     
>
> Good point.  I moved these new options into a new enum and adjusted the
> code accordingly.
>
> V2 attached.
>   

Any comments on V2 of this patch?

Regards,
Jim

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-actions-to-virDomainLifecycle-enum.patch
Type: text/x-diff
Size: 14757 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100820/807e2d79/attachment-0001.bin>


More information about the libvir-list mailing list