virt-manager: Stop suppressing all warnings

This is hiding useful stuff. It was enabled once upon a time because
it was too noisy but let's see if it gets bad like the past

Signed-off-by: Cole Robinson <crobinso@redhat.com>

virtManager/asyncjob.py

@@ -9,7 +9,6 @@ import traceback
 
 from gi.repository import Gdk
 from gi.repository import GLib
-from gi.repository import Vte
 
 import libvirt
 
@@ -319,6 +318,7 @@ class vmmAsyncJob(vmmGObjectUI):
 
     @idle_wrapper
     def details_enable(self):
+        from gi.repository import Vte
         self._details_widget = Vte.Terminal()
         self.widget("details-box").add(self._details_widget)
         self._details_widget.set_visible(True)

virtManager/engine.py

@@ -83,8 +83,8 @@ class vmmEngine(vmmGObject):
 
 
     def _cleanup(self):
-        if self._timer is not None:
-            GLib.source_remove(self._timer)
+        # self._timer should be automatically cleaned up
+        pass
 
 
     #################

virtManager/systray.py

@@ -18,6 +18,8 @@ from .connmanager import vmmConnectionManager
 
 try:
     # pylint: disable=ungrouped-imports
+    import gi
+    gi.require_version('AppIndicator3', '0.1')
     from gi.repository import AppIndicator3
 except Exception:  # pragma: no cover
     AppIndicator3 = None

virtManager/virtmanager.py

@@ -22,13 +22,6 @@ from virtinst import log
 
 from .lib.testmock import CLITestOptionsClass
 
-# This is massively heavy handed, but I can't figure out any way to shut
-# up the slew of gtk deprecation warnings that clog up our very useful
-# stdout --debug output. Of course we could drop use of deprecated APIs,
-# but it's a serious quantity of churn
-import warnings  # pylint: disable=wrong-import-order
-warnings.simplefilter("ignore")
-
 try:
     gi.check_version("3.22.0")
 except (ValueError, AttributeError):  # pragma: no cover