Change in ovirt-engine[master]: engine: StoragePool storage_pool_format_type is lost in upda...

asaf at redhat.com asaf at redhat.com
Sun Aug 19 13:46:52 UTC 2012


Asaf Shakarchi has submitted this change and it was merged.

Change subject: engine: StoragePool storage_pool_format_type is lost in update (#845310)
......................................................................


engine: StoragePool storage_pool_format_type is lost in update (#845310)

https://bugzilla.redhat.com/845310

StoragePool's storage_pool_format_type property is not set in
params entity (as it's not visible in UI) so the value is lost
during storage pool update,

This patch copies the property value from the old entity.

Change-Id: I4e19bc2102e40c818b8953a08d1203c3a9b042b0
Signed-off-by: Asaf Shakarchi <asaf at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStoragePoolCommand.java
1 file changed, 13 insertions(+), 0 deletions(-)

Approvals:
  Asaf Shakarchi: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/7265
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4e19bc2102e40c818b8953a08d1203c3a9b042b0
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Asaf Shakarchi <asaf at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Asaf Shakarchi <asaf at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list