Change in ovirt-engine[master]: core:Enable update for storage domain name(#851835)

[ofrenkel at redhat.com]

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 <mlipchuk at 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 <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>