Oved Ourfali has submitted this change and it was merged. Change subject: packaging: setup: do not override answer file when applying post install removal ...................................................................... packaging: setup: do not override answer file when applying post install removal OVESETUP_SYSTEM/nfsConfigEnabled should be removed from post install file. we need to find a way to do that in generic way, but for time being we provide local solution. the last method we have done to set None at boot stage is good, however the answer file is configuration just like the post install file, we have no way to distinguish. without setting the value at boot stage, the issue would have been resolved at the 2nd setup run, but we want it to be solved at the next setup run. to mark this legacy state a new postinstall variable was added, if it is None then we know that we have legacy post-install and we do not look into the content of OVESETUP_SYSTEM/nfsConfigEnabled. Bug-Url: https://bugzilla.redhat.com/1056777 Change-Id: I907380d1d39c33ffcc43fdd49192a5096a000539 Signed-off-by: Alon Bar-Lev <alonbl@redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-setup/system/nfs.py 2 files changed, 43 insertions(+), 11 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26492 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I907380d1d39c33ffcc43fdd49192a5096a000539 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server