Browse code

Remove Fedora/libvirt workaround

An updated libvirt package to address [1] is in Fedora 20 now, so we
don't need this work-around. Modify the comments of the other part of
the work-around (restart of services) and link to the most relevant
bug

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1098376

Change-Id: I47fba7b4f273162c2af1e37902a512041449750b

Ian Wienand authored on 2014/05/27 12:58:12
Showing 1 changed files
... ...
@@ -29,19 +29,10 @@ function install_libvirt {
29 29
         install_package python-libguestfs
30 30
     fi
31 31
 
32
-    # workaround for
33
-    # https://bugzilla.redhat.com/show_bug.cgi?id=1098376; if we see
34
-    # the empty Xen proc file then remove the xen/libxl plugin
35
-    # shared-libraries (yum remove would uninstall libvirt due to
36
-    # dependencies, so let's avoid that...)
37
-    if is_fedora && [ -f /proc/xen/capabilities ] && \
38
-        [ $(stat -c '%s' /proc/xen/capabilities) -eq 0 ]; then
39
-        sudo rm -f /usr/lib64/libvirt/connection-driver/libvirt_driver_libxl.so
40
-        sudo rm -f /usr/lib64/libvirt/connection-driver/libvirt_driver_xen.so
41
-
42
-        # another bug requires these to be restarted to avoid
43
-        # potential hang of libvirtd
44
-        # https://bugzilla.redhat.com/show_bug.cgi?id=1098866
32
+    # Restart dbus/firewalld after install of libvirt to avoid a
33
+    # problem with polkit, which libvirtd brings in.  See
34
+    # https://bugzilla.redhat.com/show_bug.cgi?id=1099031
35
+    if is_fedora; then
45 36
         sudo service dbus restart
46 37
         sudo service firewalld restart
47 38
     fi