mirror of
https://github.com/virt-manager/virt-manager.git
synced 2024-12-25 23:21:45 +03:00
details: Fix max vcpus updating
Use the same refresh handler for VM state and config changes (since max vcpus can be tied to state change as well).
This commit is contained in:
parent
2e220a017f
commit
50ccdf251e
@ -414,9 +414,9 @@ class vmmDetails(gobject.GObject):
|
||||
})
|
||||
|
||||
# Deliberately keep all this after signal connection
|
||||
self.vm.connect("status-changed", self.update_widget_states)
|
||||
self.vm.connect("status-changed", self.refresh_vm_state)
|
||||
self.vm.connect("config-changed", self.refresh_vm_state)
|
||||
self.vm.connect("resources-sampled", self.refresh_resources)
|
||||
self.vm.connect("config-changed", self.refresh_vm_info)
|
||||
self.window.get_widget("hw-list").get_selection().connect(
|
||||
"changed",
|
||||
self.hw_selected)
|
||||
@ -427,13 +427,12 @@ class vmmDetails(gobject.GObject):
|
||||
finish_img = gtk.image_new_from_stock(gtk.STOCK_ADD,
|
||||
gtk.ICON_SIZE_BUTTON)
|
||||
self.window.get_widget("add-hardware-button").set_image(finish_img)
|
||||
self.update_widget_states(self.vm, self.vm.status())
|
||||
|
||||
self.populate_hw_list()
|
||||
self.repopulate_boot_list()
|
||||
|
||||
self.hw_selected()
|
||||
self.refresh_vm_info()
|
||||
self.refresh_vm_state()
|
||||
|
||||
|
||||
def show(self):
|
||||
@ -443,7 +442,7 @@ class vmmDetails(gobject.GObject):
|
||||
self.topwin.present()
|
||||
|
||||
self.engine.increment_window_counter()
|
||||
self.update_widget_states(self.vm, self.vm.status())
|
||||
self.refresh_vm_state()
|
||||
|
||||
def close(self, ignore1=None, ignore2=None):
|
||||
fs = self.window.get_widget("details-menu-view-fullscreen")
|
||||
@ -953,8 +952,12 @@ class vmmDetails(gobject.GObject):
|
||||
self.window.get_widget("details-menu-run").get_child().set_label(text)
|
||||
self.window.get_widget("control-run").set_label(strip_text)
|
||||
|
||||
def update_widget_states(self, vm, status, ignore=None):
|
||||
self.toggle_toolbar(self.window.get_widget("details-menu-view-toolbar"))
|
||||
def refresh_vm_state(self, ignore1=None, ignore2=None, ignore3=None):
|
||||
vm = self.vm
|
||||
status = self.vm.status()
|
||||
|
||||
self.toggle_toolbar(
|
||||
self.window.get_widget("details-menu-view-toolbar"))
|
||||
|
||||
destroy = vm.is_destroyable()
|
||||
run = vm.is_runable()
|
||||
@ -996,8 +999,13 @@ class vmmDetails(gobject.GObject):
|
||||
|
||||
self.console.update_widget_states(vm, status)
|
||||
|
||||
self.window.get_widget("overview-status-text").set_text(self.vm.run_status())
|
||||
self.window.get_widget("overview-status-icon").set_from_pixbuf(self.vm.run_status_icon())
|
||||
self.window.get_widget("overview-status-text").set_text(
|
||||
self.vm.run_status())
|
||||
self.window.get_widget("overview-status-icon").set_from_pixbuf(
|
||||
self.vm.run_status_icon())
|
||||
|
||||
details = self.window.get_widget("details-pages")
|
||||
self.page_refresh(details.get_current_page())
|
||||
|
||||
|
||||
#############################
|
||||
@ -1046,8 +1054,6 @@ class vmmDetails(gobject.GObject):
|
||||
else:
|
||||
self.emit("action-resume-domain", self.vm.get_connection().get_uri(), self.vm.get_uuid())
|
||||
|
||||
self.update_widget_states(self.vm, self.vm.status())
|
||||
|
||||
def control_vm_run(self, src):
|
||||
self.emit("action-run-domain", self.vm.get_connection().get_uri(), self.vm.get_uuid())
|
||||
|
||||
@ -1685,10 +1691,6 @@ class vmmDetails(gobject.GObject):
|
||||
self.get_hw_selection(HW_LIST_COL_TYPE) == HW_LIST_TYPE_STATS):
|
||||
self.refresh_stats_page()
|
||||
|
||||
def refresh_vm_info(self, ignore=None):
|
||||
details = self.window.get_widget("details-pages")
|
||||
self.page_refresh(details.get_current_page())
|
||||
|
||||
def page_refresh(self, page):
|
||||
if page != PAGE_DETAILS:
|
||||
return
|
||||
|
Loading…
Reference in New Issue
Block a user