Skip to content

Commit e9b343c

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "[Fedora] Devstack fails to install in firewalld less system"
2 parents 73ed488 + 83afcfe commit e9b343c

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

lib/nova_plugins/functions-libvirt

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,16 @@ function install_libvirt {
2929
install_package python-libguestfs
3030
fi
3131

32-
# Restart dbus/firewalld after install of libvirt to avoid a
33-
# problem with polkit, which libvirtd brings in. See
32+
# Restart firewalld after install of libvirt to avoid a problem
33+
# with polkit, which libvirtd brings in. See
3434
# https://bugzilla.redhat.com/show_bug.cgi?id=1099031
35+
36+
# Note there is a difference between F20 rackspace cloud images
37+
# and HP images used in the gate; rackspace has firewalld but hp
38+
# cloud doesn't. RHEL6 doesn't have firewalld either. So we
39+
# don't care if it fails.
3540
if is_fedora; then
36-
sudo service dbus restart
37-
sudo service firewalld restart
41+
sudo service firewalld restart || true
3842
fi
3943
}
4044

0 commit comments

Comments
 (0)