Change in ovirt-engine[master]: engine: Block provide type change

masayag at redhat.com masayag at redhat.com
Thu Jun 4 10:57:14 UTC 2015


Moti Asayag has submitted this change and it was merged.

Change subject: engine: Block provide type change
......................................................................


engine: Block provide type change

There is no sense to modify the provider type, i.e. when
an OpenStack provider is created, the ovirt-engine creates
the required storage domain for it. Modifying the type of
a provider to a different type modifies the entire nature
of the provider.

Each provider has its own definition based on its type and
modifying the type will not be supported.

Change-Id: I3f5b0980977bc9fe556b1e3a72b1942bd8d86cdc
Bug-Url: https://bugzilla.redhat.com/1227320
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/UpdateProviderCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/EditProviderModel.java
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
6 files changed, 15 insertions(+), 1 deletion(-)

Approvals:
  Alona Kaplan: Looks good to me, but someone else must approve
  Moti Asayag: Verified; Looks good to me, approved
  Oved Ourfali: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3f5b0980977bc9fe556b1e3a72b1942bd8d86cdc
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list