Martin Betak has uploaded a new change for review.

Change subject: core: Fix mapping of US Eastern Standard Time Zone
......................................................................

core: Fix mapping of US Eastern Standard Time Zone

The WindowsJavaTimezoneMapping mapped US East. ST to java time zone
GMT+5 which was not adjusted to Daylight Savings Time.
Now we map it to America/New_York which takes DST into account.

Also fixed another occurence of mapping to raw GMT timezone:
SA Eastern Standard Time to America/Buenos_Aires (was GMT+3) but this
change was not as consequential since Argentina does not have DST.
Although this should be more resilient if they decided to use it in the
future.

Similar story for Georia -> Asia/Tbilisi.

Change-Id: I0d17376caac96d1aae87cc894452d42e161637e4
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1159849
Signed-off-by: Martin Betak <mbe...@redhat.com>
---
M 
backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/WindowsJavaTimezoneMapping.java
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/58/34758/1

diff --git 
a/backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/WindowsJavaTimezoneMapping.java
 
b/backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/WindowsJavaTimezoneMapping.java
index f0e4df0..aba896f 100644
--- 
a/backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/WindowsJavaTimezoneMapping.java
+++ 
b/backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/WindowsJavaTimezoneMapping.java
@@ -27,7 +27,7 @@
             { "Egypt Standard Time", "Africa/Cairo" }, { "Ekaterinburg 
Standard Time", "Asia/Yekaterinburg" },
             { "FLE Standard Time", "Europe/Kiev" }, { "Fiji Standard Time", 
"Pacific/Fiji" },
             { "GMT Standard Time", "Europe/London" }, { "GTB Standard Time", 
"Europe/Istanbul" },
-            { "Georgian Standard Time", "Etc/GMT-3" }, { "Greenland Standard 
Time", "America/Godthab" },
+            { "Georgian Standard Time", "Asia/Tbilisi" }, { "Greenland 
Standard Time", "America/Godthab" },
             { "Greenwich Standard Time", "Africa/Reykjavik" }, { "Hawaiian 
Standard Time", "Pacific/Honolulu" },
             { "India Standard Time", "Asia/Calcutta" }, { "Iran Standard 
Time", "Asia/Tehran" },
             { "Israel Standard Time", "Asia/Jerusalem" }, { "Jordan Standard 
Time", "Asia/Amman" },
@@ -43,13 +43,13 @@
             { "Pacific SA Standard Time", "America/Santiago" }, { "Pacific 
Standard Time", "America/Los_Angeles" },
             { "Pacific Standard Time", "America/Tijuana" }, { "Pakistan 
Standard Time", "Asia/Karachi" },
             { "Romance Standard Time", "Europe/Paris" }, { "Russian Standard 
Time", "Europe/Moscow" },
-            { "SA Eastern Standard Time", "Etc/GMT+3" }, { "SA Pacific 
Standard Time", "America/Bogota" },
+            { "SA Eastern Standard Time", "America/Buenos_Aires" }, { "SA 
Pacific Standard Time", "America/Bogota" },
             { "SA Western Standard Time", "America/La_Paz" }, { "SE Asia 
Standard Time", "Asia/Bangkok" },
             { "Samoa Standard Time", "Pacific/Apia" }, { "Singapore Standard 
Time", "Asia/Singapore" },
             { "South Africa Standard Time", "Africa/Johannesburg" }, { "Sri 
Lanka Standard Time", "Asia/Colombo" },
             { "Taipei Standard Time", "Asia/Taipei" }, { "Tasmania Standard 
Time", "Australia/Hobart" },
             { "Tokyo Standard Time", "Asia/Tokyo" }, { "Tonga Standard Time", 
"Pacific/Tongatapu" },
-            { "US Eastern Standard Time", "Etc/GMT+5" }, { "US Mountain 
Standard Time", "America/Phoenix" },
+            { "US Eastern Standard Time", "America/New_York" }, { "US Mountain 
Standard Time", "America/Phoenix" },
             { "Venezuela Standard Time", "America/Caracas" }, { "Vladivostok 
Standard Time", "Asia/Vladivostok" },
             { "W. Australia Standard Time", "Australia/Perth" }, { "W. Central 
Africa Standard Time", "Africa/Lagos" },
             { "W. Europe Standard Time", "Europe/Berlin" }, { "West Asia 
Standard Time", "Asia/Tashkent" },


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0d17376caac96d1aae87cc894452d42e161637e4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Martin Betak <mbe...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to