Liron Ar has posted comments on this change.
Change subject: engine: check active vds before activate storage domain
......................................................................
Patch Set 3:
(2 comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ActivateStorageDomainCommand.java
Line 58:
VdcBllMessages.ACTION_TYPE_FAILED_STORAGE_POOL_STATUS_ILLEGAL)
Line 59: && checkStorageDomain()
Line 60: && storageDomainStatusIsValid()
Line 61: && (getStorageDomain().getStorageDomainType() ==
StorageDomainType.Master || checkMasterDomainIsUp())
Line 62: && checkActiveVdsInStoragePool();
In StorageCommandHandlingBase there's the method initializeVds() that basically
is being used for what we need here, perhaps you could use it as well (with
slight modifications).
Line 63: return returnValue;
Line 64: }
Line 65:
Line 66: @Override
....................................................
File backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
Line 479: VM_CANNOT_REMOVE_WITH_DETACH_DISKS_SNAPSHOTS_EXIST=Cannot ${action}
${type} with detaching disks, snapshots exist.
Line 480: VM_CANNOT_REMOVE_WITH_DETACH_DISKS_BASED_ON_TEMPLATE=Cannot ${action}
${type} with detaching disks, VM is based from template.
Line 481: ACTION_TYPE_FAILED_MASTER_STORAGE_DOMAIN_NOT_ACTIVE=Failed to
${action} ${type} due to an error on the Data Center master Storage Domain.\n\
Line 482: -Please activate the master Storage Domain first.
Line 483: ACTION_TYPE_FAILED_NO_ACTIVE_HOST_IN_STORAGE_POOL=Failed to ${action}
${type}. No active host found. Please make sure there is at least one 'Up' host
in Data Center.
Just a note - apperantely there is already a message for that scenario-
ACTION_TYPE_FAILED_NO_VDS_IN_POOL
Line 484: TAGS_SPECIFY_TAG_IS_NOT_EXISTS=The specified Tag does not exist.
Line 485: NETWORK_CANNOT_REMOVE_NETWORK_IN_USE_BY_TEMPLATE=Cannot ${action}
${type}. The Network is in use by a Template.\n\
Line 486: Network definition cannot be detached unless all references to
it are cleared.
Line 487: NETWORK_CANNOT_CHANGE_STATUS_WHEN_NOT_DOWN_UP=Cannot ${action}
${type}. Changing interface properties is allowed only when the VM is in 'Up'
or 'Down' status.
--
To view, visit http://gerrit.ovirt.org/17940
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia3e45b53d15b757237af8c76457f213c00a73909
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: plysan <ply...@gmail.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Itamar Heim <ih...@redhat.com>
Gerrit-Reviewer: LiJiansheng <lijiangshe...@gmail.com>
Gerrit-Reviewer: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-Reviewer: plysan <ply...@gmail.com>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
