Change in ovirt-engine[master]: core:Enable update for storage domain name(#851835)
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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>