Anonymous Coward has submitted this change and it was merged. Change subject: core: WipeAfterDelete defaults to true in GUI, but false in REST API (#845466) ...................................................................... core: WipeAfterDelete defaults to true in GUI, but false in REST API (#845466) WipeAfterDelete in gui default to false for File Storage Domains and for Block storage domains the value is defined by SANWipeAfterDelete configuration value. The backend logic has been changed to mimic this behaviour. Change-Id: I8cb77b017282733d72ff66af499b576674674e01 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=845466 Signed-off-by: Ravi Nori <rnori@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/WipeAfterDeleteUtils.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/BaseDisk.java 3 files changed, 34 insertions(+), 4 deletions(-) Approvals: Michael Pasternak: Looks good to me, approved Objections: Omer Frenkel: Do not submit -- To view, visit http://gerrit.ovirt.org/8628 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8cb77b017282733d72ff66af499b576674674e01 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Ravi Nori <rnori@redhat.com>