Change in ovirt-engine[master]: core: Editing of template version for pools

ofrenkel at redhat.com ofrenkel at redhat.com
Thu Jun 11 16:01:37 UTC 2015


Omer Frenkel has submitted this change and it was merged.

Change subject: core: Editing of template version for pools
......................................................................


core: Editing of template version for pools

--This patch includes messages and DB changes

The feature will allow a user to change a template version for an
existing Pool.

The main issues handled are:
- Pool object - added support for changing of the template field.
- Frontend - Added support for editing of this field and introduced
some limitations to avoid non-supported behaviors.
- UpdateVmVersion Command - added support for any template version.
- Pools commands - Until now pools haven't supported changes affecting the
actual VMs and some logic that depended on this assumption needed change.
- Running VMs - Changes were required in the next-run logic on multiple locations.

Change-Id: Ib106016955c702bc28f9347d828c4602e47bd233
Bug-Url: https://bugzilla.redhat.com/1140569
Signed-off-by: Eldan Shachar <eshachar at redhat.com>
---
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/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M packaging/dbscripts/vm_templates_sp.sql
6 files changed, 9 insertions(+), 1 deletion(-)

Approvals:
  Eldan Shachar: Verified
  Omer Frenkel: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib106016955c702bc28f9347d828c4602e47bd233
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eldan Shachar <eshachar at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eldan Shachar <eshachar at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list