837301 - [pulsetranslator] Change timestamps to make use of UTC time

Status: RESOLVED FIXED

(Webtools :: Pulse, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Right now it's kinda hard to work with Pulse messages because the original ones provide times in the Pacific time zone. Via pulsetranslator we get an additional timestamp in the local timezone. That means if the translator gets run in another timezone as Pacific we don't know how to translate those time zones. 

Therefor we might want to use the UTC time zone whereever we can.

As Jonathan mentioned on bug 835749 comment 2 we do not know yet which consumers might be affected by that change. So we should handle that carefully and get an idea first what we could break.

Who knows about all the consumers we make use of internally?

Comment 1

[Jonathan Griffin (:jgriffin)]

The two consumers we need to check, that I know of, are the orange factor logparser (http://hg.mozilla.org/automation/logparser/), and autophone (https://github.com/mozilla/autophone)

Comment 2

[Jonathan Griffin (:jgriffin)]

Specifically, we need to determine if any consumers rely on the 'timestamp' field being in PST.

* The OF logparser does not use this field.
* Autolog does not use this field.

We should be good to change this field to UTC.

Comment 3

[Jonathan Griffin (:jgriffin)]

Attachment: Make timestamp UTC

I've verified there aren't any consumers that depend on local time in this field.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Comment on attachment 719079 [details] [diff] [review]
Make timestamp UTC

Review of attachment 719079 [details] [diff] [review]:
-----------------------------------------------------------------

Yay, for an universal time! Looks great and it will help us a lot. Thanks!

Comment 5

[Jonathan Griffin (:jgriffin)]

https://github.com/mozilla/pulsetranslator/commit/0991a23d8b4936ed541ae177120f80c511e8e996

Comment 6

[Jonathan Griffin (:jgriffin)]

deployed