Daniel Erez has uploaded a new change for review. Change subject: webadmin: grayed-out LUNs reasons - modify check ......................................................................
webadmin: grayed-out LUNs reasons - modify check Verify that the selected LUN is not apart of a DirectLUN disk first; since a LUN for DirectLUN disk is grayed-out only if it's already apart of a Disk in the system. Change-Id: Ie5d044ed4a75182954bd4a01b5cb071f24da6bd1 Signed-off-by: Daniel Erez <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java 1 file changed, 5 insertions(+), 5 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/04/12904/1 diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java index 2761c6f..a633459 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java @@ -289,14 +289,14 @@ lunModel.setIsGrayedOut(isIgnoreGrayedOut() ? lun.getDiskId() != null : nonEmpty); // Adding 'GrayedOutReasons' - if (lun.getStorageDomainId() != null) { - lunModel.getGrayedOutReasons().add( - messages.lunAlreadyPartOfStorageDomainWarning(lun.getStorageDomainName())); - } - else if (lun.getDiskId() != null) { + if (lun.getDiskId() != null) { lunModel.getGrayedOutReasons().add( messages.lunUsedByDiskWarning(lun.getDiskAlias())); } + else if (lun.getStorageDomainId() != null) { + lunModel.getGrayedOutReasons().add( + messages.lunAlreadyPartOfStorageDomainWarning(lun.getStorageDomainName())); + } else if (lun.getStatus() == LunStatus.Unusable) { lunModel.getGrayedOutReasons().add( constants.lunUnusable()); -- To view, visit http://gerrit.ovirt.org/12904 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ie5d044ed4a75182954bd4a01b5cb071f24da6bd1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
