Change in ovirt-release[master]: node: Fix cp and add firewalld dependency

[sbonazzo at redhat.com]

Sandro Bonazzola has submitted this change and it was merged.

Change subject: node: Fix cp and add firewalld dependency
......................................................................


node: Fix cp and add firewalld dependency

Currently, in the installation/update cp -u will update the
existing backup file which can cause misleading conf.
To avoid this case, first remove any existing
backup file before copying the current firewalld conf to backup.

In addition a dependency on firewalld is added, to ensure
that firewalld completed all logic around the configuration
files and we can modify them to our needs.
Previously firewalld could be instaleld after the host-node pkg
which would lead to situation that host-node modified the firewalld.conf
but firewalld overwrote it again, once it got installed.

Change-Id: I4beaeb0b77d9bccea2ba0d6de84973d6c2c8efb5
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M ovirt-release-master/ovirt-release-master.spec.in
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Fabian Deutsch: Verified
  Sandro Bonazzola: Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/54037
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4beaeb0b77d9bccea2ba0d6de84973d6c2c8efb5
Gerrit-PatchSet: 3
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: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>