Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: Clean up treatment of /etc/exports ...................................................................... packaging: setup: Clean up treatment of /etc/exports Without this change, /etc/exports: - gets overwritten on setup if there is no /etc/exports.d - is added empty lines when edited by setup - is not restored to previous content on cleanup With this change: - If there is /etc/exports.d: -- Setup writes there a file with the iso domain path, which is removed on cleanup (if not changed since setup) -- Setup removes this path from /etc/exports if it's there (supposedly added by a previous legacy Setup), and this change is not reverted on cleanup - If there is only /etc/exports: -- Setup adds to it the path if it's not there already -- Cleanup reverts this change if the line was not changed since setup Bug-Url: https://bugzilla.redhat.com/1035511 Change-Id: I035817c048518f77fc666cc6d5212c8e46edd65d Signed-off-by: Yedidyah Bar David <didi(a)redhat.com&gt; --- M packaging/setup/plugins/ovirt-engine-setup/system/exportfs.py 1 file changed, 80 insertions(+), 45 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22318 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I035817c048518f77fc666cc6d5212c8e46edd65d 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: Kiril Nesenko <knesenko(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