[GitHub] [cloudstack] andrijapanicsb commented on pull request #4055: db.properties: Enforce UTC timezone by default

[GitBox <gi...@apache.org>]

andrijapanicsb commented on pull request #4055:
URL: https://github.com/apache/cloudstack/pull/4055#issuecomment-624290713


   Alright....
   
   tested this thoroughly enough, I believe:
   
   ## ACS 4.13, clean install (no db.properties customisation), usage job runs in CEST!
   
   - UTC timezone on mgmt server = all times in DB (cloud and cloud_usage!) are in UTC, but i.e. events in GUI displayed with my local time (of my laptop, which is CEST - from my perspective that is how it should be)
   - Changed mgmt server time to CEST, rebooted server (for fun) = no changes in DB/GUI
   
   ## Upgraded to 4.14 with adding "serverTimezone=UTC" so that mgmt starts fine
   - all times still in UTC in DBs (no change to before) = good/consistency
   - changed db.properties to serverTimezone=Europe/Belgrade on both lines (this is CEST actually) 
     - no changes in DB at all...I'm talking both about events in "cloud" and the usage records (both raw and digested in cloud_usage table) - all stay in UTC times - i.e. DB is not affected by this change - which is funny - but I'm OK with this
     - GUI is now showing times from UTC (i.e. my time minus 2h) - which is wrong - i.e. we are not really honouring the timezone in DB records, but it IS reflected in GUI - this is not optimal - but is also NOT critical whatsoever...
   
   /cc @rhtyd @DaanHoogland @Doni7722 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org