[ovirt-devel] New VM status functions - your comments?

[Eli Mesika]

----- Original Message -----
> From: "Shmuel Melamud" <smelamud at redhat.com>
> To: "devel" <devel at ovirt.org>
> Sent: Tuesday, December 15, 2015 3:41:21 PM
> Subject: [ovirt-devel] New VM status functions - your comments?
> 
> Hi!
> 
> https://docs.google.com/a/redhat.com/spreadsheets/d/1yb-JdTAGh_4bOC6tL7KLz1Q2VR-UkF3eMSo1o_kU3MM/edit?usp=sharing
> 
> In the document above I've summarized the changes I plan to make to
> reorganize VM status checks in the Engine.
> 
> The second page of the documents contains the list of the new functions and
> the statuses they will check.
> 
> On the first page I've collected all the places I'm planning to change -
> class name, method name and line number.

I am not sure line number is suitable here, code is changing and line numbers become not relevant, maybe a link to a gerrit draft with the change is better or a code fragment 


> "New method" column is the new
> check that will be put in this place.
> 
> Changing to new functions will often add/remove a state or two, that's why
> I'll be VERY pleased to see your comments. I've sorted the classes by topic,
> so it will be easy to look on those you're familiar with.
> 
> Ask yourself questions:
> 
> 1. If Up state is required, what about PoweringUp, PoweringDown,
> RebootInProgress?
> 2. If Up state is required, what about WaitingForLaunch? Paused?
> 3. What about migration?
> 4. What about hibernation?
> 5. If Down state is required, should it allow also ImageLocked and
> ImageIllegal?
> 6. If Down state is required, what about Suspended?
> 
> Thank you in advance.
> 
> Shmuel
> 
> _______________________________________________
> Devel mailing list
> Devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel