[virt-tools-list] [PATCH 2/4] Updated startdomain.py to use vmmDomain objects.
Darryl L. Pierce
dpierce at redhat.com
Mon May 23 21:47:21 UTC 2011
Broke out the start functionality to include unpausing a suspended
domain as well as starting up a stopped domain.
---
src/virtManagerTui/libvirtworker.py | 5 -----
src/virtManagerTui/startdomain.py | 11 +++++++----
2 files changed, 7 insertions(+), 9 deletions(-)
diff --git a/src/virtManagerTui/libvirtworker.py b/src/virtManagerTui/libvirtworker.py
index 0172fe1..225cf38 100644
--- a/src/virtManagerTui/libvirtworker.py
+++ b/src/virtManagerTui/libvirtworker.py
@@ -146,11 +146,6 @@ class LibvirtWorker:
if name in domains: return True
return False
- def create_domain(self, name):
- '''Creates the specified domain.'''
- domain = self.get_domain(name)
- domain.create()
-
def destroy_domain(self, name):
'''Destroys the specified domain.'''
domain = self.get_domain(name)
diff --git a/src/virtManagerTui/startdomain.py b/src/virtManagerTui/startdomain.py
index 468f591..b796645 100644
--- a/src/virtManagerTui/startdomain.py
+++ b/src/virtManagerTui/startdomain.py
@@ -30,7 +30,7 @@ class StartDomainConfigScreen(DomainListConfigScreen):
def get_elements_for_page(self, screen, page):
if page is self.LIST_PAGE:
- return self.get_domain_list_page(screen, started = False)
+ return self.get_domain_list_page(screen, created = False)
elif page is self.START_PAGE:
return self.get_start_domain_page(screen)
@@ -47,17 +47,20 @@ class StartDomainConfigScreen(DomainListConfigScreen):
if self.get_selected_domain() is not None:
domain = self.get_selected_domain()
try:
- self.get_libvirt().create_domain(domain)
+ if domain.is_unpauseable():
+ domain.resume()
+ else:
+ domain.startup()
return True
except Exception, error:
- errors.append("There was an error creating the domain: %s" % domain)
+ errors.append("There was an error creating the domain: %s" % domain.get_name())
errors.append(str(error))
else:
errors.append("You must first select a domain to start.")
def get_start_domain_page(self, screen):
grid = Grid(1, 1)
- grid.setField(Label("%s was successfully started." % self.get_selected_domain()), 0, 0)
+ grid.setField(Label("%s was successfully started." % self.get_selected_domain().get_name()), 0, 0)
return [grid]
def StartDomain():
--
1.7.4.4
More information about the virt-tools-list
mailing list