[PATCH] virt-manager: Fix broken CPU details tab

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi everybody,
I've recompiled and installed newest libvirt and virt-manager and I found out that revision 1135 introduced the exception on CPU details tab by renaming get_type() to get_hv_type() so that is the fix for that.

If anything, please let me know.

Thanks,
Michal
# HG changeset patch
# User Michal Novotny <minovotn@xxxxxxxxxx>
# Date 1245074096 -7200
# Node ID 74f1f8309b139af9d84edfb1f0186c2306c0f332
# Parent  f9ad1d0c29d08bbb0ed94df9878d666904d018a2
Make Processor tab item in VM details show correctly

The Processor tab in VM details was not showing at all because of exception.
The exception occured everytime the tab was selected because of call to non
existing function get_type() that has been renamed to get_hv_type() in the
newest version of virt-manager's code by revision 1135.

diff -r f9ad1d0c29d0 -r 74f1f8309b13 src/virtManager/details.py
--- a/src/virtManager/details.py	Thu Jun 04 16:28:40 2009 -0400
+++ b/src/virtManager/details.py	Mon Jun 15 15:54:56 2009 +0200
@@ -945,7 +945,7 @@
         self.window.get_widget("state-host-cpus").set_text("%d" % self.vm.get_connection().host_active_processor_count())
         status = self.vm.status()
         if status in [ libvirt.VIR_DOMAIN_SHUTOFF, libvirt.VIR_DOMAIN_CRASHED ]:
-            cpu_max = self.vm.get_connection().get_max_vcpus(self.vm.get_type())
+            cpu_max = self.vm.get_connection().get_max_vcpus(self.vm.get_hv_type())
             self.window.get_widget("config-vcpus").get_adjustment().upper = cpu_max
             self.window.get_widget("state-vm-maxvcpus").set_text(str(cpu_max))
         else:
_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux