Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: revert nfs config on cleanup ...................................................................... packaging: setup: revert nfs config on cleanup Revert /etc/sysconfig/nfs to its original contents on cleanup. Added a function editConfigContent in util to generalize edit loops that read conf files and do some changes to them. Used it to edit /etc/sysconfig/nfs as well as /etc/httpd/conf.d/ssl.conf and /var/lib/pgsql/data/postgresql.conf. Bug-Url: https://bugzilla.redhat.com/1013347 Change-Id: I74dd4c1556bd6479fcf10f85fcbe083f215e0854 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Signed-off-by: Yedidyah Bar David <didi(a)redhat.com&gt; --- M packaging/setup/ovirt_engine_setup/util.py M packaging/setup/plugins/ovirt-engine-setup/apache/ssl.py M packaging/setup/plugins/ovirt-engine-setup/provisioning/postgres.py M packaging/setup/plugins/ovirt-engine-setup/system/nfs.py 4 files changed, 203 insertions(+), 79 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20204 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74dd4c1556bd6479fcf10f85fcbe083f215e0854 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com&gt; Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com&gt; Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com&gt; Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server