Change in ovirt-engine[master]: webadmin: Set properties only for File SD.

mlipchuk at redhat.com mlipchuk at redhat.com
Thu Jan 7 09:40:42 UTC 2016


Maor Lipchuk has submitted this change and it was merged.

Change subject: webadmin: Set properties only for File SD.
......................................................................


webadmin: Set properties only for File SD.

The use of storageModel for block SD is null.
the GUI should set of Storage Type only for File Storage Domain
since the block Storage Domain is handled differently.
Changing the code to be used only when adding file SD.

Change-Id: I5cd6e0ea1209af0cdd3d5ea12d6108022115df30
Bug-Url: https://bugzilla.redhat.com/1295852
Bug-Url: https://bugzilla.redhat.com/1222571
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
1 file changed, 2 insertions(+), 6 deletions(-)

Approvals:
  Daniel Erez: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Maor Lipchuk: Verified



-- 
To view, visit https://gerrit.ovirt.org/51474
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5cd6e0ea1209af0cdd3d5ea12d6108022115df30
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list