Change in ovirt-engine[master]: core: make the getTimeZone query OS bounded

rgolan at redhat.com rgolan at redhat.com
Sun Jul 1 10:14:35 UTC 2012


Roy Golan has submitted this change and it was merged.

Change subject: core: make the getTimeZone query OS bounded
......................................................................


core: make the getTimeZone query OS bounded

this query will return different timezone for windows OS (the default
and old behaviour) otherwise the java timezone lists
as maintained in java.util.TimeZone see http://doc.java.sun.com/DocWeb/api/java.util.TimeZone.

purpose of this change is to support setting timezone for all VMs.
note: the timezone is used to set the hardware clock of the virtual bios and
not the OS timezone, which is done by sysprep for windows and future
sysprep-like for linux.

Change-Id: Ie4389f058d0a451ba4f7b9e50daab8fab87df450
Signed-off-by: Roy Golan <rgolan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetTimeZonesQuery.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetTimeZonesQueryTest.java
A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/TimeZoneQueryParams.java
M backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/WindowsJavaTimezoneMapping.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilderBase.java
5 files changed, 117 insertions(+), 28 deletions(-)

Approvals:
  Roy Golan: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie4389f058d0a451ba4f7b9e50daab8fab87df450
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>



More information about the Engine-commits mailing list