fix(stackdriver): fix millis_to_time so it works on developer machine…

…s that use localized time. (#219) The tests were passing in CI but failing locally. Ensuring the timestamp is using UTC instead of localized time fixes the inconsistency.
spinnaker · Sep 18, 2019 · acb2028 · acb2028
1 parent 922385d
commit acb2028
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/spinnaker-monitoring-daemon/spinnaker-monitoring/google_service.py b/spinnaker-monitoring-daemon/spinnaker-monitoring/google_service.py
@@ -209,7 +209,8 @@ class GoogleMonitoringService(object):
 
   @staticmethod
   def millis_to_time(millis):
-    return datetime.fromtimestamp(millis // 1000).isoformat('T') + 'Z'
+    # matches expected format StackdriverSurveyor.__date_format = '%Y-%m-%dT%H:%M:%SZ'
+    return datetime.utcfromtimestamp(millis // 1000).strftime("%Y-%m-%dT%H:%M:%SZ")
 
   @property
   def project(self):