Change in ovirt-engine[master]: webadmin: Default storage format once adding SD.

mlipchuk at redhat.com mlipchuk at redhat.com
Sun Sep 18 12:24:41 UTC 2016


Maor Lipchuk has submitted this change and it was merged.

Change subject: webadmin: Default storage format once adding SD.
......................................................................


webadmin: Default storage format once adding SD.

If a Data Center is in an uninitialized status its format type
is also uninitialized although the Data Center might not support
specific storage formats.
For example a v4.0 Data Center should not support v4 storage domains

Today, once a storage domain is being added to an uninitialized 4.0 Data
Center the default storage format will be V4 for new storage domains
since the Data Center's format type is still un-initialized,
and the add operation will fail for a reason of unsupported version format.

The proposed fix sets the Data Center's format type once it is uninitialized
based on its compatibility version

Change-Id: Ia117efc7369056f70854b8bbfbb6dd7dec7ce318
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageModel.java
1 file changed, 11 insertions(+), 7 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Daniel Erez: Looks good to me, approved
  Allon Mureinik: Looks good to me, but someone else must approve
  Maor Lipchuk: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia117efc7369056f70854b8bbfbb6dd7dec7ce318
Gerrit-PatchSet: 6
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: 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