Skip to content

Commit 7d95967

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Remove firewalld on Fedora 20 by default"
2 parents 3224f60 + 85ebb3a commit 7d95967

2 files changed

Lines changed: 16 additions & 1 deletion

File tree

lib/nova_plugins/functions-libvirt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ function install_libvirt {
3737
# and HP images used in the gate; rackspace has firewalld but hp
3838
# cloud doesn't. RHEL6 doesn't have firewalld either. So we
3939
# don't care if it fails.
40-
if is_fedora; then
40+
if is_fedora && is_package_installed firewalld; then
4141
sudo service firewalld restart || true
4242
fi
4343
}

tools/fixup_stuff.sh

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,21 @@ if is_fedora; then
103103
if selinuxenabled; then
104104
sudo setenforce 0
105105
fi
106+
107+
FORCE_FIREWALLD=$(trueorfalse False $FORCE_FIREWALLD)
108+
if [[ ${DISTRO} =~ (f19|f20) && $FORCE_FIREWALLD == "False" ]]; then
109+
# On Fedora 19 and 20 firewalld interacts badly with libvirt and
110+
# slows things down significantly. However, for those cases
111+
# where that combination is desired, allow this fix to be skipped.
112+
113+
# There was also an additional issue with firewalld hanging
114+
# after install of libvirt with polkit. See
115+
# https://bugzilla.redhat.com/show_bug.cgi?id=1099031
116+
if is_package_installed firewalld; then
117+
uninstall_package firewalld
118+
fi
119+
fi
120+
106121
fi
107122

108123
# RHEL6

0 commit comments

Comments
 (0)