Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: refactored firewalld support ...................................................................... packaging: refactored firewalld support Added one service configuration template file for each service that should be allowed by firewalld under /usr/share/ovirt-engine/firewalld/*. During firewalld configuration the template files will be processed by processTemplate with provided ports. Then the service files will be copied to /etc/firewalld/services and enabled for the active zones. The ovirt-aio service will be enabled only if the plugin is installed and enabled. Change-Id: If6a60e1667182f926f399e22bbfbb939e7171cb0 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; --- M Makefile M packaging/fedora/setup/basedefs.py M packaging/fedora/setup/engine-setup.py D packaging/fedora/setup/firewalld.ovirt.xml M packaging/fedora/setup/output_messages.py M packaging/fedora/setup/plugins/all_in_one_100.py M packaging/fedora/spec/ovirt-engine.spec.in A packaging/firewalld/aio/ovirt-aio.xml.in A packaging/firewalld/base/ovirt-http.xml.in A packaging/firewalld/base/ovirt-https.xml.in A packaging/firewalld/base/ovirt-nfs.xml.in 11 files changed, 115 insertions(+), 44 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13638 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If6a60e1667182f926f399e22bbfbb939e7171cb0 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com&gt; Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com&gt; Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com&gt; Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com&gt; Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com&gt; Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt;