Change in ovirt-engine[master]: core: Set default timezone to a Templates and VMs

shavivi at redhat.com shavivi at redhat.com
Tue Jun 3 07:55:16 UTC 2014


Shahar Havivi has submitted this change and it was merged.

Change subject: core: Set default timezone to a Templates and VMs
......................................................................


core: Set default timezone to a Templates and VMs

When adding a new template we don't set the default timezone which cause
the UI to display a blank timezone.

This patch also address the importing VM/Template and adding an upgrade
script to set the default timezone to an already created VMs/Templates
if its timezone is null.

Change-Id: Id55b02a10c2faa17a7a50b64516d902c3d97e06d
Bug-Url: https://bugzilla.redhat.com/1087917
Signed-off-by: Shahar Havivi <shaharh at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmTemplateCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmCommandTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmFromTemplateCommandTest.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java
A packaging/dbscripts/upgrade/03_05_0590_set_default_time_zone.sql
9 files changed, 60 insertions(+), 22 deletions(-)

Approvals:
  Shahar Havivi: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/28168
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id55b02a10c2faa17a7a50b64516d902c3d97e06d
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list