Summary: | [ews-app] Timestamp shown in status-bubble hover-over message is in future | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||||||
Component: | Tools / Tests | Assignee: | Aakash Jain <aakash_jain> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | aakash_jain, ap, commit-queue, jbedard, kocsen_chung, lforschler, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | Other | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=208920 | ||||||||||
Attachments: |
|
This seems to be due to incorrect timezone (UTC) in ews-app's settings.py Created attachment 367013 [details]
Patch
Comment on attachment 367013 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=367013&action=review > Tools/BuildSlaveSupport/ews-app/ews-app/settings.py:142 > -TIME_ZONE = 'UTC' > +TIME_ZONE = 'America/Los_Angeles' What is the benefit of changing time zone here, not in UI? Ideally, we would be showing the correct time to everyone, which can only be done in client side JavaScript. Created attachment 393771 [details]
Patch
I finally got to the bottom of this issue. We are sending incorrect timestamp from buildbot. We are converting the timestamp to local time (pst) while generating epoch timestamp. While generating status-bubble (in django app) we interpret the timestamp as UTC time, which makes the timestamp in future. Previous patch was just a workaround. Updated patch fixes the root-cause. Tested on uat instance. Committed r258586: <https://trac.webkit.org/changeset/258586> All reviewed patches have been landed. Closing bug and clearing flags on attachment 393771 [details]. |
Created attachment 366761 [details] Screenshot indicating the issue Timestamp shown in ews-app's status-bubble hover-over message is in future. For example in the attached screenshot, the status-bubble is for currently in-progress build. It shows a timestamp of "April 05 2019 01:28:13 GMT -0400 (EDT)". This seems to be 8 hours ahead of the actual time.