----- Original Message -----
From: "Shmuel Melamud" <smelamud(a)redhat.com>
To: "devel" <devel(a)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_4bOC6tL7KL...
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(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel