Sandro Bonazzola has submitted this change and it was merged.
Change subject: [Node]: replace firewalld approach to set ovirthostnode as default zone
......................................................................
[Node]: replace firewalld approach to set ovirthostnode as default zone
systemctl status firewalld always return 0 in anaconda/kickstart so it's
not possible to determine if we should use firewalld-cmd or
firewalld-offline-cmd. As it's supported as well
the manual installation of release pkgs via dnf/yum this patch
will copy current firewalld.conf for later restore and use
sed cmd to update the zone.
For more info:
Running in chroot, ignoring request -- exit code 0
https://bugzilla.redhat.com/show_bug.cgi?id=1110675
Change-Id: Idef65e4125044fee920c7865203d329ce90efb6f
Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com>
---
M ovirt-release-master/ovirt-release-master.spec.in
1 file changed, 17 insertions(+), 16 deletions(-)
Approvals:
Fabian Deutsch: Looks good to me, but someone else must approve
Douglas Schilling Landgraf: Verified
Sandro Bonazzola: Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/53153
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idef65e4125044fee920c7865203d329ce90efb6f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-release
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>