Omer Frenkel has submitted this change and it was merged. Change subject: core:Enable update for storage domain name(#851835) ......................................................................
core:Enable update for storage domain name(#851835) https://bugzilla.redhat.com/show_bug.cgi?id=851835 Update storage domain name fails since lastTimeUsedAsMaster property is sent from the client is not synchronized with the DB value. This is because this field is not reflected to the user and it is internally used in the engine. The proposed fix is to update the field with the old value from the DB before update the storage domain in the DB. Change-Id: I77de69fb8bd28a3c2c1ec1a9be576cb7b4e9dd69 Signed-off-by: Maor Lipchuk <mlipc...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStorageDomainCommand.java 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: Omer Frenkel: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit http://gerrit.ovirt.org/7598 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77de69fb8bd28a3c2c1ec1a9be576cb7b4e9dd69 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipc...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Ayal Baron <aba...@redhat.com> Gerrit-Reviewer: Daniel Erez <de...@redhat.com> Gerrit-Reviewer: Liron Aravot <lara...@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
