Allon Mureinik has submitted this change and it was merged. Change subject: core: fix compensation mechanism in RefreshLunsSizeCommand ...................................................................... core: fix compensation mechanism in RefreshLunsSizeCommand This patch fixes two issues in RefreshLunsSizeCommand's compensation mechanism: 1. RefreshLunsSizeCommand doesn't contain the required compensation constructor. 2. The updateStorageDomain method takes a snapshot of the new storage domain dynamic object instead of the old one, which leads the compensation of the same object in the db, instead of the old one. Change-Id: I60d802529533aa073b178c273f4f21c89975b5ea Bug-Url: https://bugzilla.redhat.com/1370912 Signed-off-by: Idan Shaby <ishaby@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/RefreshLunsSizeCommand.java 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/62887 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I60d802529533aa073b178c273f4f21c89975b5ea Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>