Change in ovirt-release[master]: [Node]: replace firewalld approach to set ovirthostnode as d...

sbonazzo at redhat.com sbonazzo at redhat.com
Thu Feb 25 12:16:29 UTC 2016


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 at 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 at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list